February 2022 - Wine-commits

Nikolay Sivov : mf/tests: Add some tests for rate support in SAR.
by Alexandre Julliard 17 Feb '22

17 Feb '22

Module: wine Branch: master Commit: d576a61f45b4369b6ac480c400661d383918f3dd URL: https://source.winehq.org/git/wine.git/?a=commit;h=d576a61f45b4369b6ac480c4… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Thu Feb 17 12:18:14 2022 +0300 mf/tests: Add some tests for rate support in SAR. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mf/tests/mf.c | 110 +++++++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 110 insertions(+) diff --git a/dlls/mf/tests/mf.c b/dlls/mf/tests/mf.c index b3f50665e79..63df50cf3b8 100644 --- a/dlls/mf/tests/mf.c +++ b/dlls/mf/tests/mf.c @@ -3842,6 +3842,108 @@ static void test_quality_manager(void) ok(hr == S_OK, "Shutdown failure, hr %#x.\n", hr); } +static void check_sar_rate_support(IMFMediaSink *sink) +{ + IMFRateSupport *rate_support; + IMFMediaTypeHandler *handler; + IMFStreamSink *stream_sink; + IMFMediaType *media_type; + HRESULT hr; + float rate; + + hr = IMFMediaSink_QueryInterface(sink, &IID_IMFRateSupport, (void **)&rate_support); + todo_wine + ok(hr == S_OK, "Unexpected hr %#x.\n", hr); + if (FAILED(hr)) return; + + hr = IMFMediaSink_GetStreamSinkByIndex(sink, 0, &stream_sink); + if (hr == MF_E_SHUTDOWN) + { + hr = IMFRateSupport_GetSlowestRate(rate_support, MFRATE_FORWARD, FALSE, NULL); + ok(hr == E_POINTER, "Unexpected hr %#x.\n", hr); + + hr = IMFRateSupport_GetSlowestRate(rate_support, MFRATE_FORWARD, FALSE, &rate); + ok(hr == MF_E_SHUTDOWN, "Unexpected hr %#x.\n", hr); + + hr = IMFRateSupport_GetFastestRate(rate_support, MFRATE_FORWARD, FALSE, &rate); + ok(hr == MF_E_SHUTDOWN, "Unexpected hr %#x.\n", hr); + + IMFRateSupport_Release(rate_support); + return; + } + ok(hr == S_OK, "Unexpected hr %#x.\n", hr); + + hr = IMFStreamSink_GetMediaTypeHandler(stream_sink, &handler); + ok(hr == S_OK, "Unexpected hr %#x.\n", hr); + + IMFStreamSink_Release(stream_sink); + + hr = IMFRateSupport_GetSlowestRate(rate_support, MFRATE_FORWARD, FALSE, NULL); + ok(hr == E_POINTER, "Unexpected hr %#x.\n", hr); + + hr = IMFRateSupport_GetFastestRate(rate_support, MFRATE_FORWARD, FALSE, NULL); + ok(hr == E_POINTER, "Unexpected hr %#x.\n", hr); + + hr = IMFMediaTypeHandler_GetCurrentMediaType(handler, &media_type); + if (SUCCEEDED(hr)) + { + hr = IMFRateSupport_GetSlowestRate(rate_support, MFRATE_FORWARD, FALSE, &rate); + ok(hr == S_OK, "Unexpected hr %#x.\n", hr); + + hr = IMFRateSupport_GetSlowestRate(rate_support, MFRATE_FORWARD, TRUE, &rate); + ok(hr == S_OK, "Unexpected hr %#x.\n", hr); + + hr = IMFRateSupport_GetSlowestRate(rate_support, MFRATE_REVERSE, FALSE, &rate); + ok(hr == S_OK, "Unexpected hr %#x.\n", hr); + + hr = IMFRateSupport_GetSlowestRate(rate_support, MFRATE_REVERSE, TRUE, &rate); + ok(hr == S_OK, "Unexpected hr %#x.\n", hr); + + hr = IMFRateSupport_GetFastestRate(rate_support, MFRATE_FORWARD, FALSE, &rate); + ok(hr == S_OK, "Unexpected hr %#x.\n", hr); + + hr = IMFRateSupport_GetFastestRate(rate_support, MFRATE_FORWARD, TRUE, &rate); + ok(hr == S_OK, "Unexpected hr %#x.\n", hr); + + hr = IMFRateSupport_GetFastestRate(rate_support, MFRATE_REVERSE, FALSE, &rate); + ok(hr == S_OK, "Unexpected hr %#x.\n", hr); + + hr = IMFRateSupport_GetFastestRate(rate_support, MFRATE_REVERSE, TRUE, &rate); + ok(hr == S_OK, "Unexpected hr %#x.\n", hr); + + IMFMediaType_Release(media_type); + } + else + { + hr = IMFRateSupport_GetSlowestRate(rate_support, MFRATE_FORWARD, FALSE, &rate); + ok(hr == MF_E_NOT_INITIALIZED, "Unexpected hr %#x.\n", hr); + + hr = IMFRateSupport_GetSlowestRate(rate_support, MFRATE_FORWARD, TRUE, &rate); + ok(hr == MF_E_NOT_INITIALIZED, "Unexpected hr %#x.\n", hr); + + hr = IMFRateSupport_GetSlowestRate(rate_support, MFRATE_REVERSE, FALSE, &rate); + ok(hr == MF_E_NOT_INITIALIZED, "Unexpected hr %#x.\n", hr); + + hr = IMFRateSupport_GetSlowestRate(rate_support, MFRATE_REVERSE, TRUE, &rate); + ok(hr == MF_E_NOT_INITIALIZED, "Unexpected hr %#x.\n", hr); + + hr = IMFRateSupport_GetFastestRate(rate_support, MFRATE_FORWARD, FALSE, &rate); + ok(hr == MF_E_NOT_INITIALIZED, "Unexpected hr %#x.\n", hr); + + hr = IMFRateSupport_GetFastestRate(rate_support, MFRATE_FORWARD, TRUE, &rate); + ok(hr == MF_E_NOT_INITIALIZED, "Unexpected hr %#x.\n", hr); + + hr = IMFRateSupport_GetFastestRate(rate_support, MFRATE_REVERSE, FALSE, &rate); + ok(hr == MF_E_NOT_INITIALIZED, "Unexpected hr %#x.\n", hr); + + hr = IMFRateSupport_GetFastestRate(rate_support, MFRATE_REVERSE, TRUE, &rate); + ok(hr == MF_E_NOT_INITIALIZED, "Unexpected hr %#x.\n", hr); + } + + IMFMediaTypeHandler_Release(handler); + IMFRateSupport_Release(rate_support); +} + static void test_sar(void) { IMFPresentationClock *present_clock, *present_clock2; @@ -4073,9 +4175,13 @@ if (SUCCEEDED(hr)) IMFMediaType_Release(mediatype); + check_sar_rate_support(sink); + hr = IMFMediaTypeHandler_SetCurrentMediaType(handler, mediatype2); ok(hr == S_OK, "Failed to set current type, hr %#x.\n", hr); + check_sar_rate_support(sink); + hr = IMFMediaTypeHandler_GetCurrentMediaType(handler, &mediatype); ok(hr == S_OK, "Unexpected hr %#x.\n", hr); ok(mediatype == mediatype2, "Unexpected instance.\n"); @@ -4184,6 +4290,8 @@ if (SUCCEEDED(hr)) hr = IMFMediaSink_GetPresentationClock(sink, &present_clock2); ok(hr == MF_E_SHUTDOWN, "Unexpected hr %#x.\n", hr); + check_sar_rate_support(sink); + IMFMediaSink_Release(sink); /* Activation */ @@ -4253,6 +4361,8 @@ if (SUCCEEDED(hr)) hr = MFCreateAudioRenderer(attributes, &sink); ok(hr == MF_E_NO_AUDIO_PLAYBACK_DEVICE, "Failed to create a sink, hr %#x.\n", hr); + IMFAttributes_Release(attributes); + CoUninitialize(); }

