March 2021 - Wine-commits - list.winehq.org

Francois Gouget : testbot/TestWTBS: Don't expect CVS revisions on the test report start lines.
by Alexandre Julliard 26 Mar '21

26 Mar '21

Module: tools Branch: master Commit: 83da6130c431dc501324faa996907eba07368d1f URL: https://source.winehq.org/git/tools.git/?a=commit;h=83da6130c431dc501324faa… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Fri Mar 26 04:19:34 2021 +0100 testbot/TestWTBS: Don't expect CVS revisions on the test report start lines. Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- testbot/tests/TestWTBS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/testbot/tests/TestWTBS b/testbot/tests/TestWTBS index 81b56c4..b76aad5 100755 --- a/testbot/tests/TestWTBS +++ b/testbot/tests/TestWTBS @@ -495,7 +495,7 @@ sub GetReportTestUnits($) { foreach my $Line (<$LogFile>) { - if ($Line =~ m%^([_.a-z0-9-]+:[_a-z0-9]*) start (?:-|[/_.a-z0-9-]+) (?:-|[.0-9a-f]+)%) + if ($Line =~ m%^([_.a-z0-9-]+:[_a-z0-9]*) start (?:-|[/_.a-z0-9-]+)%) { $TestUnits->{$1} = 1; }

1 0

Francois Gouget : testbot/TestWTBS: Fix a couple of typos in the documentation.
by Alexandre Julliard 26 Mar '21

26 Mar '21

Module: tools Branch: master Commit: 4bd805aad4d35dd070009c4d618db588f74c74ea URL: https://source.winehq.org/git/tools.git/?a=commit;h=4bd805aad4d35dd070009c4… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Fri Mar 26 17:13:36 2021 +0100 testbot/TestWTBS: Fix a couple of typos in the documentation. Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- testbot/tests/TestWTBS | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/testbot/tests/TestWTBS b/testbot/tests/TestWTBS index b76aad5..8067a62 100755 --- a/testbot/tests/TestWTBS +++ b/testbot/tests/TestWTBS @@ -675,8 +675,8 @@ there is no error one would declare an empty group such as below, though a g 0 tests.report.kernel32 Finally, note that while the error directives are inherited by subcategories, -they are not merged. So if if both win and win32 have error-matching -directives, win32 will not inherit anything from the win category. +they are not merged. So if both win and win32 have error-matching directives, +win32 will not inherit anything from the win category. =item <tasks.(report|log|testbot).Errors> @@ -873,8 +873,8 @@ Checks that the TestBot applied the patch and did not find it necessary to run make_requests and make_opengl. Finally, note that while the Grep* directives are inherited by subcategories, -they are not merged. So if if both win and win32 have Grep directives, win32 -will not inherit anything from the win category. +they are not merged. So if both win and win32 have Grep directives, win32 will +not inherit anything from the win category. =cut

1 0

Francois Gouget : testbot/TestLauncher: Fix the spelling of an error message.
by Alexandre Julliard 26 Mar '21

26 Mar '21

Module: tools Branch: master Commit: 71ec3a5096711cf2a4cb19f0b4412e7e1194b87f URL: https://source.winehq.org/git/tools.git/?a=commit;h=71ec3a5096711cf2a4cb19f… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Fri Mar 26 17:13:53 2021 +0100 testbot/TestLauncher: Fix the spelling of an error message. Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- testbot/src/TestLauncher/TestLauncher.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/testbot/src/TestLauncher/TestLauncher.c b/testbot/src/TestLauncher/TestLauncher.c index 58eed76..2a423d2 100644 --- a/testbot/src/TestLauncher/TestLauncher.c +++ b/testbot/src/TestLauncher/TestLauncher.c @@ -180,7 +180,7 @@ int main(int argc, char *argv[]) { if (GetFullPathNameA(argv[Arg], ARRAY_SIZE(TestExeFullName), TestExeFullName, &TestExeFileName) == 0) { - Error("Cauld not get the test executable full path %s (error %lu)\n", + Error("Could not get the test executable full path %s (error %lu)\n", argv[Arg], GetLastError()); return 1; }

1 0

Nikolay Sivov : rtworkq: Partially implement shared queues.
by Alexandre Julliard 25 Mar '21

25 Mar '21

