April 2023 - Wine-commits - list.winehq.org

Connor McAdams : uiautomationcore: Implement UIA_BoundingRectanglePropertyId for default HWND provider.
by Alexandre Julliard 17 Apr '23

17 Apr '23

Module: wine Branch: master Commit: 9b1243fb6b7ef7a738dea1695b75f9f4d978c70b URL: https://gitlab.winehq.org/wine/wine/-/commit/9b1243fb6b7ef7a738dea1695b75f9… Author: Connor McAdams <cmcadams(a)codeweavers.com> Date: Thu Mar 23 11:05:48 2023 -0400 uiautomationcore: Implement UIA_BoundingRectanglePropertyId for default HWND provider. Signed-off-by: Connor McAdams <cmcadams(a)codeweavers.com> --- dlls/uiautomationcore/tests/uiautomation.c | 126 +++++++++++++++++++++++++---- dlls/uiautomationcore/uia_provider.c | 109 +++++++++++++++++++++++++ 2 files changed, 219 insertions(+), 16 deletions(-)

1 0

Connor McAdams : uiautomationcore: Implement NavigateDirection_Parent navigation for default HWND provider.
by Alexandre Julliard 17 Apr '23

17 Apr '23

Module: wine Branch: master Commit: 2a089f52f06962dfcf78cb7dd5a7ee1f590a013e URL: https://gitlab.winehq.org/wine/wine/-/commit/2a089f52f06962dfcf78cb7dd5a7ee… Author: Connor McAdams <cmcadams(a)codeweavers.com> Date: Thu Mar 23 13:34:46 2023 -0400 uiautomationcore: Implement NavigateDirection_Parent navigation for default HWND provider. Signed-off-by: Connor McAdams <cmcadams(a)codeweavers.com> --- dlls/uiautomationcore/tests/uiautomation.c | 155 +++++++++++++++++++++++------ dlls/uiautomationcore/uia_provider.c | 48 ++++++++- 2 files changed, 172 insertions(+), 31 deletions(-)

1 0

Connor McAdams : uiautomationcore: Add a default ProviderType_Proxy MSAA bridge clientside provider.
by Alexandre Julliard 17 Apr '23

17 Apr '23

Module: wine Branch: master Commit: a19c1ff1697358ae69fb65ee3108a69829657e51 URL: https://gitlab.winehq.org/wine/wine/-/commit/a19c1ff1697358ae69fb65ee3108a6… Author: Connor McAdams <cmcadams(a)codeweavers.com> Date: Thu Mar 23 14:54:32 2023 -0400 uiautomationcore: Add a default ProviderType_Proxy MSAA bridge clientside provider. Signed-off-by: Connor McAdams <cmcadams(a)codeweavers.com> --- dlls/uiautomationcore/tests/uiautomation.c | 112 +++++++++++++++++++++++++++-- dlls/uiautomationcore/uia_client.c | 14 +++- dlls/uiautomationcore/uia_private.h | 2 + dlls/uiautomationcore/uia_provider.c | 60 ++++++++++------ 4 files changed, 160 insertions(+), 28 deletions(-)

1 0

Rémi Bernon : imm32: Forward ImmGetContext to NtUserGetWindowInputContext directly.
by Alexandre Julliard 17 Apr '23

17 Apr '23

Module: wine Branch: master Commit: 422ee56c1b1f489d3a7a7aab97950ee93cc91549 URL: https://gitlab.winehq.org/wine/wine/-/commit/422ee56c1b1f489d3a7a7aab97950e… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Mon Apr 17 10:08:58 2023 +0200 imm32: Forward ImmGetContext to NtUserGetWindowInputContext directly. --- dlls/imm32/imm.c | 19 +++---------------- 1 file changed, 3 insertions(+), 16 deletions(-) diff --git a/dlls/imm32/imm.c b/dlls/imm32/imm.c index 0bd71244203..5a32138561c 100644 --- a/dlls/imm32/imm.c +++ b/dlls/imm32/imm.c @@ -1776,23 +1776,10 @@ BOOL WINAPI ImmGetCompositionWindow( HIMC himc, COMPOSITIONFORM *composition ) * ImmGetContext (IMM32.@) * */ -HIMC WINAPI ImmGetContext(HWND hWnd) +HIMC WINAPI ImmGetContext( HWND hwnd ) { - HIMC rc; - - TRACE("%p\n", hWnd); - - rc = NtUserGetWindowInputContext(hWnd); - - if (rc) - { - struct imc *data = get_imc_data( rc ); - if (!data) rc = 0; - } - - TRACE("returning %p\n", rc); - - return rc; + TRACE( "hwnd %p\n", hwnd ); + return NtUserGetWindowInputContext( hwnd ); } /***********************************************************************