1 0

Hans Leidekker : ncrypt: Fix resizing property array.
by Alexandre Julliard 17 Feb '22

17 Feb '22

Module: wine Branch: master Commit: 26d8f334796bca16dfb17ae7ecf9e47de4577cce URL: https://source.winehq.org/git/wine.git/?a=commit;h=26d8f334796bca16dfb17ae7… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Thu Feb 17 11:06:29 2022 +0100 ncrypt: Fix resizing property array. Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ncrypt/main.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/dlls/ncrypt/main.c b/dlls/ncrypt/main.c index 6b1a87ea785..ca70b2d92c3 100644 --- a/dlls/ncrypt/main.c +++ b/dlls/ncrypt/main.c @@ -312,7 +312,7 @@ SECURITY_STATUS WINAPI NCryptOpenStorageProvider(NCRYPT_PROV_HANDLE *provider, c static SECURITY_STATUS set_object_property(struct object *object, const WCHAR *name, BYTE *value, DWORD value_size) { - struct object_property *property = &object->properties[object->num_properties]; + struct object_property *property; FIXME("check duplicates\n"); if (!object->num_properties) @@ -322,18 +322,18 @@ static SECURITY_STATUS set_object_property(struct object *object, const WCHAR *n ERR("Error allocating memory."); return NTE_NO_MEMORY; } - object->num_properties++; + property = &object->properties[object->num_properties++]; } else { struct object_property *tmp; - if (!(tmp = realloc(object->properties, sizeof(*property) * object->num_properties + 1))) + if (!(tmp = realloc(object->properties, sizeof(*property) * (object->num_properties + 1)))) { ERR("Error allocating memory."); return NTE_NO_MEMORY; } object->properties = tmp; - object->num_properties++; + property = &object->properties[object->num_properties++]; } memset(property, 0, sizeof(*property));

1 0

Santino Mazza : ncrypt: Set a couple of key properties.
by Alexandre Julliard 17 Feb '22

17 Feb '22

Module: wine Branch: master Commit: 9b1c28e84b65109805b0fa73029215a27414359c URL: https://source.winehq.org/git/wine.git/?a=commit;h=9b1c28e84b65109805b0fa73… Author: Santino Mazza <mazzasantino1206(a)gmail.com> Date: Thu Feb 17 11:06:30 2022 +0100 ncrypt: Set a couple of key properties. Signed-off-by: Santino Mazza <mazzasantino1206(a)gmail.com> Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ncrypt/main.c | 100 ++++++++++++++++++++++-------------------- dlls/ncrypt/ncrypt_internal.h | 1 + dlls/ncrypt/tests/ncrypt.c | 22 +++++++--- include/ncrypt.h | 34 ++++++++++++++ 4 files changed, 104 insertions(+), 53 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=9b1c28e84b65109805b0…

1 0

Santino Mazza : ncrypt: NCryptGetProperty should always fill result size.
by Alexandre Julliard 17 Feb '22

17 Feb '22

Module: wine Branch: master Commit: 88e027b523e704c56b3434148fb27018bca922bb URL: https://source.winehq.org/git/wine.git/?a=commit;h=88e027b523e704c56b343414… Author: Santino Mazza <mazzasantino1206(a)gmail.com> Date: Thu Feb 17 11:06:31 2022 +0100 ncrypt: NCryptGetProperty should always fill result size. Signed-off-by: Santino Mazza <mazzasantino1206(a)gmail.com> Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ncrypt/main.c | 7 ++----- dlls/ncrypt/tests/ncrypt.c | 2 -- 2 files changed, 2 insertions(+), 7 deletions(-) diff --git a/dlls/ncrypt/main.c b/dlls/ncrypt/main.c index 82ce7a0b4dc..2c2a0dc6f56 100644 --- a/dlls/ncrypt/main.c +++ b/dlls/ncrypt/main.c @@ -165,11 +165,8 @@ SECURITY_STATUS WINAPI NCryptGetProperty(NCRYPT_HANDLE handle, const WCHAR *name if (flags) FIXME("flags %#lx not supported\n", flags); if (!(property = get_object_property(object, name))) return NTE_INVALID_PARAMETER; - if (!output) - { - *result = property->value_size; - return ERROR_SUCCESS; - } + *result = property->value_size; + if (!output) return ERROR_SUCCESS; if (outsize < property->value_size) return NTE_BUFFER_TOO_SMALL; memcpy(output, property->value, property->value_size); diff --git a/dlls/ncrypt/tests/ncrypt.c b/dlls/ncrypt/tests/ncrypt.c index 070b0c3f13e..891e5b60254 100644 --- a/dlls/ncrypt/tests/ncrypt.c +++ b/dlls/ncrypt/tests/ncrypt.c @@ -186,9 +186,7 @@ static void test_get_property(void) value[0] = 0; ret = NCryptGetProperty(key, NCRYPT_ALGORITHM_GROUP_PROPERTY, (BYTE *)value, sizeof(value), &size, 0); ok(ret == ERROR_SUCCESS, "got %#lx\n", ret); - todo_wine { ok(size == 8, "got %lu\n", size); - } ok(!lstrcmpW(value, L"RSA"), "The string doesn't match with 'RSA'\n"); size = 0;

