Re: [PATCH v2 0/3] MR8761: mshtml: Fire BeforeNavigate2 for documents in async_open.

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/mshtml/nsio.c:
> -
> - hres = IUri_GetDisplayUri(This->uri->uri, &display_uri);
> - if (FAILED(hres))
> -
>    return NS_ERROR_UNEXPECTED;
> - hres = IHTMLWindow2_get_name(&window->base.IHTMLWindow2_iface, &frame_name);
> - if (FAILED(hres))
> - {
> -
>    SysFreeString(display_uri);
> -
>    return NS_ERROR_UNEXPECTED;
> - }
> - hres = IHTMLWindow2_QueryInterface(&window->base.IHTMLWindow2_iface, &IID_IServiceProvider, (void**)&service_provider);
> - if (SUCCEEDED(hres))
> - {
> -
>    hres = IServiceProvider_QueryService(service_provider, &SID_SWebBrowserApp, &IID_IWebBrowser2, (void**)&web_browser);
> -
>    IServiceProvider_Release(service_provider);
> - }

Could we use `window->browser->doc->doc_object_service` instead here?