[PATCH 1/4] wined3d: Show the device window when changing fullscreen resolutions.
Stefan Dösinger
stefan at codeweavers.com
Thu Oct 12 10:14:27 CDT 2017
World of Warplanes calls SetWindowLong(window, GWL_STYLE, WS_POPUP)
before calling reset, effectively hiding the window. It does not call
SetWindowPos(SWP_FRAMECHANGED), so the effect isn't visible and the game
displays fine after the reset. However, after the next mouse click
WINPOS_WindowFromPoint skips the window because it doesn't have
WS_VISIBLE and returns the desktop window. This in turn triggers a focus
loss and d3d9 minimizes the game.
Signed-off-by: Stefan Dösinger <stefan at codeweavers.com>
---
dlls/d3d9/tests/d3d9ex.c | 133 +++++++++++++++++++++++++++++++++++++++++++--
dlls/d3d9/tests/device.c | 136 +++++++++++++++++++++++++++++++++++++++++++++--
dlls/wined3d/swapchain.c | 1 +
3 files changed, 262 insertions(+), 8 deletions(-)
diff --git a/dlls/d3d9/tests/d3d9ex.c b/dlls/d3d9/tests/d3d9ex.c
index 6bd8139e90..34c7112058 100644
--- a/dlls/d3d9/tests/d3d9ex.c
+++ b/dlls/d3d9/tests/d3d9ex.c
@@ -2451,6 +2451,7 @@ struct message
enum message_window window;
BOOL check_wparam;
WPARAM expect_wparam;
+ WINDOWPOS *store_wp;
};
static const struct message *expect_messages;
@@ -2499,6 +2500,9 @@ static LRESULT CALLBACK test_proc(HWND hwnd, UINT message, WPARAM wparam, LPARAM
"Got unexpected wparam %lx for message %x, expected %lx.\n",
wparam, message, expect_messages->expect_wparam);
+ if (expect_messages->store_wp)
+ *expect_messages->store_wp = *(WINDOWPOS *)lparam;
+
++expect_messages;
}
}
@@ -2566,9 +2570,10 @@ static void test_wndproc(void)
D3DDISPLAYMODE d3ddm;
DWORD d3d_width = 0, d3d_height = 0, user32_width = 0, user32_height = 0;
DEVMODEW devmode;
- LONG change_ret;
+ LONG change_ret, device_style;
BOOL ret;
IDirect3D9Ex *d3d9ex;
+ WINDOWPOS windowpos, windowpos2;
static const struct message create_messages[] =
{
@@ -2661,16 +2666,59 @@ static void test_wndproc(void)
/* WM_SIZE(SIZE_MAXIMIZED) is unreliable on native. */
{0, 0, FALSE, 0},
};
- static const struct
+ struct message mode_change_messages[] =
+ {
+ {WM_WINDOWPOSCHANGING, DEVICE_WINDOW, FALSE, 0, &windowpos},
+ {WM_WINDOWPOSCHANGED, DEVICE_WINDOW, FALSE, 0},
+ {WM_SIZE, DEVICE_WINDOW, FALSE, 0},
+ /* TODO: WM_DISPLAYCHANGE is sent to the focus window too, but the order is
+ * differs between Wine and Windows. */
+ /* TODO 2: Windows sends a second WM_WINDOWPOSCHANGING(SWP_NOMOVE | SWP_NOSIZE
+ * | SWP_NOACTIVATE) in this situation, suggesting a difference in their ShowWindow
+ * implementation. This SetWindowPos call could in theory affect the Z order. Wine's
+ * ShowWindow does not send such a message because the window is already visible. */
+ {0, 0, FALSE, 0},
+ };
+ struct message mode_change_messages_hidden[] =
+ {
+ {WM_WINDOWPOSCHANGING, DEVICE_WINDOW, FALSE, 0, &windowpos},
+ {WM_WINDOWPOSCHANGED, DEVICE_WINDOW, FALSE, 0},
+ {WM_SIZE, DEVICE_WINDOW, FALSE, 0},
+ {WM_SHOWWINDOW, DEVICE_WINDOW, FALSE, 0},
+ {WM_WINDOWPOSCHANGING, DEVICE_WINDOW, FALSE, 0, &windowpos2},
+ {WM_WINDOWPOSCHANGED, DEVICE_WINDOW, FALSE, 0},
+ /* TODO: WM_DISPLAYCHANGE is sent to the focus window too, but the order is
+ * differs between Wine and Windows. */
+ {0, 0, FALSE, 0},
+ };
+ static const struct message mode_change_messages_nowc[] =
+ {
+ {WM_DISPLAYCHANGE, FOCUS_WINDOW, FALSE, 0},
+ {0, 0, FALSE, 0},
+ };
+ struct
{
DWORD create_flags;
const struct message *focus_loss_messages;
+ const struct message *mode_change_messages, *mode_change_messages_hidden;
BOOL iconic;
}
tests[] =
{
- {0, focus_loss_messages, TRUE},
- {CREATE_DEVICE_NOWINDOWCHANGES, focus_loss_messages_nowc, FALSE},
+ {
+ 0,
+ focus_loss_messages,
+ mode_change_messages,
+ mode_change_messages_hidden,
+ TRUE
+ },
+ {
+ CREATE_DEVICE_NOWINDOWCHANGES,
+ focus_loss_messages_nowc,
+ mode_change_messages_nowc,
+ mode_change_messages_nowc,
+ FALSE
+ },
};
hr = pDirect3DCreate9Ex(D3D_SDK_VERSION, &d3d9ex);
@@ -3032,6 +3080,83 @@ static void test_wndproc(void)
skip("Failed to create a D3D device, skipping tests.\n");
goto done;
}
+ filter_messages = NULL;
+ flush_events();
+
+ device_desc.width = user32_width;
+ device_desc.height = user32_height;
+ memset(&windowpos, 0, sizeof(windowpos));
+ memset(&windowpos2, 0, sizeof(windowpos2));
+
+ expect_messages = tests[i].mode_change_messages;
+ filter_messages = focus_window;
+ hr = reset_device(device, &device_desc);
+ ok(SUCCEEDED(hr), "Failed to reset device, hr %#x.\n", hr);
+ filter_messages = NULL;
+
+ flush_events();
+ ok(!expect_messages->message, "Expected message %#x for window %#x, but didn't receive it, i=%u.\n",
+ expect_messages->message, expect_messages->window, i);
+
+ if (!(tests[i].create_flags & CREATE_DEVICE_NOWINDOWCHANGES))
+ {
+ ok(windowpos.hwnd == device_window && !windowpos.hwndInsertAfter
+ && !windowpos.x && !windowpos.y
+ && windowpos.cx == device_desc.width && windowpos.cy == device_desc.height
+ && windowpos.flags == (SWP_NOACTIVATE | SWP_NOZORDER),
+ "Got unexpected WINDOWPOS hwnd=%p, insertAfter=%p, x=%d, y=%d, cx=%d, cy=%d, flags=%x\n",
+ windowpos.hwnd, windowpos.hwndInsertAfter, windowpos.x, windowpos.y, windowpos.cx,
+ windowpos.cy, windowpos.flags);
+ }
+
+ /* World of Warplanes hides the window by removing WS_VISIBLE and expects Reset() to show it again. */
+ device_style = GetWindowLongA(device_window, GWL_STYLE);
+ SetWindowLongA(device_window, GWL_STYLE, device_style & ~WS_VISIBLE);
+
+ flush_events();
+ device_desc.width = d3d_width;
+ device_desc.height = d3d_height;
+ memset(&windowpos, 0, sizeof(windowpos));
+ memset(&windowpos2, 0, sizeof(windowpos2));
+
+ expect_messages = tests[i].mode_change_messages_hidden;
+ filter_messages = focus_window;
+ hr = reset_device(device, &device_desc);
+ ok(SUCCEEDED(hr), "Failed to reset device, hr %#x.\n", hr);
+ filter_messages = NULL;
+
+ flush_events();
+ ok(!expect_messages->message, "Expected message %#x for window %#x, but didn't receive it, i=%u.\n",
+ expect_messages->message, expect_messages->window, i);
+
+ if (!(tests[i].create_flags & CREATE_DEVICE_NOWINDOWCHANGES))
+ {
+ ok(windowpos.hwnd == device_window && !windowpos.hwndInsertAfter
+ && !windowpos.x && !windowpos.y
+ && windowpos.cx == device_desc.width && windowpos.cy == device_desc.height
+ && windowpos.flags == (SWP_NOACTIVATE | SWP_NOZORDER),
+ "Got unexpected WINDOWPOS hwnd=%p, insertAfter=%p, x=%d, y=%d, cx=%d, cy=%d, flags=%x\n",
+ windowpos.hwnd, windowpos.hwndInsertAfter, windowpos.x, windowpos.y, windowpos.cx,
+ windowpos.cy, windowpos.flags);
+ ok(windowpos2.hwnd == device_window && !windowpos2.hwndInsertAfter
+ && !windowpos2.x && !windowpos2.y && !windowpos2.cx && !windowpos2.cy
+ && windowpos2.flags == (SWP_SHOWWINDOW | SWP_NOMOVE | SWP_NOSIZE),
+ "Got unexpected WINDOWPOS hwnd=%p, insertAfter=%p, x=%d, y=%d, cx=%d, cy=%d, flags=%x\n",
+ windowpos2.hwnd, windowpos2.hwndInsertAfter, windowpos2.x, windowpos2.y, windowpos2.cx,
+ windowpos2.cy, windowpos2.flags);
+ }
+
+ device_style = GetWindowLongA(device_window, GWL_STYLE);
+ if (tests[i].create_flags & CREATE_DEVICE_NOWINDOWCHANGES)
+ {
+ todo_wine ok(!(device_style & WS_VISIBLE), "Expected the device window to be hidden, i=%u.\n", i);
+ ShowWindow(device_window, SW_MINIMIZE);
+ ShowWindow(device_window, SW_RESTORE);
+ }
+ else
+ {
+ ok(device_style & WS_VISIBLE, "Expected the device window to be visible, i=%u.\n", i);
+ }
proc = SetWindowLongPtrA(focus_window, GWLP_WNDPROC, (LONG_PTR)DefWindowProcA);
ok(proc != (LONG_PTR)test_proc, "Expected wndproc != %#lx.\n",
diff --git a/dlls/d3d9/tests/device.c b/dlls/d3d9/tests/device.c
index b24136af76..e1cb9ef345 100644
--- a/dlls/d3d9/tests/device.c
+++ b/dlls/d3d9/tests/device.c
@@ -3392,6 +3392,7 @@ struct message
enum message_window window;
BOOL check_wparam;
WPARAM expect_wparam;
+ WINDOWPOS *store_wp;
};
static const struct message *expect_messages;
@@ -3440,6 +3441,9 @@ static LRESULT CALLBACK test_proc(HWND hwnd, UINT message, WPARAM wparam, LPARAM
"Got unexpected wparam %lx for message %x, expected %lx.\n",
wparam, message, expect_messages->expect_wparam);
+ if (expect_messages->store_wp)
+ *expect_messages->store_wp = *(WINDOWPOS *)lparam;
+
++expect_messages;
}
}
@@ -3508,8 +3512,9 @@ static void test_wndproc(void)
D3DDISPLAYMODE d3ddm;
DWORD d3d_width = 0, d3d_height = 0, user32_width = 0, user32_height = 0;
DEVMODEW devmode;
- LONG change_ret;
+ LONG change_ret, device_style;
BOOL ret;
+ WINDOWPOS windowpos, windowpos2;
static const struct message create_messages[] =
{
@@ -3622,16 +3627,61 @@ static void test_wndproc(void)
/* WM_SIZE(SIZE_MAXIMIZED) is unreliable on native. */
{0, 0, FALSE, 0},
};
- static const struct
+ struct message mode_change_messages[] =
+ {
+ {WM_WINDOWPOSCHANGING, DEVICE_WINDOW, FALSE, 0, &windowpos},
+ {WM_WINDOWPOSCHANGED, DEVICE_WINDOW, FALSE, 0},
+ {WM_SIZE, DEVICE_WINDOW, FALSE, 0},
+ /* TODO: WM_DISPLAYCHANGE is sent to the focus window too, but the order is
+ * differs between Wine and Windows. */
+ /* TODO 2: Windows sends a second WM_WINDOWPOSCHANGING(SWP_NOMOVE | SWP_NOSIZE
+ * | SWP_NOACTIVATE) in this situation, suggesting a difference in their ShowWindow
+ * implementation. This SetWindowPos call could in theory affect the Z order. Wine's
+ * ShowWindow does not send such a message because the window is already visible. */
+ {0, 0, FALSE, 0},
+ };
+ struct message mode_change_messages_hidden[] =
+ {
+ {WM_WINDOWPOSCHANGING, DEVICE_WINDOW, FALSE, 0, &windowpos},
+ {WM_WINDOWPOSCHANGED, DEVICE_WINDOW, FALSE, 0},
+ {WM_SIZE, DEVICE_WINDOW, FALSE, 0},
+ {WM_SHOWWINDOW, DEVICE_WINDOW, FALSE, 0},
+ {WM_WINDOWPOSCHANGING, DEVICE_WINDOW, FALSE, 0, &windowpos2},
+ {WM_WINDOWPOSCHANGED, DEVICE_WINDOW, FALSE, 0},
+ /* TODO: WM_DISPLAYCHANGE is sent to the focus window too, but the order is
+ * differs between Wine and Windows. */
+ {0, 0, FALSE, 0},
+ };
+ static const struct message mode_change_messages_nowc[] =
+ {
+ {WM_DISPLAYCHANGE, FOCUS_WINDOW, FALSE, 0},
+ {0, 0, FALSE, 0},
+ };
+ struct
{
DWORD create_flags;
const struct message *focus_loss_messages, *reactivate_messages;
+ const struct message *mode_change_messages, *mode_change_messages_hidden;
BOOL iconic;
}
tests[] =
{
- {0, focus_loss_messages, reactivate_messages, TRUE},
- {CREATE_DEVICE_NOWINDOWCHANGES, focus_loss_messages_nowc, reactivate_messages_nowc, FALSE},
+ {
+ 0,
+ focus_loss_messages,
+ reactivate_messages,
+ mode_change_messages,
+ mode_change_messages_hidden,
+ TRUE
+ },
+ {
+ CREATE_DEVICE_NOWINDOWCHANGES,
+ focus_loss_messages_nowc,
+ reactivate_messages_nowc,
+ mode_change_messages_nowc,
+ mode_change_messages_nowc,
+ FALSE
+ },
};
d3d9 = Direct3DCreate9(D3D_SDK_VERSION);
@@ -4017,11 +4067,89 @@ static void test_wndproc(void)
skip("Failed to create a D3D device, skipping tests.\n");
goto done;
}
+ filter_messages = NULL;
+ flush_events();
+
+ device_desc.width = user32_width;
+ device_desc.height = user32_height;
+ memset(&windowpos, 0, sizeof(windowpos));
+ memset(&windowpos2, 0, sizeof(windowpos2));
+
+ expect_messages = tests[i].mode_change_messages;
+ filter_messages = focus_window;
+ hr = reset_device(device, &device_desc);
+ ok(SUCCEEDED(hr), "Failed to reset device, hr %#x.\n", hr);
+ filter_messages = NULL;
+
+ flush_events();
+ ok(!expect_messages->message, "Expected message %#x for window %#x, but didn't receive it, i=%u.\n",
+ expect_messages->message, expect_messages->window, i);
+
+ if (!(tests[i].create_flags & CREATE_DEVICE_NOWINDOWCHANGES))
+ {
+ ok(windowpos.hwnd == device_window && !windowpos.hwndInsertAfter
+ && !windowpos.x && !windowpos.y
+ && windowpos.cx == device_desc.width && windowpos.cy == device_desc.height
+ && windowpos.flags == (SWP_NOACTIVATE | SWP_NOZORDER),
+ "Got unexpected WINDOWPOS hwnd=%p, insertAfter=%p, x=%d, y=%d, cx=%d, cy=%d, flags=%x\n",
+ windowpos.hwnd, windowpos.hwndInsertAfter, windowpos.x, windowpos.y, windowpos.cx,
+ windowpos.cy, windowpos.flags);
+ }
+
+ /* World of Warplanes hides the window by removing WS_VISIBLE and expects Reset() to show it again. */
+ device_style = GetWindowLongA(device_window, GWL_STYLE);
+ SetWindowLongA(device_window, GWL_STYLE, device_style & ~WS_VISIBLE);
+
+ flush_events();
+ device_desc.width = d3d_width;
+ device_desc.height = d3d_height;
+ memset(&windowpos, 0, sizeof(windowpos));
+ memset(&windowpos2, 0, sizeof(windowpos2));
+
+ expect_messages = tests[i].mode_change_messages_hidden;
+ filter_messages = focus_window;
+ hr = reset_device(device, &device_desc);
+ ok(SUCCEEDED(hr), "Failed to reset device, hr %#x.\n", hr);
+ filter_messages = NULL;
+
+ flush_events();
+ ok(!expect_messages->message, "Expected message %#x for window %#x, but didn't receive it, i=%u.\n",
+ expect_messages->message, expect_messages->window, i);
+
+ if (!(tests[i].create_flags & CREATE_DEVICE_NOWINDOWCHANGES))
+ {
+ ok(windowpos.hwnd == device_window && !windowpos.hwndInsertAfter
+ && !windowpos.x && !windowpos.y
+ && windowpos.cx == device_desc.width && windowpos.cy == device_desc.height
+ && windowpos.flags == (SWP_NOACTIVATE | SWP_NOZORDER),
+ "Got unexpected WINDOWPOS hwnd=%p, insertAfter=%p, x=%d, y=%d, cx=%d, cy=%d, flags=%x\n",
+ windowpos.hwnd, windowpos.hwndInsertAfter, windowpos.x, windowpos.y, windowpos.cx,
+ windowpos.cy, windowpos.flags);
+ ok(windowpos2.hwnd == device_window && !windowpos2.hwndInsertAfter
+ && !windowpos2.x && !windowpos2.y && !windowpos2.cx && !windowpos2.cy
+ && windowpos2.flags == (SWP_SHOWWINDOW | SWP_NOMOVE | SWP_NOSIZE),
+ "Got unexpected WINDOWPOS hwnd=%p, insertAfter=%p, x=%d, y=%d, cx=%d, cy=%d, flags=%x\n",
+ windowpos2.hwnd, windowpos2.hwndInsertAfter, windowpos2.x, windowpos2.y, windowpos2.cx,
+ windowpos2.cy, windowpos2.flags);
+ }
+
+ device_style = GetWindowLongA(device_window, GWL_STYLE);
+ if (tests[i].create_flags & CREATE_DEVICE_NOWINDOWCHANGES)
+ {
+ todo_wine ok(!(device_style & WS_VISIBLE), "Expected the device window to be hidden, i=%u.\n", i);
+ ShowWindow(device_window, SW_MINIMIZE);
+ ShowWindow(device_window, SW_RESTORE);
+ }
+ else
+ {
+ ok(device_style & WS_VISIBLE, "Expected the device window to be visible, i=%u.\n", i);
+ }
proc = SetWindowLongPtrA(focus_window, GWLP_WNDPROC, (LONG_PTR)DefWindowProcA);
ok(proc != (LONG_PTR)test_proc, "Expected wndproc != %#lx, i=%u.\n",
(LONG_PTR)test_proc, i);
+ filter_messages = focus_window;
ref = IDirect3DDevice9_Release(device);
ok(ref == 0, "The device was not properly freed: refcount %u, i=%u.\n", ref, i);
diff --git a/dlls/wined3d/swapchain.c b/dlls/wined3d/swapchain.c
index e23e99b67b..a26185ae4d 100644
--- a/dlls/wined3d/swapchain.c
+++ b/dlls/wined3d/swapchain.c
@@ -1429,6 +1429,7 @@ HRESULT CDECL wined3d_swapchain_set_fullscreen(struct wined3d_swapchain *swapcha
device->filter_messages = TRUE;
MoveWindow(swapchain->device_window, 0, 0, width, height, TRUE);
+ ShowWindow(swapchain->device_window, SW_SHOWNORMAL);
device->filter_messages = filter_messages;
}
--
2.13.6
More information about the wine-patches
mailing list