1 0

Santino Mazza : ncrypt: Add NCryptVerifySignature stub.
by Alexandre Julliard 17 Feb '22

17 Feb '22

Module: wine Branch: master Commit: 12cb5d2efedebe1594a7fd8cb6fb4cad804f20fa URL: https://source.winehq.org/git/wine.git/?a=commit;h=12cb5d2efedebe1594a7fd8c… Author: Santino Mazza <mazzasantino1206(a)gmail.com> Date: Thu Feb 17 11:06:32 2022 +0100 ncrypt: Add NCryptVerifySignature stub. With this change now some applications which uses ncrypt for signature validation work. For example, the visual studio installer (using dotnet from winetricks). Signed-off-by: Santino Mazza <mazzasantino1206(a)gmail.com> Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ncrypt/main.c | 8 ++++++++ dlls/ncrypt/ncrypt.spec | 2 +- 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/dlls/ncrypt/main.c b/dlls/ncrypt/main.c index 2c2a0dc6f56..8e789573924 100644 --- a/dlls/ncrypt/main.c +++ b/dlls/ncrypt/main.c @@ -367,3 +367,11 @@ SECURITY_STATUS WINAPI NCryptSetProperty(NCRYPT_HANDLE handle, const WCHAR *name return set_object_property(object, name, input, insize); } + +SECURITY_STATUS WINAPI NCryptVerifySignature(NCRYPT_KEY_HANDLE handle, void *padding, BYTE *hash, DWORD hash_size, + BYTE *signature, DWORD signature_size, DWORD flags) +{ + FIXME("(%#Ix, %p, %p, %lu, %p, %lu, %#lx): stub\n", handle, padding, hash, hash_size, signature, + signature_size, flags); + return ERROR_SUCCESS; +} diff --git a/dlls/ncrypt/ncrypt.spec b/dlls/ncrypt/ncrypt.spec index 0e92609790c..bb914616373 100644 --- a/dlls/ncrypt/ncrypt.spec +++ b/dlls/ncrypt/ncrypt.spec @@ -101,7 +101,7 @@ @ stub NCryptUnprotectKey @ stub NCryptUnprotectSecret @ stub NCryptVerifyClaim -@ stub NCryptVerifySignature +@ stdcall NCryptVerifySignature(ptr ptr ptr long ptr long long) @ stub SslChangeNotify @ stub SslComputeClientAuthHash @ stub SslComputeEapKeyBlock

1 0

Jacek Caban : user32: Delay load setupapi.
by Alexandre Julliard 17 Feb '22

17 Feb '22

Module: wine Branch: master Commit: cc0ad23d7365b7b668fe0d3ba072d2a8dda94853 URL: https://source.winehq.org/git/wine.git/?a=commit;h=cc0ad23d7365b7b668fe0d3b… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Feb 17 15:41:30 2022 +0100 user32: Delay load setupapi. This reverts 50aeb5e777d9a8836f5530755afab10e042c623f. Previously problematic code is now moved to win32u and doesn't use setupapi. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/Makefile.in | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/user32/Makefile.in b/dlls/user32/Makefile.in index b3970d44e4d..73f6016933a 100644 --- a/dlls/user32/Makefile.in +++ b/dlls/user32/Makefile.in @@ -1,9 +1,9 @@ EXTRADEFS = -DWINE_NO_LONG_TYPES -D_USER32_ -D_WINABLE_ MODULE = user32.dll IMPORTLIB = user32 -IMPORTS = $(PNG_PE_LIBS) setupapi gdi32 version sechost advapi32 kernelbase win32u +IMPORTS = $(PNG_PE_LIBS) gdi32 version sechost advapi32 kernelbase win32u EXTRAINCL = $(PNG_PE_CFLAGS) -DELAYIMPORTS = hid imm32 +DELAYIMPORTS = hid setupapi imm32 C_SRCS = \ button.c \

1 0

Jacek Caban : gdi32: Link directly to user32.
by Alexandre Julliard 17 Feb '22

17 Feb '22