Module: wine Branch: master Commit: d1d1e3f56ffa1beab271fcd5b2e4c7a18294a772 URL: https://source.winehq.org/git/wine.git/?a=commit;h=d1d1e3f56ffa1beab271fcd5… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Thu Mar 25 19:06:19 2021 +0300 rtworkq: Partially implement shared queues. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mfplat/mfplat.spec | 1 + dlls/mfplat/tests/mfplat.c | 46 ++++++++++++++++++++++++++++++++++++++++++++++ dlls/rtworkq/queue.c | 34 ++++++++++++++++++++++++++++++++-- include/mfapi.h | 1 + 4 files changed, 80 insertions(+), 2 deletions(-) diff --git a/dlls/mfplat/mfplat.spec b/dlls/mfplat/mfplat.spec index 7d014e334c8..cbda9404aa9 100644 --- a/dlls/mfplat/mfplat.spec +++ b/dlls/mfplat/mfplat.spec @@ -132,6 +132,7 @@ @ stub MFJoinIoPort @ stdcall MFJoinWorkQueue(long long ptr) rtworkq.RtwqJoinWorkQueue @ stdcall MFLockPlatform() rtworkq.RtwqLockPlatform +@ stdcall MFLockSharedWorkQueue(wstr long ptr ptr) rtworkq.RtwqLockSharedWorkQueue @ stdcall MFLockWorkQueue(long) rtworkq.RtwqLockWorkQueue @ stdcall MFMapDX9FormatToDXGIFormat(long) @ stdcall MFMapDXGIFormatToDX9Format(long) diff --git a/dlls/mfplat/tests/mfplat.c b/dlls/mfplat/tests/mfplat.c index 18361d4d721..1b24dce2fe0 100644 --- a/dlls/mfplat/tests/mfplat.c +++ b/dlls/mfplat/tests/mfplat.c @@ -240,6 +240,7 @@ static HRESULT (WINAPI *pMFCreateVideoSampleAllocatorEx)(REFIID riid, void **all static HRESULT (WINAPI *pMFCreateDXGISurfaceBuffer)(REFIID riid, IUnknown *surface, UINT subresource, BOOL bottomup, IMFMediaBuffer **buffer); static HRESULT (WINAPI *pMFCreateVideoMediaTypeFromSubtype)(const GUID *subtype, IMFVideoMediaType **media_type); +static HRESULT (WINAPI *pMFLockSharedWorkQueue)(const WCHAR *name, LONG base_priority, DWORD *taskid, DWORD *queue); static HWND create_window(void) { @@ -907,6 +908,7 @@ static void init_functions(void) X(MFCreateVideoSampleAllocatorEx); X(MFGetPlaneSize); X(MFGetStrideForBitmapInfoHeader); + X(MFLockSharedWorkQueue); X(MFMapDX9FormatToDXGIFormat); X(MFMapDXGIFormatToDX9Format); X(MFPutWaitingWorkItem); @@ -7016,6 +7018,49 @@ done: DestroyWindow(window); } +static void test_MFLockSharedWorkQueue(void) +{ + DWORD taskid, queue, queue2; + HRESULT hr; + + if (!pMFLockSharedWorkQueue) + { + win_skip("MFLockSharedWorkQueue() is not available.\n"); + return; + } + + hr = MFStartup(MF_VERSION, MFSTARTUP_FULL); + ok(hr == S_OK, "Failed to start up, hr %#x.\n", hr); + + hr = pMFLockSharedWorkQueue(NULL, 0, &taskid, &queue); + ok(hr == E_POINTER, "Unexpected hr %#x.\n", hr); + + hr = pMFLockSharedWorkQueue(NULL, 0, NULL, &queue); + ok(hr == E_POINTER, "Unexpected hr %#x.\n", hr); + + taskid = 0; + hr = pMFLockSharedWorkQueue(L"", 0, &taskid, &queue); + ok(hr == E_INVALIDARG, "Unexpected hr %#x.\n", hr); + + queue = 0; + hr = pMFLockSharedWorkQueue(L"", 0, NULL, &queue); + ok(queue & MFASYNC_CALLBACK_QUEUE_PRIVATE_MASK, "Unexpected queue id.\n"); + + queue2 = 0; + hr = pMFLockSharedWorkQueue(L"", 0, NULL, &queue2); + ok(hr == S_OK, "Unexpected hr %#x.\n", hr); + ok(queue == queue2, "Unexpected queue %#x.\n", queue2); + + hr = MFUnlockWorkQueue(queue2); + ok(hr == S_OK, "Unexpected hr %#x.\n", hr); + + hr = MFUnlockWorkQueue(queue); + ok(hr == S_OK, "Unexpected hr %#x.\n", hr); + + hr = MFShutdown(); + ok(hr == S_OK, "Failed to shut down, hr %#x.\n", hr); +} + START_TEST(mfplat) { char **argv; @@ -7044,6 +7089,7 @@ START_TEST(mfplat) test_source_resolver(); test_MFCreateAsyncResult(); test_allocate_queue(); + test_MFLockSharedWorkQueue(); test_MFCopyImage(); test_MFCreateCollection(); test_MFHeapAlloc(); diff --git a/dlls/rtworkq/queue.c b/dlls/rtworkq/queue.c index f4e9f10d3f0..6817b06f284 100644 --- a/dlls/rtworkq/queue.c +++ b/dlls/rtworkq/queue.c @@ -58,6 +58,7 @@ static struct queue_handle user_queues[MAX_USER_QUEUE_HANDLES]; static struct queue_handle *next_free_user_queue; static struct queue_handle *next_unused_user_queue = user_queues; static WORD queue_generation; +static DWORD shared_mt_queue; static CRITICAL_SECTION queues_section; static CRITICAL_SECTION_DEBUG queues_critsect_debug = @@ -217,6 +218,7 @@ static HRESULT unlock_user_queue(DWORD queue) { if (--entry->refcount == 0) { + if (shared_mt_queue == queue) shared_mt_queue = 0; shutdown_queue((struct queue *)entry->obj); heap_free(entry->obj); entry->obj = next_free_user_queue; @@ -1438,9 +1440,37 @@ HRESULT WINAPI RtwqSetLongRunning(DWORD queue_id, BOOL enable) HRESULT WINAPI RtwqLockSharedWorkQueue(const WCHAR *usageclass, LONG priority, DWORD *taskid, DWORD *queue) { - FIXME("%s, %d, %p, %p.\n", debugstr_w(usageclass), priority, taskid, queue); + struct queue_desc desc; + HRESULT hr; + + TRACE("%s, %d, %p, %p.\n", debugstr_w(usageclass), priority, taskid, queue); + + if (!usageclass) + return E_POINTER; + + if (!*usageclass && taskid) + return E_INVALIDARG; + + if (*usageclass) + FIXME("Class name is ignored.\n"); + + EnterCriticalSection(&queues_section); - return RtwqAllocateWorkQueue(RTWQ_STANDARD_WORKQUEUE, queue); + if (shared_mt_queue) + hr = lock_user_queue(shared_mt_queue); + else + { + desc.queue_type = RTWQ_MULTITHREADED_WORKQUEUE; + desc.ops = &pool_queue_ops; + desc.target_queue = 0; + hr = alloc_user_queue(&desc, &shared_mt_queue); + } + + *queue = shared_mt_queue; + + LeaveCriticalSection(&queues_section); + + return hr; } HRESULT WINAPI RtwqSetDeadline(DWORD queue_id, LONGLONG deadline, HANDLE *request) diff --git a/include/mfapi.h b/include/mfapi.h index d6969984da7..c1cf766c005 100644 --- a/include/mfapi.h +++ b/include/mfapi.h @@ -553,6 +553,7 @@ HRESULT WINAPI MFInitAttributesFromBlob(IMFAttributes *attributes, const UINT8 * HRESULT WINAPI MFInitMediaTypeFromWaveFormatEx(IMFMediaType *mediatype, const WAVEFORMATEX *format, UINT32 size); HRESULT WINAPI MFInvokeCallback(IMFAsyncResult *result); HRESULT WINAPI MFLockPlatform(void); +HRESULT WINAPI MFLockSharedWorkQueue(const WCHAR *name, LONG base_priority, DWORD *taskid, DWORD *queue); DXGI_FORMAT WINAPI MFMapDX9FormatToDXGIFormat(DWORD format); DWORD WINAPI MFMapDXGIFormatToDX9Format(DXGI_FORMAT dxgi_format); HRESULT WINAPI MFPutWaitingWorkItem(HANDLE event, LONG priority, IMFAsyncResult *result, MFWORKITEM_KEY *key);