1 0

Byeong-Sik Jeon : imm32: Avoid updating INPUTCONTEXT hWnd on ImmSetActiveContext deactivation.
by Alexandre Julliard 17 Apr '23

17 Apr '23

Module: wine Branch: master Commit: 93e5d7b317e9a9dc90a95dc731e5e07e8b1ec776 URL: https://gitlab.winehq.org/wine/wine/-/commit/93e5d7b317e9a9dc90a95dc731e5e0… Author: Byeong-Sik Jeon <bsjeon(a)hanmail.net> Date: Mon Apr 17 10:10:28 2023 +0200 imm32: Avoid updating INPUTCONTEXT hWnd on ImmSetActiveContext deactivation. --- dlls/imm32/imm.c | 2 +- dlls/imm32/tests/imm32.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/imm32/imm.c b/dlls/imm32/imm.c index 5a32138561c..2fd247f36c7 100644 --- a/dlls/imm32/imm.c +++ b/dlls/imm32/imm.c @@ -852,7 +852,7 @@ BOOL WINAPI ImmSetActiveContext(HWND hwnd, HIMC himc, BOOL activate) if (data) { - data->IMC.hWnd = activate ? hwnd : NULL; + if (activate) data->IMC.hWnd = hwnd; if ((ime = imc_select_ime( data ))) ime->pImeSetActiveContext( himc, activate ); } diff --git a/dlls/imm32/tests/imm32.c b/dlls/imm32/tests/imm32.c index d9f8c7c446a..92b59b7d005 100644 --- a/dlls/imm32/tests/imm32.c +++ b/dlls/imm32/tests/imm32.c @@ -5314,7 +5314,7 @@ static void test_ImmSetActiveContext(void) ctx->hWnd = (HWND)0xdeadbeef; ok_ret( 1, ImmSetActiveContext( hwnd, himc, FALSE ) ); - todo_wine ok_eq( (HWND)0xdeadbeef, ctx->hWnd, HWND, "%p" ); + ok_eq( (HWND)0xdeadbeef, ctx->hWnd, HWND, "%p" ); deactivate_2_seq[0].himc = himc; ok_seq( deactivate_2_seq );

1 0

Rémi Bernon : user32: Move WM_IME_COMPOSITION DefWindowProc handlers in separate helpers.
by Alexandre Julliard 17 Apr '23

17 Apr '23

