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.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5156#note_62841