Module: wine Branch: master Commit: 2353ef9bd5c509b3903a5c6d9cb9690907af9427 URL: https://source.winehq.org/git/wine.git/?a=commit;h=2353ef9bd5c509b3903a5c6d… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Feb 17 15:42:01 2022 +0100 gdi32: Link directly to user32. On Windows, loading gdi32.dll causes user32.dll to be loaded. It's important for win32u, which is not fully functional until user32 sets KernelCallbackTable. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdi32/Makefile.in | 2 +- dlls/gdi32/objects.c | 25 ++++--------------------- 2 files changed, 5 insertions(+), 22 deletions(-) diff --git a/dlls/gdi32/Makefile.in b/dlls/gdi32/Makefile.in index 3e0495f877b..ce877995ee2 100644 --- a/dlls/gdi32/Makefile.in +++ b/dlls/gdi32/Makefile.in @@ -1,7 +1,7 @@ EXTRADEFS = -D_GDI32_ MODULE = gdi32.dll IMPORTLIB = gdi32 -IMPORTS = advapi32 win32u +IMPORTS = user32 advapi32 win32u DELAYIMPORTS = setupapi C_SRCS = \ diff --git a/dlls/gdi32/objects.c b/dlls/gdi32/objects.c index f578543b937..0260027a484 100644 --- a/dlls/gdi32/objects.c +++ b/dlls/gdi32/objects.c @@ -415,23 +415,6 @@ HGDIOBJ WINAPI GetCurrentObject( HDC hdc, UINT type ) return NtGdiGetDCObject( hdc, obj_type ); } -/****************************************************************************** - * get_system_dpi - * - * Get the system DPI, based on the DPI awareness mode. - */ -static DWORD get_system_dpi(void) -{ - static UINT (WINAPI *pGetDpiForSystem)(void); - - if (!pGetDpiForSystem) - { - HMODULE user = GetModuleHandleW( L"user32.dll" ); - if (user) pGetDpiForSystem = (void *)GetProcAddress( user, "GetDpiForSystem" ); - } - return pGetDpiForSystem ? pGetDpiForSystem() : 96; -} - /*********************************************************************** * GetStockObject (GDI32.@) */ @@ -443,16 +426,16 @@ HGDIOBJ WINAPI GetStockObject( INT obj ) switch (obj) { case OEM_FIXED_FONT: - if (get_system_dpi() != 96) obj = 9; + if (GetDpiForSystem() != 96) obj = 9; break; case SYSTEM_FONT: - if (get_system_dpi() != 96) obj = STOCK_LAST + 2; + if (GetDpiForSystem() != 96) obj = STOCK_LAST + 2; break; case SYSTEM_FIXED_FONT: - if (get_system_dpi() != 96) obj = STOCK_LAST + 3; + if (GetDpiForSystem() != 96) obj = STOCK_LAST + 3; break; case DEFAULT_GUI_FONT: - if (get_system_dpi() != 96) obj = STOCK_LAST + 4; + if (GetDpiForSystem() != 96) obj = STOCK_LAST + 4; break; }

1 0

Jacek Caban : win32u: Set user callbacks from user32 DllMain.
by Alexandre Julliard 17 Feb '22

17 Feb '22