1 0

Jacek Caban : mshtml/tests: Use winetest.js helpers in asyncscriptload.js.
by Alexandre Julliard 25 Mar '21

25 Mar '21

Module: wine Branch: master Commit: da30fa03574816cbd69c636ab39452ee9fe2882f URL: https://source.winehq.org/git/wine.git/?a=commit;h=da30fa03574816cbd69c636a… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Mar 25 19:28:04 2021 +0100 mshtml/tests: Use winetest.js helpers in asyncscriptload.js. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mshtml/tests/asyncscriptload.js | 19 +++++++------------ 1 file changed, 7 insertions(+), 12 deletions(-) diff --git a/dlls/mshtml/tests/asyncscriptload.js b/dlls/mshtml/tests/asyncscriptload.js index a73e5e849e8..6dc333afdf4 100644 --- a/dlls/mshtml/tests/asyncscriptload.js +++ b/dlls/mshtml/tests/asyncscriptload.js @@ -16,6 +16,7 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ +var tests = []; var head = document.getElementsByTagName("head")[0]; /* Dynamically created script element is downloaded as soon as src property is set, @@ -24,7 +25,7 @@ var detached_elem_executed = false; var detached_elem = document.createElement("script"); detached_elem.src = "jsstream.php?detached_script"; -function test_detached_script_elem() { +async_test("detached_script_elem", function() { var oncomplete_called = false; detached_elem.onreadystatechange = guard(function() { if(detached_elem.readyState == "complete") { @@ -44,7 +45,7 @@ function test_detached_script_elem() { }); external.writeStream("detached_script", 'detached_elem_executed = true;'); -} +}); /* Dynamically created script elements are evaluated as soon as they are loaded, no matter * how they are ordered in the tree. */ @@ -58,7 +59,7 @@ var attached_elem2 = document.createElement("script"); attached_elem2.src = "jsstream.php?attached_script2"; head.appendChild(attached_elem2); -function test_attached_script_elem() { +async_test("attached_script_elem", function() { attached_elem1.onreadystatechange = guard(function() { ok(attached_elem1.readyState == "loaded", "attached_elem1.readyState = " + attached_elem2.readyState); ok(attached_elem1_executed, "attached element 1 not executed before readyState complete"); @@ -73,9 +74,9 @@ function test_attached_script_elem() { }); external.writeStream("attached_script2", 'attached_elem2_executed = true;'); -} +}); -function test_dynamic_element() { +async_test("append_script", function() { var elem = document.createElement("script"); var ready_states = ""; @@ -88,10 +89,4 @@ function test_dynamic_element() { document.body.appendChild(elem); elem.src = "jsstream.php?simple"; external.writeStream("simple", " "); -} - -var tests = [ - test_detached_script_elem, - test_attached_script_elem, - test_dynamic_element -]; +});

