Re: [PATCH v2 0/2] MR419: win32u/window: Ignore SetWindowPos() for message-only windows.

[Jinoh Kang (＠iamahuman)]

Jinoh Kang (@iamahuman) commented about dlls/user32/tests/win.c:
> +    cls.lpfnWndProc = message_window_topmost_proc;
> +    cls.hInstance = GetModuleHandleW(NULL);
> +    cls.lpszClassName = L"test_message_window_topmost";
> +    atom = RegisterClassW(&cls);
> +    ok(!!atom, "Cannot register window class\n");
> +
> +    hwnd = CreateWindowW(L"test_message_window_topmost", L"main window", WS_OVERLAPPEDWINDOW | WS_VISIBLE,
> +            CW_USEDEFAULT, CW_USEDEFAULT, CW_USEDEFAULT, CW_USEDEFAULT, NULL, NULL, GetModuleHandleW(NULL), NULL);
> +    ok(!!hwnd, "Cannot create main window\n");
> +
> +    hwnd_msg = CreateWindowW(L"test_message_window_topmost", L"message window", 0,
> +            CW_USEDEFAULT, CW_USEDEFAULT, CW_USEDEFAULT, CW_USEDEFAULT, HWND_MESSAGE, NULL, GetModuleHandleW(NULL), NULL);
> +    ok(!!hwnd_msg, "Cannot create message window\n");
> +
> +    SetLastError(0xdeadbeef);
> +    ret = SetWindowPos(hwnd_msg, HWND_TOPMOST, 0, 0, 0, 0, SWP_NOMOVE | SWP_NOSIZE);
> Maybe extend the test to touching every aspect of the window position / z-order?
> After all, this MR introduces a drastic change of SetWindowPos semantics w.r.t message windows.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/419#note_3967