Module: wine Branch: master Commit: e557f48cdf6ef1eafe468829ec9298a3eb6ac80b URL: https://source.winehq.org/git/wine.git/?a=commit;h=e557f48cdf6ef1eafe468829… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Feb 17 15:42:36 2022 +0100 win32u: Set user callbacks from user32 DllMain. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/user_main.c | 11 +++++++ dlls/win32u/driver.c | 12 ++------ dlls/win32u/ntuser_private.h | 9 ++++++ dlls/win32u/palette.c | 11 +------ dlls/win32u/region.c | 7 +---- dlls/win32u/sysparams.c | 2 ++ dlls/win32u/win32u_private.h | 9 ------ dlls/win32u/wrappers.c | 73 +------------------------------------------- include/ntuser.h | 1 + 9 files changed, 28 insertions(+), 107 deletions(-) diff --git a/dlls/user32/user_main.c b/dlls/user32/user_main.c index c2ec561a800..a91db57cdb4 100644 --- a/dlls/user32/user_main.c +++ b/dlls/user32/user_main.c @@ -208,6 +208,14 @@ static void dpiaware_init(void) } } +static const struct user_callbacks user_funcs = +{ + GetDesktopWindow, + GetWindowRect, + RedrawWindow, + SendMessageTimeoutW, + WindowFromDC, +}; static const void *kernel_callback_table[NtUserCallCount] = { @@ -223,6 +231,9 @@ static BOOL process_attach(void) { NtCurrentTeb()->Peb->KernelCallbackTable = kernel_callback_table; + /* FIXME: should not be needed */ + NtUserCallOneParam( (UINT_PTR)&user_funcs, NtUserSetCallbacks ); + dpiaware_init(); register_desktop_class(); diff --git a/dlls/win32u/driver.c b/dlls/win32u/driver.c index b3a3e70f83f..5e6cee33bc6 100644 --- a/dlls/win32u/driver.c +++ b/dlls/win32u/driver.c @@ -24,21 +24,13 @@ #endif #include <assert.h> -#include <stdarg.h> -#include <string.h> -#include <stdio.h> #include <pthread.h> #include "ntstatus.h" #define WIN32_NO_STATUS -#include "windef.h" -#include "winbase.h" -#include "wingdi.h" -#include "winreg.h" -#include "wine/winbase16.h" -#include "winuser.h" - #include "ntgdi_private.h" +#include "ntuser_private.h" +#include "wine/winbase16.h" #include "wine/list.h" #include "wine/debug.h" diff --git a/dlls/win32u/ntuser_private.h b/dlls/win32u/ntuser_private.h index fec04d94948..f0e0f8835bf 100644 --- a/dlls/win32u/ntuser_private.h +++ b/dlls/win32u/ntuser_private.h @@ -24,6 +24,15 @@ #include "ntuser.h" +struct user_callbacks +{ + HWND (WINAPI *pGetDesktopWindow)(void); + BOOL (WINAPI *pGetWindowRect)( HWND hwnd, LPRECT rect ); + BOOL (WINAPI *pRedrawWindow)( HWND, const RECT*, HRGN, UINT ); + LRESULT (WINAPI *pSendMessageTimeoutW)( HWND, UINT, WPARAM, LPARAM, UINT, UINT, PDWORD_PTR ); + HWND (WINAPI *pWindowFromDC)( HDC ); +}; + /* this is the structure stored in TEB->Win32ClientInfo */ /* no attempt is made to keep the layout compatible with the Windows one */ struct user_thread_info diff --git a/dlls/win32u/palette.c b/dlls/win32u/palette.c index 3aeda89851f..1c2f2c70434 100644 --- a/dlls/win32u/palette.c +++ b/dlls/win32u/palette.c @@ -27,17 +27,8 @@ #pragma makedep unix #endif -#include <stdarg.h> -#include <stdlib.h> -#include <string.h> - -#include "windef.h" -#include "winbase.h" -#include "winerror.h" -#include "wingdi.h" -#include "winuser.h" - #include "ntgdi_private.h" +#include "ntuser_private.h" #include "wine/debug.h" WINE_DEFAULT_DEBUG_CHANNEL(palette); diff --git a/dlls/win32u/region.c b/dlls/win32u/region.c index 86554f21e1c..17eae861cbf 100644 --- a/dlls/win32u/region.c +++ b/dlls/win32u/region.c @@ -99,13 +99,8 @@ SOFTWARE. #endif #include <assert.h> -#include <stdarg.h> -#include <stdlib.h> -#include <string.h> -#include "windef.h" -#include "winbase.h" -#include "wingdi.h" #include "ntgdi_private.h" +#include "ntuser_private.h" #include "wine/debug.h" WINE_DEFAULT_DEBUG_CHANNEL(region); diff --git a/dlls/win32u/sysparams.c b/dlls/win32u/sysparams.c index c9885f89d87..7af1e20f008 100644 --- a/dlls/win32u/sysparams.c +++ b/dlls/win32u/sysparams.c @@ -4544,6 +4544,8 @@ ULONG_PTR WINAPI NtUserCallOneParam( ULONG_PTR arg, ULONG code ) return get_entry( &entry_DESKPATTERN, 256, (WCHAR *)arg ); case NtUserIncrementKeyStateCounter: return InterlockedAdd( &global_key_state_counter, arg ); + case NtUserSetCallbacks: + return (UINT_PTR)InterlockedExchangePointer( (void **)&user_callbacks, (void *)arg ); default: FIXME( "invalid code %u\n", code ); return 0; diff --git a/dlls/win32u/win32u_private.h b/dlls/win32u/win32u_private.h index c3419d83858..3017b7e6a72 100644 --- a/dlls/win32u/win32u_private.h +++ b/dlls/win32u/win32u_private.h @@ -30,15 +30,6 @@ #include "wine/unixlib.h" #include "wine/debug.h" -struct user_callbacks -{ - HWND (WINAPI *pGetDesktopWindow)(void); - BOOL (WINAPI *pGetWindowRect)( HWND hwnd, LPRECT rect ); - BOOL (WINAPI *pRedrawWindow)( HWND, const RECT*, HRGN, UINT ); - LRESULT (WINAPI *pSendMessageTimeoutW)( HWND, UINT, WPARAM, LPARAM, UINT, UINT, PDWORD_PTR ); - HWND (WINAPI *pWindowFromDC)( HDC ); -}; - extern const struct user_callbacks *user_callbacks DECLSPEC_HIDDEN; struct unix_funcs diff --git a/dlls/win32u/wrappers.c b/dlls/win32u/wrappers.c index 2823eae9fe1..922e5a26b00 100644 --- a/dlls/win32u/wrappers.c +++ b/dlls/win32u/wrappers.c @@ -947,79 +947,8 @@ void CDECL __wine_set_display_driver( struct user_driver_funcs *funcs, UINT vers return unix_funcs->set_display_driver( funcs, version ); } -static void *get_user_proc( const char *name, BOOL force_load ) -{ - ANSI_STRING name_str; - FARPROC proc = NULL; - static HMODULE user32; - - if (!user32) - { - UNICODE_STRING str; - NTSTATUS status; - RtlInitUnicodeString( &str, L"user32.dll" ); - status = force_load - ? LdrLoadDll( NULL, 0, &str, &user32 ) - : LdrGetDllHandle( NULL, 0, &str, &user32 ); - if (status < 0) return NULL; - } - RtlInitAnsiString( &name_str, name ); - LdrGetProcedureAddress( user32, &name_str, 0, (void**)&proc ); - return proc; -} - -static HWND WINAPI call_GetDesktopWindow(void) -{ - static HWND (WINAPI *pGetDesktopWindow)(void); - if (!pGetDesktopWindow) - pGetDesktopWindow = get_user_proc( "GetDesktopWindow", TRUE ); - return pGetDesktopWindow ? pGetDesktopWindow() : NULL; -} - -static BOOL WINAPI call_GetWindowRect( HWND hwnd, LPRECT rect ) -{ - static BOOL (WINAPI *pGetWindowRect)( HWND hwnd, LPRECT rect ); - if (!pGetWindowRect) - pGetWindowRect = get_user_proc( "GetWindowRect", FALSE ); - return pGetWindowRect && pGetWindowRect( hwnd, rect ); -} - -static BOOL WINAPI call_RedrawWindow( HWND hwnd, const RECT *rect, HRGN rgn, UINT flags ) -{ - static BOOL (WINAPI *pRedrawWindow)( HWND, const RECT*, HRGN, UINT ); - if (!pRedrawWindow) - pRedrawWindow = get_user_proc( "RedrawWindow", FALSE ); - return pRedrawWindow && pRedrawWindow( hwnd, rect, rgn, flags ); -} - -static LRESULT WINAPI call_SendMessageTimeoutW( HWND hwnd, UINT msg, WPARAM wparam, LPARAM lparam, - UINT flags, UINT timeout, PDWORD_PTR res_ptr ) -{ - static LRESULT (WINAPI *pSendMessageTimeoutW)( HWND, UINT, WPARAM, LPARAM, UINT, UINT, PDWORD_PTR ); - if (!pSendMessageTimeoutW) pSendMessageTimeoutW = get_user_proc( "SendMessageTimeoutW", FALSE ); - if (!pSendMessageTimeoutW) return 0; - return pSendMessageTimeoutW( hwnd, msg, wparam, lparam, flags, timeout, res_ptr ); -} - -static HWND WINAPI call_WindowFromDC( HDC hdc ) -{ - static HWND (WINAPI *pWindowFromDC)( HDC ); - if (!pWindowFromDC) - pWindowFromDC = get_user_proc( "WindowFromDC", FALSE ); - return pWindowFromDC ? pWindowFromDC( hdc ) : NULL; -} - -static const struct user_callbacks user_funcs = -{ - call_GetDesktopWindow, - call_GetWindowRect, - call_RedrawWindow, - call_SendMessageTimeoutW, - call_WindowFromDC, -}; - extern void wrappers_init( unixlib_handle_t handle ) { - const void *args = &user_funcs; + const void *args; if (!__wine_unix_call( handle, 1, &args )) unix_funcs = args; } diff --git a/include/ntuser.h b/include/ntuser.h index 6e4183b1e14..6e941c29c6d 100644 --- a/include/ntuser.h +++ b/include/ntuser.h @@ -85,6 +85,7 @@ enum NtUserFlushWindowSurfaces, NtUserGetDeskPattern, NtUserIncrementKeyStateCounter, + NtUserSetCallbacks, }; /* NtUserCallTwoParam codes, not compatible with Windows */

