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.