Module: wine Branch: master Commit: 45b096abb489f0a92b03a73328de7ca9958a4322 URL: https://gitlab.winehq.org/wine/wine/-/commit/45b096abb489f0a92b03a73328de7c… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Mon Apr 17 09:35:04 2023 +0200 user32: Move WM_IME_COMPOSITION DefWindowProc handlers in separate helpers. --- dlls/user32/defwnd.c | 110 +++++++++++++++++++++++++-------------------------- 1 file changed, 55 insertions(+), 55 deletions(-) diff --git a/dlls/user32/defwnd.c b/dlls/user32/defwnd.c index b4b42b86ae1..6c45f89e392 100644 --- a/dlls/user32/defwnd.c +++ b/dlls/user32/defwnd.c @@ -25,6 +25,59 @@ WINE_DEFAULT_DEBUG_CHANNEL(win); +static void default_ime_compositionW( HWND hwnd, WPARAM wparam, LPARAM lparam ) +{ + WCHAR *buf = NULL; + LONG size, i; + HIMC himc; + + if (!(lparam & GCS_RESULTSTR) || !(himc = ImmGetContext( hwnd ))) return; + + if ((size = ImmGetCompositionStringW( himc, GCS_RESULTSTR, NULL, 0 ))) + { + if (!(buf = HeapAlloc( GetProcessHeap(), 0, size * sizeof(WCHAR) ))) size = 0; + else size = ImmGetCompositionStringW( himc, GCS_RESULTSTR, buf, size * sizeof(WCHAR) ); + } + ImmReleaseContext( hwnd, himc ); + + for (i = 0; i < size / sizeof(WCHAR); i++) + SendMessageW( hwnd, WM_IME_CHAR, buf[i], 1 ); + HeapFree( GetProcessHeap(), 0, buf ); +} + +static void default_ime_compositionA( HWND hwnd, WPARAM wparam, LPARAM lparam ) +{ + unsigned char lead = 0; + char *buf = NULL; + LONG size, i; + HIMC himc; + + if (!(lparam & GCS_RESULTSTR) || !(himc = ImmGetContext( hwnd ))) return; + + if ((size = ImmGetCompositionStringA( himc, GCS_RESULTSTR, NULL, 0 ))) + { + if (!(buf = HeapAlloc( GetProcessHeap(), 0, size ))) size = 0; + else size = ImmGetCompositionStringA( himc, GCS_RESULTSTR, buf, size ); + } + ImmReleaseContext( hwnd, himc ); + + for (i = 0; i < size; i++) + { + unsigned char c = buf[i]; + if (!lead) + { + if (IsDBCSLeadByte( c )) lead = c; + else SendMessageA( hwnd, WM_IME_CHAR, c, 1 ); + } + else + { + SendMessageA( hwnd, WM_IME_CHAR, MAKEWORD(c, lead), 1 ); + lead = 0; + } + } + + HeapFree( GetProcessHeap(), 0, buf ); +} /*********************************************************************** * DefWindowProcA (USER32.@) @@ -66,41 +119,7 @@ LRESULT WINAPI DefWindowProcA( HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam break; case WM_IME_COMPOSITION: - if (lParam & GCS_RESULTSTR) - { - LONG size, i; - unsigned char lead = 0; - char *buf = NULL; - HIMC himc = ImmGetContext( hwnd ); - - if (himc) - { - if ((size = ImmGetCompositionStringA( himc, GCS_RESULTSTR, NULL, 0 ))) - { - if (!(buf = HeapAlloc( GetProcessHeap(), 0, size ))) size = 0; - else size = ImmGetCompositionStringA( himc, GCS_RESULTSTR, buf, size ); - } - ImmReleaseContext( hwnd, himc ); - - for (i = 0; i < size; i++) - { - unsigned char c = buf[i]; - if (!lead) - { - if (IsDBCSLeadByte( c )) - lead = c; - else - SendMessageA( hwnd, WM_IME_CHAR, c, 1 ); - } - else - { - SendMessageA( hwnd, WM_IME_CHAR, MAKEWORD(c, lead), 1 ); - lead = 0; - } - } - HeapFree( GetProcessHeap(), 0, buf ); - } - } + default_ime_compositionA( hwnd, wParam, lParam ); /* fall through */ default: @@ -159,26 +178,7 @@ LRESULT WINAPI DefWindowProcW( break; case WM_IME_COMPOSITION: - if (lParam & GCS_RESULTSTR) - { - LONG size, i; - WCHAR *buf = NULL; - HIMC himc = ImmGetContext( hwnd ); - - if (himc) - { - if ((size = ImmGetCompositionStringW( himc, GCS_RESULTSTR, NULL, 0 ))) - { - if (!(buf = HeapAlloc( GetProcessHeap(), 0, size * sizeof(WCHAR) ))) size = 0; - else size = ImmGetCompositionStringW( himc, GCS_RESULTSTR, buf, size * sizeof(WCHAR) ); - } - ImmReleaseContext( hwnd, himc ); - - for (i = 0; i < size / sizeof(WCHAR); i++) - SendMessageW( hwnd, WM_IME_CHAR, buf[i], 1 ); - HeapFree( GetProcessHeap(), 0, buf ); - } - } + default_ime_compositionW( hwnd, wParam, lParam ); /* fall through */ default:

1 0

Rémi Bernon : user32: Ignore WM_IME_COMPOSITION from the IME UI window in DefWindowProc.
by Alexandre Julliard 17 Apr '23

17 Apr '23

