Re: [PATCH v2 0/4] MR5156: mshtml: Always use the event target dispex

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/mshtml/htmldoc.c:
>                              0, flags, dp, res, ei, caller);
>
> }
>
> +static HRESULT HTMLDocumentNode_pre_handle_event(DispatchEx* dispex, eventid_t eid, nsIDOMEvent *event)
> +{
> - HTMLDocumentNode *doc = impl_from_DispatchEx(dispex);
> - switch(eid) {
> - case EVENTID_DOMCONTENTLOADED: {
> -
>    if(doc->window)
> -
>        doc->window->dom_content_loaded_event_start_time = get_time_stamp();

This event may be fired by a script, so we should probably check if it's the real event here. This could be done using nsIDOMEvent::GetIsTrusted or, perhaps better, pass event as DOMEvent (instead of nsIDOMEvent) and just check event->trusted.