1 0

Jacek Caban : mshtml: Support onload handlers in script elements.
by Alexandre Julliard 25 Mar '21

25 Mar '21

Module: wine Branch: master Commit: 34dd3dd21762b69fed684f37c741768ec6b904ad URL: https://source.winehq.org/git/wine.git/?a=commit;h=34dd3dd21762b69fed684f37… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Mar 25 19:28:21 2021 +0100 mshtml: Support onload handlers in script elements. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mshtml/script.c | 24 +++++++++++++++++++----- dlls/mshtml/tests/documentmode.js | 37 +++++++++++++++++++++++++++++++++++++ 2 files changed, 56 insertions(+), 5 deletions(-) diff --git a/dlls/mshtml/script.c b/dlls/mshtml/script.c index 8f2c7f598ea..c52c62f56d9 100644 --- a/dlls/mshtml/script.c +++ b/dlls/mshtml/script.c @@ -723,12 +723,22 @@ static void dispatch_script_readystatechange_event(HTMLScriptElement *script) DOMEvent *event; HRESULT hres; - hres = create_document_event(script->element.node.doc, EVENTID_READYSTATECHANGE, &event); - if(FAILED(hres)) - return; + if(script->readystate != READYSTATE_LOADED || + dispex_compat_mode(&script->element.node.event_target.dispex) < COMPAT_MODE_IE10) { + hres = create_document_event(script->element.node.doc, EVENTID_READYSTATECHANGE, &event); + if(SUCCEEDED(hres)) { + dispatch_event(&script->element.node.event_target, event); + IDOMEvent_Release(&event->IDOMEvent_iface); + } + } - dispatch_event(&script->element.node.event_target, event); - IDOMEvent_Release(&event->IDOMEvent_iface); + if(script->readystate == READYSTATE_LOADED) { + hres = create_document_event(script->element.node.doc, EVENTID_LOAD, &event); + if(SUCCEEDED(hres)) { + dispatch_event(&script->element.node.event_target, event); + IDOMEvent_Release(&event->IDOMEvent_iface); + } + } } typedef struct { @@ -758,6 +768,10 @@ static void set_script_elem_readystate(HTMLScriptElement *script_elem, READYSTAT { script_elem->readystate = readystate; + if(readystate != READYSTATE_LOADED && + dispex_compat_mode(&script_elem->element.node.event_target.dispex) >= COMPAT_MODE_IE11) + return; + if(readystate != READYSTATE_INTERACTIVE) { if(!script_elem->element.node.doc->window->parser_callback_cnt) { fire_readystatechange_task_t *task; diff --git a/dlls/mshtml/tests/documentmode.js b/dlls/mshtml/tests/documentmode.js index d61d3658151..b079042afff 100644 --- a/dlls/mshtml/tests/documentmode.js +++ b/dlls/mshtml/tests/documentmode.js @@ -323,3 +323,40 @@ sync_test("conditional_comments", function() { test_version(7); test_version(8); }); + +var ready_states; + +async_test("script_load", function() { + var v = document.documentMode; + if(v < 9) { + next_test(); + return; + } + + var elem = document.createElement("script"); + ready_states = ""; + + elem.onreadystatechange = guard(function() { + ok(v < 11, "unexpected onreadystatechange call"); + ready_states += elem.readyState + ","; + }); + + elem.onload = guard(function() { + switch(v) { + case 9: + ok(ready_states === "loading,exec,loaded,", "ready_states = " + ready_states); + break; + case 10: + ok(ready_states === "loading,exec,", "ready_states = " + ready_states); + break; + case 11: + ok(ready_states === "exec,", "ready_states = " + ready_states); + break; + } + next_test(); + }); + + document.body.appendChild(elem); + elem.src = "jsstream.php?simple"; + external.writeStream("simple", "ready_states += 'exec,';"); +});

