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.