Re: [4/4] urlmon: Added basic MapUrlToZoneEx2 implementation

[Jacek Caban]

Hi Thomas,

On 08/24/11 01:58, Thomas Mullaly wrote:
> +static HRESULT map_uri_to_zone(IUri *uri, DWORD *zone)
> +{
> +    HRESULT hres;
> +    BOOL has_scheme = FALSE;
> +
> +    *zone = URLZONE_INVALID;
> +
> +    hres = IUri_HasProperty(uri, Uri_PROPERTY_SCHEME_NAME,&has_scheme);
> +    if(hres != S_OK)
> +        return hres;
> +
> +    if(has_scheme) {
> +        BSTR display_uri;
> +
> +        hres = IUri_GetDisplayUri(uri,&display_uri);
> +        if(FAILED(hres))
> +            return hres;
> +
> +        hres = map_url_to_zone(display_uri, zone, NULL);

I think map_url_to_zone should be integrated to map_uri_to_zone and 
operate on IUri instead of URL string (ideally we should get rid of 
map_url_to_zone once transition to IUri-based API will be done), not the 
other way around.

Jacek