1 0

Jacek Caban : mshtml: Block load event until dynamically created script elements that are part of the document are loaded.
by Alexandre Julliard 25 Mar '21

25 Mar '21

Module: wine Branch: master Commit: e5da8dbfb78c0f19b22af4f4406ad837b43b9f2e URL: https://source.winehq.org/git/wine.git/?a=commit;h=e5da8dbfb78c0f19b22af4f4… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Mar 25 19:28:36 2021 +0100 mshtml: Block load event until dynamically created script elements that are part of the document are loaded. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mshtml/binding.h | 1 + dlls/mshtml/nsio.c | 18 ++++++++++++++++ dlls/mshtml/script.c | 42 ++++++++++++++++++++++++++++++++++++ dlls/mshtml/tests/asyncscriptload.js | 16 ++++++++++++++ 4 files changed, 77 insertions(+) diff --git a/dlls/mshtml/binding.h b/dlls/mshtml/binding.h index 3d1ea6ecc8d..26a337fe470 100644 --- a/dlls/mshtml/binding.h +++ b/dlls/mshtml/binding.h @@ -154,6 +154,7 @@ HRESULT create_channelbsc(IMoniker*,const WCHAR*,BYTE*,DWORD,BOOL,nsChannelBSC** HRESULT channelbsc_load_stream(HTMLInnerWindow*,IMoniker*,IStream*) DECLSPEC_HIDDEN; void channelbsc_set_channel(nsChannelBSC*,nsChannel*,nsIStreamListener*,nsISupports*) DECLSPEC_HIDDEN; IUri *nsuri_get_uri(nsWineURI*) DECLSPEC_HIDDEN; +nsresult create_onload_blocker_request(nsIRequest**) DECLSPEC_HIDDEN; HRESULT read_stream(BSCallback*,IStream*,void*,DWORD,DWORD*) DECLSPEC_HIDDEN; diff --git a/dlls/mshtml/nsio.c b/dlls/mshtml/nsio.c index 1ffc6325725..ea8eae2007d 100644 --- a/dlls/mshtml/nsio.c +++ b/dlls/mshtml/nsio.c @@ -3980,3 +3980,21 @@ void release_nsio(void) nsio = NULL; } } + +nsresult create_onload_blocker_request(nsIRequest **ret) +{ + nsIChannel *channel; + nsACString spec; + nsresult nsres; + + nsACString_InitDepend(&spec, "about:wine-script-onload-blocker"); + nsres = nsIIOService_NewChannel(nsio, &spec, NULL, NULL, &channel); + nsACString_Finish(&spec); + if(NS_FAILED(nsres)) { + ERR("Failed to create channel: %08x\n", nsres); + return nsres; + } + + *ret = (nsIRequest *)channel; + return NS_OK; +} diff --git a/dlls/mshtml/script.c b/dlls/mshtml/script.c index c52c62f56d9..ec739c4777f 100644 --- a/dlls/mshtml/script.c +++ b/dlls/mshtml/script.c @@ -822,6 +822,8 @@ typedef struct { BSCallback bsc; HTMLScriptElement *script_elem; + nsILoadGroup *load_group; + nsIRequest *request; DWORD scheme; DWORD size; @@ -945,6 +947,13 @@ static void ScriptBSC_destroy(BSCallback *bsc) This->script_elem = NULL; } + if(This->request) { + ERR("Unfinished request\n"); + nsIRequest_Release(This->request); + } + if(This->load_group) + nsILoadGroup_Release(This->load_group); + heap_free(This->buf); heap_free(This); } @@ -957,9 +966,19 @@ static HRESULT ScriptBSC_init_bindinfo(BSCallback *bsc) static HRESULT ScriptBSC_start_binding(BSCallback *bsc) { ScriptBSC *This = impl_from_BSCallback(bsc); + nsresult nsres; This->script_elem->binding = &This->bsc; + if(This->load_group) { + nsres = create_onload_blocker_request(&This->request); + if(NS_SUCCEEDED(nsres)) { + nsres = nsILoadGroup_AddRequest(This->load_group, This->request, NULL); + if(NS_FAILED(nsres)) + ERR("AddRequest failed: %08x\n", nsres); + } + } + /* FIXME: We should find a better to decide if 'loading' state is supposed to be used by the protocol. */ if(This->scheme == URL_SCHEME_HTTPS || This->scheme == URL_SCHEME_HTTP) set_script_elem_readystate(This->script_elem, READYSTATE_LOADING); @@ -970,6 +989,7 @@ static HRESULT ScriptBSC_start_binding(BSCallback *bsc) static HRESULT ScriptBSC_stop_binding(BSCallback *bsc, HRESULT result) { ScriptBSC *This = impl_from_BSCallback(bsc); + nsresult nsres; if(SUCCEEDED(result) && !This->script_elem) result = E_UNEXPECTED; @@ -989,6 +1009,14 @@ static HRESULT ScriptBSC_stop_binding(BSCallback *bsc, HRESULT result) This->size = 0; } + if(This->request) { + nsres = nsILoadGroup_RemoveRequest(This->load_group, This->request, NULL, NS_OK); + if(NS_FAILED(nsres)) + ERR("RemoveRequest failed: %08x\n", nsres); + nsIRequest_Release(This->request); + This->request = NULL; + } + IHTMLScriptElement_Release(&This->script_elem->IHTMLScriptElement_iface); This->script_elem = NULL; return S_OK; @@ -1099,6 +1127,20 @@ HRESULT load_script(HTMLScriptElement *script_elem, const WCHAR *src, BOOL async IHTMLScriptElement_AddRef(&script_elem->IHTMLScriptElement_iface); bsc->script_elem = script_elem; + if(window->bscallback && window->bscallback->nschannel && + window->bscallback->nschannel->load_group) { + cpp_bool contains; + nsresult nsres; + + nsres = nsIDOMNode_Contains(script_elem->element.node.doc->node.nsnode, + script_elem->element.node.nsnode, &contains); + if(NS_SUCCEEDED(nsres) && contains) { + TRACE("script %p will block load event\n", script_elem); + bsc->load_group = window->bscallback->nschannel->load_group; + nsILoadGroup_AddRef(bsc->load_group); + } + } + hres = start_binding(window, &bsc->bsc, NULL); IBindStatusCallback_Release(&bsc->bsc.IBindStatusCallback_iface); diff --git a/dlls/mshtml/tests/asyncscriptload.js b/dlls/mshtml/tests/asyncscriptload.js index 6dc333afdf4..207df133e91 100644 --- a/dlls/mshtml/tests/asyncscriptload.js +++ b/dlls/mshtml/tests/asyncscriptload.js @@ -90,3 +90,19 @@ async_test("append_script", function() { elem.src = "jsstream.php?simple"; external.writeStream("simple", " "); }); + +function unexpected_load(e) { + ok(false, "onload event before executing script"); +} + +guard(function() { + var elem = document.createElement("script"); + document.head.appendChild(elem); + elem.src = "jsstream.php?blockload"; + + window.addEventListener("load", unexpected_load, true); + + setTimeout(guard(function() { + external.writeStream("blockload", "window.removeEventListener('load', unexpected_load, true);"); + }), 100); +})();

