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