1 0

Jacek Caban : win32u: Move NtUserGetForegroundWindow implementation from user32.
by Alexandre Julliard 17 Feb '22

17 Feb '22

Module: wine Branch: master Commit: fa3bd8f417123848c343c45f93a152cd42b3e5ff URL: https://source.winehq.org/git/wine.git/?a=commit;h=fa3bd8f417123848c343c45f… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Feb 17 15:43:07 2022 +0100 win32u: Move NtUserGetForegroundWindow implementation from user32. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/focus.c | 19 +------------------ dlls/user32/message.c | 2 +- dlls/user32/user32.spec | 2 +- dlls/user32/user_main.c | 2 +- dlls/user32/win.c | 2 +- dlls/user32/winpos.c | 2 +- dlls/win32u/input.c | 16 ++++++++++++++++ dlls/win32u/syscall.c | 1 + dlls/win32u/win32u.spec | 2 +- dlls/wow64win/syscall.h | 1 + dlls/wow64win/user.c | 5 +++++ include/ntuser.h | 1 + 12 files changed, 31 insertions(+), 24 deletions(-) diff --git a/dlls/user32/focus.c b/dlls/user32/focus.c index 4ba2bfef193..a5fc376bdc6 100644 --- a/dlls/user32/focus.c +++ b/dlls/user32/focus.c @@ -161,7 +161,7 @@ static BOOL set_active_window( HWND hwnd, HWND *prev, BOOL mouse, BOOL focus ) if (IsWindow(hwnd)) { - SendMessageW( hwnd, WM_NCACTIVATE, (hwnd == GetForegroundWindow()), (LPARAM)previous ); + SendMessageW( hwnd, WM_NCACTIVATE, hwnd == NtUserGetForegroundWindow(), (LPARAM)previous ); SendMessageW( hwnd, WM_ACTIVATE, MAKEWPARAM( mouse ? WA_CLICKACTIVE : WA_ACTIVE, IsIconic(hwnd) ), (LPARAM)previous ); @@ -371,23 +371,6 @@ HWND WINAPI GetFocus(void) } -/******************************************************************* - * GetForegroundWindow (USER32.@) - */ -HWND WINAPI GetForegroundWindow(void) -{ - HWND ret = 0; - - SERVER_START_REQ( get_thread_input ) - { - req->tid = 0; - if (!wine_server_call_err( req )) ret = wine_server_ptr_handle( reply->foreground ); - } - SERVER_END_REQ; - return ret; -} - - /*********************************************************************** * SetShellWindowEx (USER32.@) * hwndShell = Progman[Program Manager] diff --git a/dlls/user32/message.c b/dlls/user32/message.c index a5fcc9068a7..b4850cc1937 100644 --- a/dlls/user32/message.c +++ b/dlls/user32/message.c @@ -1876,7 +1876,7 @@ static LRESULT handle_internal_message( HWND hwnd, UINT msg, WPARAM wparam, LPAR if (is_desktop_window( hwnd )) return 0; return WIN_SetStyle(hwnd, wparam, lparam); case WM_WINE_SETACTIVEWINDOW: - if (!wparam && GetForegroundWindow() == hwnd) return 0; + if (!wparam && NtUserGetForegroundWindow() == hwnd) return 0; return (LRESULT)SetActiveWindow( (HWND)wparam ); case WM_WINE_KEYBOARD_LL_HOOK: case WM_WINE_MOUSE_LL_HOOK: diff --git a/dlls/user32/user32.spec b/dlls/user32/user32.spec index d60b424321b..3baa7594e84 100644 --- a/dlls/user32/user32.spec +++ b/dlls/user32/user32.spec @@ -302,7 +302,7 @@ @ stdcall GetDpiForSystem() @ stdcall GetDpiForWindow(long) @ stdcall GetFocus() -@ stdcall GetForegroundWindow() +@ stdcall GetForegroundWindow() NtUserGetForegroundWindow @ stdcall GetGestureConfig(long long long ptr ptr long) @ stdcall GetGestureInfo(long ptr) @ stdcall GetGUIThreadInfo(long ptr) diff --git a/dlls/user32/user_main.c b/dlls/user32/user_main.c index a91db57cdb4..8ca55b283f7 100644 --- a/dlls/user32/user_main.c +++ b/dlls/user32/user_main.c @@ -97,7 +97,7 @@ static HPALETTE WINAPI UserSelectPalette( HDC hDC, HPALETTE hPal, BOOL bForceBac HWND hwnd = WindowFromDC( hDC ); if (hwnd) { - HWND hForeground = GetForegroundWindow(); + HWND hForeground = NtUserGetForegroundWindow(); /* set primary palette if it's related to current active */ if (hForeground == hwnd || IsChild(hForeground,hwnd)) { diff --git a/dlls/user32/win.c b/dlls/user32/win.c index ee16144b179..1a037297ac1 100644 --- a/dlls/user32/win.c +++ b/dlls/user32/win.c @@ -3843,7 +3843,7 @@ BOOL WINAPI FlashWindowEx( PFLASHWINFO pfinfo ) hwnd = wndPtr->obj.handle; /* make it a full handle */ if (pfinfo->dwFlags) wparam = !(wndPtr->flags & WIN_NCACTIVATED); - else wparam = (hwnd == GetForegroundWindow()); + else wparam = (hwnd == NtUserGetForegroundWindow()); WIN_ReleasePtr( wndPtr ); SendMessageW( hwnd, WM_NCACTIVATE, wparam, 0 ); diff --git a/dlls/user32/winpos.c b/dlls/user32/winpos.c index abf9c25e6c6..db3408c63ba 100644 --- a/dlls/user32/winpos.c +++ b/dlls/user32/winpos.c @@ -1640,7 +1640,7 @@ void WINPOS_ActivateOtherWindow(HWND hwnd) } done: - fg = GetForegroundWindow(); + fg = NtUserGetForegroundWindow(); TRACE("win = %p fg = %p\n", hwndTo, fg); if (!fg || (hwnd == fg)) { diff --git a/dlls/win32u/input.c b/dlls/win32u/input.c index d73b03fe7fd..9c7a7e75253 100644 --- a/dlls/win32u/input.c +++ b/dlls/win32u/input.c @@ -1004,3 +1004,19 @@ int WINAPI NtUserGetMouseMovePointsEx( UINT size, MOUSEMOVEPOINT *ptin, MOUSEMOV return copied; } + +/******************************************************************* + * NtUserGetForegroundWindow (win32u.@) + */ +HWND WINAPI NtUserGetForegroundWindow(void) +{ + HWND ret = 0; + + SERVER_START_REQ( get_thread_input ) + { + req->tid = 0; + if (!wine_server_call_err( req )) ret = wine_server_ptr_handle( reply->foreground ); + } + SERVER_END_REQ; + return ret; +} diff --git a/dlls/win32u/syscall.c b/dlls/win32u/syscall.c index d97688d7c8c..2d37d605479 100644 --- a/dlls/win32u/syscall.c +++ b/dlls/win32u/syscall.c @@ -115,6 +115,7 @@ static void * const syscalls[] = NtUserGetCursor, NtUserGetDoubleClickTime, NtUserGetDpiForMonitor, + NtUserGetForegroundWindow, NtUserGetKeyState, NtUserGetKeyboardLayout, NtUserGetKeyboardLayoutName, diff --git a/dlls/win32u/win32u.spec b/dlls/win32u/win32u.spec index 20faca70c5e..03ab1ff885c 100644 --- a/dlls/win32u/win32u.spec +++ b/dlls/win32u/win32u.spec @@ -926,7 +926,7 @@ @ stub NtUserGetDpiForCurrentProcess @ stdcall -syscall NtUserGetDpiForMonitor(long long ptr ptr) @ stub NtUserGetExtendedPointerDeviceProperty -@ stub NtUserGetForegroundWindow +@ stdcall -syscall NtUserGetForegroundWindow() @ stub NtUserGetGUIThreadInfo @ stub NtUserGetGestureConfig @ stub NtUserGetGestureExtArgs diff --git a/dlls/wow64win/syscall.h b/dlls/wow64win/syscall.h index 47e95a98a4b..22140c33f80 100644 --- a/dlls/wow64win/syscall.h +++ b/dlls/wow64win/syscall.h @@ -102,6 +102,7 @@ SYSCALL_ENTRY( NtUserGetCursor ) \ SYSCALL_ENTRY( NtUserGetDoubleClickTime ) \ SYSCALL_ENTRY( NtUserGetDpiForMonitor ) \ + SYSCALL_ENTRY( NtUserGetForegroundWindow ) \ SYSCALL_ENTRY( NtUserGetKeyState ) \ SYSCALL_ENTRY( NtUserGetKeyboardLayout ) \ SYSCALL_ENTRY( NtUserGetKeyboardLayoutName ) \ diff --git a/dlls/wow64win/user.c b/dlls/wow64win/user.c index cd4dc32d407..a2be5fedb36 100644 --- a/dlls/wow64win/user.c +++ b/dlls/wow64win/user.c @@ -415,3 +415,8 @@ NTSTATUS WINAPI wow64_NtUserUnhookWindowsHookEx( UINT *args ) return NtUserUnhookWindowsHookEx( handle ); } + +NTSTATUS WINAPI wow64_NtUserGetForegroundWindow( UINT *args ) +{ + return HandleToUlong( NtUserGetForegroundWindow() ); +} diff --git a/include/ntuser.h b/include/ntuser.h index 6e941c29c6d..51f1c624ba6 100644 --- a/include/ntuser.h +++ b/include/ntuser.h @@ -156,6 +156,7 @@ LONG WINAPI NtUserGetDisplayConfigBufferSizes( UINT32 flags, UINT32 *num_path UINT32 *num_mode_info ); UINT WINAPI NtUserGetDoubleClickTime(void); BOOL WINAPI NtUserGetDpiForMonitor( HMONITOR monitor, UINT type, UINT *x, UINT *y ); +HWND WINAPI NtUserGetForegroundWindow(void); INT WINAPI NtUserGetKeyNameText( LONG lparam, WCHAR *buffer, INT size ); SHORT WINAPI NtUserGetKeyState( INT vkey ); HKL WINAPI NtUserGetKeyboardLayout( DWORD thread_id );