1 0

Jacek Caban : urlmon: Make default user agent compatible with IE11.
by Alexandre Julliard 25 Mar '21

25 Mar '21

Module: wine Branch: master Commit: 266dc7e8e94f1602ec273b38351d6a32db468178 URL: https://source.winehq.org/git/wine.git/?a=commit;h=266dc7e8e94f1602ec273b38… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Mar 25 19:33:51 2021 +0100 urlmon: Make default user agent compatible with IE11. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/urlmon/session.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/urlmon/session.c b/dlls/urlmon/session.c index 5154bdb61f8..764f52c5630 100644 --- a/dlls/urlmon/session.c +++ b/dlls/urlmon/session.c @@ -528,7 +528,7 @@ static void ensure_useragent(void) else os_type = L""; - swprintf(buf, ARRAY_SIZE(buf), L"Mozilla/4.0 (compatible; MSIE 8.0; Windows %s%d.%d; %sTrident/5.0", + swprintf(buf, ARRAY_SIZE(buf), L"Mozilla/4.0 (compatible; MSIE 7.0; Windows %s%d.%d; %sTrident/7.0", is_nt, info.dwMajorVersion, info.dwMinorVersion, os_type); len = lstrlenW(buf);

1 0

Jacek Caban : urlmon: Factor out obtain_user_agent.
by Alexandre Julliard 25 Mar '21

