Re: [PATCH v5 0/5] MR6375: mshtml: Don't process tasks recursively.

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/mshtml/tests/events.c:
> - V_VT(&v) = VT_EMPTY;
> - bstr = SysAllocString(L"about:blank");
> - bstr2 = SysAllocString(L"");
> - hres = IHTMLWindow2_QueryInterface(window, &IID_IHTMLPrivateWindow, (void**)&priv_window);
> - ok(hres == S_OK, "Could not get IHTMLPrivateWindow) interface: %08lx\n", hres);
> - hres = IHTMLPrivateWindow_SuperNavigate(priv_window, bstr, bstr2, NULL, NULL, &v, &v, 0);
> - ok(hres == S_OK, "SuperNavigate failed: %08lx\n", hres);
> - IHTMLPrivateWindow_Release(priv_window);
> - IHTMLWindow2_Release(window);
> - SysFreeString(bstr2);
> - SysFreeString(bstr);
> -
> - ok(nav_notif_test_depth == 1, "nav_notif_test_depth = %u\n", nav_notif_test_depth);
> - pump_msgs(NULL);
> - pump_msgs(NULL);
> - pump_msgs(NULL);

This looks weird, a single `pump_msgs` should be enough. I guess you meant some sort of larger timeout here, but a timeout of three `PeekMessage` calls overhead doesn't make much sense.