Module: wine Branch: master Commit: 6fb47669476a2cdd982b5e3a461d0d97ebf74dec URL: https://gitlab.winehq.org/wine/wine/-/commit/6fb47669476a2cdd982b5e3a461d0d… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Mon Apr 17 09:37:58 2023 +0200 user32: Ignore WM_IME_COMPOSITION from the IME UI window in DefWindowProc. As in 6fd3bd9b62f405a54db29dc5a72805063a6099ca. --- dlls/user32/defwnd.c | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/dlls/user32/defwnd.c b/dlls/user32/defwnd.c index 6c45f89e392..60afec4e7c6 100644 --- a/dlls/user32/defwnd.c +++ b/dlls/user32/defwnd.c @@ -119,8 +119,13 @@ LRESULT WINAPI DefWindowProcA( HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam break; case WM_IME_COMPOSITION: + { + HWND ime_hwnd = NtUserGetDefaultImeWindow( hwnd ); + if (!ime_hwnd || ime_hwnd == NtUserGetParent( hwnd )) break; + default_ime_compositionA( hwnd, wParam, lParam ); /* fall through */ + } default: result = NtUserMessageCall( hwnd, msg, wParam, lParam, 0, NtUserDefWindowProc, TRUE ); @@ -178,8 +183,13 @@ LRESULT WINAPI DefWindowProcW( break; case WM_IME_COMPOSITION: + { + HWND ime_hwnd = NtUserGetDefaultImeWindow( hwnd ); + if (!ime_hwnd || ime_hwnd == NtUserGetParent( hwnd )) break; + default_ime_compositionW( hwnd, wParam, lParam ); /* fall through */ + } default: result = NtUserMessageCall( hwnd, msg, wParam, lParam, 0, NtUserDefWindowProc, FALSE );

1 0

Francois Gouget : user32/tests: Fix the DBCS WM_CHAR tests in mixed locales.
by Alexandre Julliard 17 Apr '23

17 Apr '23

Module: wine Branch: master Commit: 8b8d787d7dc0b2ff2894968173bfcf763f16e131 URL: https://gitlab.winehq.org/wine/wine/-/commit/8b8d787d7dc0b2ff2894968173bfcf… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Mon Apr 17 14:44:06 2023 +0200 user32/tests: Fix the DBCS WM_CHAR tests in mixed locales. WM_CHAR interprets the character codes using a code page tied to the keyboard layout, not CP_ACP. Also skip the test if WideCharToMultiByte() used the default character since that breaks the round-trip. Trace the code page when skipping the tests. Wine-Bug: https://bugs.winehq.org//show_bug.cgi?id=54424 --- dlls/user32/tests/edit.c | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/dlls/user32/tests/edit.c b/dlls/user32/tests/edit.c index 78328ee1729..bbde9b16616 100644 --- a/dlls/user32/tests/edit.c +++ b/dlls/user32/tests/edit.c @@ -3368,15 +3368,22 @@ static void test_wordbreak_proc(void) static void test_dbcs_WM_CHAR(void) { + HKL hkl; + UINT cp; WCHAR textW[] = { 0x4e00, 0x4e8c, 0x4e09, 0 }; /* one, two, three */ unsigned char bytes[7]; + BOOL hasdefault; HWND hwnd[2]; int i; - WideCharToMultiByte(CP_ACP, 0, textW, -1, (char *)bytes, ARRAY_SIZE(bytes), NULL, NULL); - if (!IsDBCSLeadByte(bytes[0])) + hkl = GetKeyboardLayout( 0 ); + if (!GetLocaleInfoW(LOWORD(hkl), LOCALE_IDEFAULTANSICODEPAGE | LOCALE_RETURN_NUMBER, (WCHAR *)&cp, sizeof(cp) / sizeof(WCHAR))) + cp = CP_ACP; + + WideCharToMultiByte(cp, WC_NO_BEST_FIT_CHARS, textW, -1, (char *)bytes, ARRAY_SIZE(bytes), NULL, &hasdefault); + if (hasdefault || !IsDBCSLeadByteEx(cp, bytes[0])) { - skip("Skipping DBCS WM_CHAR test in this codepage\n"); + skip("Skipping DBCS WM_CHAR test in the %d codepage\n", cp); return; } hwnd[0] = create_editcontrol(ES_AUTOHSCROLL | ES_AUTOVSCROLL, 0); @@ -3396,6 +3403,7 @@ static void test_dbcs_WM_CHAR(void) r = SetWindowTextA(hwnd[i], ""); ok(r, "SetWindowText failed\n"); + /* sending the lead byte separately works for DBCS locales */ for (p = bytes; *p; p++) PostMessageA(hwnd[i], WM_CHAR, *p, 1);

1 0

Gabriel Ivăncescu : mshtml: Implement MediaQueryList's addListener method.
by Alexandre Julliard 17 Apr '23

17 Apr '23

