Re: [PATCH v2 0/4] MR1373: mshtml: Page-wide events implementation.

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/mshtml/nsevents.c:
>          dispatch_event(&doc->window->event_target, load_event);
>          IDOMEvent_Release(&load_event->IDOMEvent_iface);
>      }
> -
> -
>    if(doc->dom_document && doc->document_mode >= COMPAT_MODE_IE11) {
> -
>        hres = create_document_event(doc, EVENTID_PAGESHOW, &load_event);
> -
>        if(SUCCEEDED(hres)) {
> -
>            dispatch_event(&doc->window->event_target, load_event);
> -
>            IDOMEvent_Release(&load_event->IDOMEvent_iface);
> -
>        }
> -
>    }

It seems that Gecko implements this event (and others in the series), could we just use that instead? As far as I can see we currently don't handle EVENT_BIND_TO_TARGET for window events, but that should be easy to change. Maybe we should even use window object instead of document for EVENT_DEFAULTLISTENER?