25 Mar '21

Module: wine Branch: master Commit: 16a7293f37a807cb8160053dbf2987964f2c8438 URL: https://source.winehq.org/git/wine.git/?a=commit;h=16a7293f37a807cb8160053d… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Mar 25 19:34:00 2021 +0100 urlmon: Factor out obtain_user_agent. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/urlmon/session.c | 60 +++++++++++++++++++++++---------------------------- 1 file changed, 27 insertions(+), 33 deletions(-) diff --git a/dlls/urlmon/session.c b/dlls/urlmon/session.c index 764f52c5630..6084ae94b11 100644 --- a/dlls/urlmon/session.c +++ b/dlls/urlmon/session.c @@ -505,19 +505,15 @@ static BOOL get_url_encoding(HKEY root, DWORD *encoding) static LPWSTR user_agent; -static void ensure_useragent(void) +static size_t obtain_user_agent(WCHAR *ret, size_t size) { OSVERSIONINFOW info = {sizeof(info)}; const WCHAR *os_type, *is_nt; - WCHAR buf[512], *ret, *tmp; DWORD res, idx=0; - size_t len, size; BOOL is_wow; + size_t len; HKEY key; - if(user_agent) - return; - GetVersionExW(&info); is_nt = info.dwPlatformId == VER_PLATFORM_WIN32_NT ? L"NT " : L""; @@ -528,16 +524,9 @@ static void ensure_useragent(void) else os_type = L""; - swprintf(buf, ARRAY_SIZE(buf), L"Mozilla/4.0 (compatible; MSIE 7.0; Windows %s%d.%d; %sTrident/7.0", + swprintf(ret, size, L"Mozilla/4.0 (compatible; MSIE 7.0; Windows %s%d.%d; %sTrident/7.0", is_nt, info.dwMajorVersion, info.dwMinorVersion, os_type); - len = lstrlenW(buf); - - size = len+40; - ret = heap_alloc(size * sizeof(WCHAR)); - if(!ret) - return; - - memcpy(ret, buf, len*sizeof(WCHAR)); + len = lstrlenW(ret); res = RegOpenKeyW(HKEY_LOCAL_MACHINE, L"SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\" "Internet Settings\\5.0\\User Agent\\Post Platform", &key); @@ -545,23 +534,16 @@ static void ensure_useragent(void) DWORD value_len; while(1) { - value_len = ARRAY_SIZE(buf); - res = RegEnumValueW(key, idx, buf, &value_len, NULL, NULL, NULL, NULL); + if(idx) { + ret[len++] = ';'; + ret[len++] = ' '; + } + value_len = size - len - 2; + res = RegEnumValueW(key, idx, ret + len, &value_len, NULL, NULL, NULL, NULL); if(res != ERROR_SUCCESS) break; idx++; - if(len + value_len + 2 /* strlen("; ") */ + 1 /* trailing ')' */ >= size) { - tmp = heap_realloc(ret, (size*2+value_len)*sizeof(WCHAR)); - if(!tmp) - break; - ret = tmp; - size = size*2+value_len; - } - - ret[len++] = ';'; - ret[len++] = ' '; - memcpy(ret+len, buf, value_len*sizeof(WCHAR)); len += value_len; } @@ -570,16 +552,28 @@ static void ensure_useragent(void) ret[len++] = ')'; ret[len++] = 0; + TRACE("Using user agent %s\n", debugstr_w(ret)); + return len; +} + +static void ensure_user_agent(void) +{ + EnterCriticalSection(&session_cs); - user_agent = ret; - TRACE("Using user agent %s\n", debugstr_w(user_agent)); + if(!user_agent) { + WCHAR buf[1024]; + obtain_user_agent(buf, ARRAY_SIZE(buf)); + user_agent = heap_strdupW(buf); + } + + LeaveCriticalSection(&session_cs); } LPWSTR get_useragent(void) { LPWSTR ret; - ensure_useragent(); + ensure_user_agent(); EnterCriticalSection(&session_cs); ret = heap_strdupW(user_agent); @@ -606,7 +600,7 @@ HRESULT WINAPI UrlMkGetSessionOption(DWORD dwOption, LPVOID pBuffer, DWORD dwBuf EnterCriticalSection(&session_cs); - ensure_useragent(); + ensure_user_agent(); if(user_agent) { size = WideCharToMultiByte(CP_ACP, 0, user_agent, -1, NULL, 0, NULL, NULL); *pdwBufferLength = size; @@ -703,7 +697,7 @@ HRESULT WINAPI ObtainUserAgentString(DWORD dwOption, LPSTR pcszUAOut, DWORD *cbS EnterCriticalSection(&session_cs); - ensure_useragent(); + ensure_user_agent(); if(user_agent) { size = WideCharToMultiByte(CP_ACP, 0, user_agent, -1, NULL, 0, NULL, NULL);