Module: wine Branch: master Commit: 1a01fb23a3aed816030da4a22586ed6755d8e2dd URL: https://gitlab.winehq.org/wine/wine/-/commit/1a01fb23a3aed816030da4a22586ed… Author: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> Date: Fri Apr 14 16:52:50 2023 +0300 mshtml: Implement MediaQueryList's addListener method. Signed-off-by: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> --- dlls/mshtml/htmlevent.c | 2 +- dlls/mshtml/mshtml_private.h | 1 + dlls/mshtml/nsiface.idl | 10 +++ dlls/mshtml/omnavigator.c | 147 ++++++++++++++++++++++++++++++++++++++++++- dlls/mshtml/tests/es5.js | 63 ++++++++++++++++++- dlls/mshtml/tests/script.c | 22 +++++++ 6 files changed, 240 insertions(+), 5 deletions(-)

1 0

Gabriel Ivăncescu : mshtml: Implement MediaQueryList's removeListener method.
by Alexandre Julliard 17 Apr '23

17 Apr '23

Module: wine Branch: master Commit: bce0ebafa6e0d021f9ba27960abe74c83b63a3cc URL: https://gitlab.winehq.org/wine/wine/-/commit/bce0ebafa6e0d021f9ba27960abe74… Author: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> Date: Fri Apr 14 16:52:51 2023 +0300 mshtml: Implement MediaQueryList's removeListener method. Signed-off-by: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> --- dlls/mshtml/omnavigator.c | 17 +++++++++++++++-- dlls/mshtml/tests/es5.js | 10 +++++++++- 2 files changed, 24 insertions(+), 3 deletions(-) diff --git a/dlls/mshtml/omnavigator.c b/dlls/mshtml/omnavigator.c index b95d2d74e93..298d99f86eb 100644 --- a/dlls/mshtml/omnavigator.c +++ b/dlls/mshtml/omnavigator.c @@ -2817,10 +2817,23 @@ static HRESULT WINAPI media_query_list_addListener(IWineMSHTMLMediaQueryList *if static HRESULT WINAPI media_query_list_removeListener(IWineMSHTMLMediaQueryList *iface, VARIANT *listener) { struct media_query_list *media_query_list = impl_from_IWineMSHTMLMediaQueryList(iface); + struct media_query_list_listener *entry; - FIXME("(%p)->(%s)\n", media_query_list, debugstr_variant(listener)); + TRACE("(%p)->(%s)\n", media_query_list, debugstr_variant(listener)); - return E_NOTIMPL; + if(V_VT(listener) != VT_DISPATCH || !V_DISPATCH(listener)) + return S_OK; + + LIST_FOR_EACH_ENTRY(entry, &media_query_list->listeners, struct media_query_list_listener, entry) { + if(entry->function == V_DISPATCH(listener)) { + list_remove(&entry->entry); + IDispatch_Release(entry->function); + free(entry); + break; + } + } + + return S_OK; } static const IWineMSHTMLMediaQueryListVtbl media_query_list_vtbl = { diff --git a/dlls/mshtml/tests/es5.js b/dlls/mshtml/tests/es5.js index 8bd8d8e7d06..982544f6b84 100644 --- a/dlls/mshtml/tests/es5.js +++ b/dlls/mshtml/tests/es5.js @@ -2101,6 +2101,8 @@ async_test("matchMedia", function() { ok(!("removeEventListener" in mql), "removeEventListener in MediaQueryList"); r = mql.addListener(null); ok(r === undefined, "addListener with null returned " + r); + r = mql.removeListener(null); + ok(r === undefined, "removeListener with null returned " + r); r = mql.addListener("function() { ok(false, 'string handler called'); }"); ok(r === undefined, "addListener with string returned " + r); @@ -2117,7 +2119,9 @@ async_test("matchMedia", function() { } var tests = [ [ 20, 20, function() { - var r = mql.addListener(handler); + var r = mql.removeListener("function() { ok(false, 'string handler called'); }"); + ok(r === undefined, "removeListener with string returned " + r); + r = mql.addListener(handler); ok(r === undefined, "addListener with function returned " + r); }], [ 120, 120, function() { @@ -2129,8 +2133,12 @@ async_test("matchMedia", function() { [ 30, 30, function() { ok(event_fired === true, "event not fired after changing from 120x120 to 30x30 view"); ok(event2_fired === true, "event not fired from second handler after changing from 120x120 to 30x30 view"); + var r = mql.removeListener(handler); + ok(r === undefined, "removeListener with function returned " + r); }], [ 300, 300, function() { + ok(event_fired === false, "removed event handler fired after changing from 30x30 to 300x300 view"); + ok(event2_fired === true, "event not fired from second handler after changing from 30x30 to 300x300 view"); }] ];

1 0