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

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/mshtml/nsio.c:
>              This->content_type = strdupWtoA(window->browser->doc->mime);
>          }
>      }
> -
> -
>    if (!cancel && !This->uri->channel_bsc && window->browser->doc->doc_object_service)
> -
>    {
> -
>        /* fire event for document navigations by Gecko */
> -
>        hres = IUri_GetDisplayUri(This->uri->uri, &uri_str);
> -
>        if (SUCCEEDED(hres))
> -
>        {
> -
>            hres = fire_beforenavigate2(window, uri_str);
> -
>            SysFreeString(uri_str);
> -
>            if (FAILED(hres))
> -
>                cancel = TRUE;

Maybe we should just pass `&cancel` pointer to `fire_beforenavigate2`.