1 0

Jacek Caban : urlmon: Directly use otain_user_agent in ObtainUserAgentString.
by Alexandre Julliard 25 Mar '21

25 Mar '21

Module: wine Branch: master Commit: 8efbff575a8388fdee4d2b07cfe8620544ccb50e URL: https://source.winehq.org/git/wine.git/?a=commit;h=8efbff575a8388fdee4d2b07… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Mar 25 19:34:08 2021 +0100 urlmon: Directly use otain_user_agent in ObtainUserAgentString. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/urlmon/session.c | 44 ++++++++++++++++++++++---------------------- 1 file changed, 22 insertions(+), 22 deletions(-) diff --git a/dlls/urlmon/session.c b/dlls/urlmon/session.c index 6084ae94b11..b74d56166d9 100644 --- a/dlls/urlmon/session.c +++ b/dlls/urlmon/session.c @@ -510,10 +510,18 @@ static size_t obtain_user_agent(WCHAR *ret, size_t size) OSVERSIONINFOW info = {sizeof(info)}; const WCHAR *os_type, *is_nt; DWORD res, idx=0; + size_t len = 0; BOOL is_wow; - size_t len; HKEY key; + EnterCriticalSection(&session_cs); + if(user_agent) { + len = wcslen(user_agent) + 1; + memcpy(ret, user_agent, min(size, len) * sizeof(WCHAR)); + } + LeaveCriticalSection(&session_cs); + if(len) return len; + GetVersionExW(&info); is_nt = info.dwPlatformId == VER_PLATFORM_WIN32_NT ? L"NT " : L""; @@ -685,33 +693,25 @@ HRESULT WINAPI UrlMkSetSessionOption(DWORD dwOption, LPVOID pBuffer, DWORD dwBuf /************************************************************************** * ObtainUserAgentString (URLMON.@) */ -HRESULT WINAPI ObtainUserAgentString(DWORD dwOption, LPSTR pcszUAOut, DWORD *cbSize) +HRESULT WINAPI ObtainUserAgentString(DWORD option, char *ret, DWORD *ret_size) { - DWORD size; - HRESULT hres = E_FAIL; + DWORD size, len; + WCHAR buf[1024]; + HRESULT hres = S_OK; - TRACE("(%d %p %p)\n", dwOption, pcszUAOut, cbSize); + TRACE("(%d %p %p)\n", option, ret, ret_size); - if(!pcszUAOut || !cbSize) + if(!ret || !ret_size) return E_INVALIDARG; - EnterCriticalSection(&session_cs); - - ensure_user_agent(); - if(user_agent) { - size = WideCharToMultiByte(CP_ACP, 0, user_agent, -1, NULL, 0, NULL, NULL); - - if(size <= *cbSize) { - WideCharToMultiByte(CP_ACP, 0, user_agent, -1, pcszUAOut, *cbSize, NULL, NULL); - hres = S_OK; - }else { - hres = E_OUTOFMEMORY; - } - - *cbSize = size; - } + len = obtain_user_agent(buf, ARRAY_SIZE(buf)); + size = WideCharToMultiByte(CP_ACP, 0, buf, len, NULL, 0, NULL, NULL); + if(size <= *ret_size) + WideCharToMultiByte(CP_ACP, 0, buf, len, ret, *ret_size+1, NULL, NULL); + else + hres = E_OUTOFMEMORY; - LeaveCriticalSection(&session_cs); + *ret_size = size; return hres; }

1 0