Re: [PATCH v4 0/8] MR2098: mshtml: Implement synchronous XMLHttpRequests.

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/mshtml/tests/events.c:
> HRESULT hres;
>      BSTR bstr;
>
> -    hres = IInternetProtocolSink_QueryInterface(This->sink, &IID_IServiceProvider, (void**)&service_provider);
> -    ok(hres == S_OK, "Could not get IServiceProvider iface: %08lx\n", hres);
> +    if(This->delay >= 0) {

Checking delay here is a bit weird, maybe a dedicated variable to store an info if any response on this handler was already reported would be cleaner.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/2098#note_25027