1 0

Jacek Caban : win32u: Move primary palette handling from user32.
by Alexandre Julliard 17 Feb '22

17 Feb '22

Module: wine Branch: master Commit: 4b0c0a6b23378d46f580591655c7e613a6808afc URL: https://source.winehq.org/git/wine.git/?a=commit;h=4b0c0a6b23378d46f5805916… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Feb 17 15:43:40 2022 +0100 win32u: Move primary palette handling from user32. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/user_main.c | 72 +++----------------------------------------- dlls/win32u/ntuser_private.h | 1 + dlls/win32u/palette.c | 26 ++++++++++++++-- 3 files changed, 29 insertions(+), 70 deletions(-) diff --git a/dlls/user32/user_main.c b/dlls/user32/user_main.c index 8ca55b283f7..ea30772067d 100644 --- a/dlls/user32/user_main.c +++ b/dlls/user32/user_main.c @@ -44,10 +44,6 @@ static CRITICAL_SECTION_DEBUG critsect_debug = }; static CRITICAL_SECTION user_section = { &critsect_debug, -1, 0, 0, 0, 0 }; -static HPALETTE (WINAPI *pfnGDISelectPalette)( HDC hdc, HPALETTE hpal, WORD bkgnd ); -static UINT (WINAPI *pfnGDIRealizePalette)( HDC hdc ); -static HPALETTE hPrimaryPalette; - static DWORD exiting_thread_id; extern void WDML_NotifyThreadDetach(void); @@ -86,69 +82,11 @@ void USER_CheckNotLock(void) /*********************************************************************** - * UserSelectPalette (Not a Windows API) + * UserRealizePalette (USER32.@) */ -static HPALETTE WINAPI UserSelectPalette( HDC hDC, HPALETTE hPal, BOOL bForceBackground ) +UINT WINAPI UserRealizePalette( HDC hdc ) { - WORD wBkgPalette = 1; - - if (!bForceBackground && (hPal != GetStockObject(DEFAULT_PALETTE))) - { - HWND hwnd = WindowFromDC( hDC ); - if (hwnd) - { - HWND hForeground = NtUserGetForegroundWindow(); - /* set primary palette if it's related to current active */ - if (hForeground == hwnd || IsChild(hForeground,hwnd)) - { - wBkgPalette = 0; - hPrimaryPalette = hPal; - } - } - } - return pfnGDISelectPalette( hDC, hPal, wBkgPalette); -} - - -/*********************************************************************** - * UserRealizePalette (USER32.@) - */ -UINT WINAPI UserRealizePalette( HDC hDC ) -{ - UINT realized = pfnGDIRealizePalette( hDC ); - - /* do not send anything if no colors were changed */ - if (realized && GetCurrentObject( hDC, OBJ_PAL ) == hPrimaryPalette) - { - /* send palette change notification */ - HWND hWnd = WindowFromDC( hDC ); - if (hWnd) SendMessageTimeoutW( HWND_BROADCAST, WM_PALETTECHANGED, (WPARAM)hWnd, 0, - SMTO_ABORTIFHUNG, 2000, NULL ); - } - return realized; -} - - -/*********************************************************************** - * palette_init - * - * Patch the function pointers in GDI for SelectPalette and RealizePalette - */ -static void palette_init(void) -{ - void **ptr; - HMODULE module = GetModuleHandleA( "gdi32" ); - if (!module) - { - ERR( "cannot get GDI32 handle\n" ); - return; - } - if ((ptr = (void**)GetProcAddress( module, "pfnSelectPalette" ))) - pfnGDISelectPalette = InterlockedExchangePointer( ptr, UserSelectPalette ); - else ERR( "cannot find pfnSelectPalette in GDI32\n" ); - if ((ptr = (void**)GetProcAddress( module, "pfnRealizePalette" ))) - pfnGDIRealizePalette = InterlockedExchangePointer( ptr, UserRealizePalette ); - else ERR( "cannot find pfnRealizePalette in GDI32\n" ); + return NtUserCallOneParam( HandleToUlong(hdc), NtUserRealizePalette ); } @@ -212,6 +150,7 @@ static const struct user_callbacks user_funcs = { GetDesktopWindow, GetWindowRect, + IsChild, RedrawWindow, SendMessageTimeoutW, WindowFromDC, @@ -240,9 +179,6 @@ static BOOL process_attach(void) /* Initialize system colors and metrics */ SYSPARAMS_Init(); - /* Setup palette function pointers */ - palette_init(); - return TRUE; } diff --git a/dlls/win32u/ntuser_private.h b/dlls/win32u/ntuser_private.h index f0e0f8835bf..e9d19744976 100644 --- a/dlls/win32u/ntuser_private.h +++ b/dlls/win32u/ntuser_private.h @@ -28,6 +28,7 @@ struct user_callbacks { HWND (WINAPI *pGetDesktopWindow)(void); BOOL (WINAPI *pGetWindowRect)( HWND hwnd, LPRECT rect ); + BOOL (WINAPI *pIsChild)( HWND, HWND ); BOOL (WINAPI *pRedrawWindow)( HWND, const RECT*, HRGN, UINT ); LRESULT (WINAPI *pSendMessageTimeoutW)( HWND, UINT, WPARAM, LPARAM, UINT, UINT, PDWORD_PTR ); HWND (WINAPI *pWindowFromDC)( HDC ); diff --git a/dlls/win32u/palette.c b/dlls/win32u/palette.c index 1c2f2c70434..d09fc9d4d73 100644 --- a/dlls/win32u/palette.c +++ b/dlls/win32u/palette.c @@ -508,12 +508,22 @@ static BOOL PALETTE_DeleteObject( HGDIOBJ handle ) */ HPALETTE WINAPI NtUserSelectPalette( HDC hdc, HPALETTE hpal, WORD bkg ) { + BOOL is_primary = FALSE; HPALETTE ret = 0; DC *dc; TRACE("%p %p\n", hdc, hpal ); - /* FIXME: move primary palette handling from user32 */ + if (!bkg && hpal != get_stock_object( DEFAULT_PALETTE )) + { + HWND hwnd = user_callbacks->pWindowFromDC( hdc ); + if (hwnd) + { + /* set primary palette if it's related to current active */ + HWND foreground = NtUserGetForegroundWindow(); + is_primary = foreground == hwnd || user_callbacks->pIsChild( foreground, hwnd ); + } + } if (get_gdi_object_type(hpal) != NTGDI_OBJ_PAL) { @@ -524,7 +534,7 @@ HPALETTE WINAPI NtUserSelectPalette( HDC hdc, HPALETTE hpal, WORD bkg ) { ret = dc->hPalette; dc->hPalette = hpal; - if (!bkg) hPrimaryPalette = hpal; + if (is_primary) hPrimaryPalette = hpal; release_dc_ptr( dc ); } return ret; @@ -536,6 +546,7 @@ HPALETTE WINAPI NtUserSelectPalette( HDC hdc, HPALETTE hpal, WORD bkg ) */ UINT realize_palette( HDC hdc ) { + BOOL is_primary = FALSE; UINT realized = 0; DC* dc = get_dc_ptr( hdc ); @@ -559,12 +570,23 @@ UINT realize_palette( HDC hdc ) (dc->hPalette == hPrimaryPalette) ); palPtr->unrealize = physdev->funcs->pUnrealizePalette; GDI_ReleaseObj( dc->hPalette ); + is_primary = dc->hPalette == hPrimaryPalette; } } else TRACE(" skipping (hLastRealizedPalette = %p)\n", hLastRealizedPalette); release_dc_ptr( dc ); TRACE(" realized %i colors.\n", realized ); + + /* do not send anything if no colors were changed */ + if (realized && is_primary) + { + /* send palette change notification */ + HWND hwnd = user_callbacks->pWindowFromDC( hdc ); + if (hwnd) user_callbacks->pSendMessageTimeoutW( HWND_BROADCAST, WM_PALETTECHANGED, + HandleToUlong(hwnd), 0, SMTO_ABORTIFHUNG, + 2000, NULL ); + } return realized; }

1 0