mirror of
git://source.winehq.org/git/wine.git
synced 2024-10-06 08:35:33 +00:00
user32: Don't save maximized pos of top-level wnds covering whole work rect.
Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=51672 Signed-off-by: Gabriel Ivăncescu <gabrielopcode@gmail.com> Signed-off-by: Huw Davies <huw@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>
This commit is contained in:
Gabriel Ivăncescu
Alexandre Julliard
parent
d8252025d1
commit
54eea6f072
|
|
@ -11624,8 +11624,9 @@ static void test_IsWindowEnabled(void)
|
|||
|
||||
static void test_window_placement(void)
|
||||
{
|
||||
RECT orig = {100, 200, 300, 400}, orig2 = {200, 300, 400, 500}, rect;
|
||||
RECT orig = {100, 200, 300, 400}, orig2 = {200, 300, 400, 500}, rect, work_rect;
|
||||
WINDOWPLACEMENT wp = {sizeof(wp)};
|
||||
MONITORINFO mon_info;
|
||||
HWND hwnd;
|
||||
BOOL ret;
|
||||
|
||||
|
|
@ -11633,6 +11634,10 @@ static void test_window_placement(void)
|
|||
orig.left, orig.top, orig.right - orig.left, orig.bottom - orig.top, 0, 0, 0, 0);
|
||||
ok(!!hwnd, "failed to create window, error %u\n", GetLastError());
|
||||
|
||||
mon_info.cbSize = sizeof(mon_info);
|
||||
GetMonitorInfoW(MonitorFromWindow(hwnd, MONITOR_DEFAULTTOPRIMARY), &mon_info);
|
||||
work_rect = mon_info.rcWork;
|
||||
|
||||
ret = GetWindowPlacement(hwnd, &wp);
|
||||
ok(ret, "failed to get window placement, error %u\n", GetLastError());
|
||||
ok(wp.showCmd == SW_SHOWNORMAL, "got show cmd %u\n", wp.showCmd);
|
||||
|
|
@ -11675,7 +11680,6 @@ static void test_window_placement(void)
|
|||
ok(wp.showCmd == SW_SHOWMAXIMIZED, "got show cmd %u\n", wp.showCmd);
|
||||
ok(wp.ptMinPosition.x == -32000 && wp.ptMinPosition.y == -32000,
|
||||
"got minimized pos (%d,%d)\n", wp.ptMinPosition.x, wp.ptMinPosition.y);
|
||||
todo_wine
|
||||
ok(wp.ptMaxPosition.x == -1 && wp.ptMaxPosition.y == -1,
|
||||
"got maximized pos (%d,%d)\n", wp.ptMaxPosition.x, wp.ptMaxPosition.y);
|
||||
ok(EqualRect(&wp.rcNormalPosition, &orig), "got normal pos %s\n",
|
||||
|
|
@ -11693,6 +11697,42 @@ todo_wine
|
|||
ok(EqualRect(&wp.rcNormalPosition, &orig), "got normal pos %s\n",
|
||||
wine_dbgstr_rect(&wp.rcNormalPosition));
|
||||
|
||||
SetWindowPos(hwnd, 0, work_rect.left, work_rect.top, work_rect.right - work_rect.left,
|
||||
work_rect.bottom - work_rect.top, SWP_NOZORDER | SWP_NOACTIVATE);
|
||||
ret = GetWindowPlacement(hwnd, &wp);
|
||||
ok(ret, "failed to get window placement, error %u\n", GetLastError());
|
||||
ok(wp.showCmd == SW_SHOWMAXIMIZED, "got show cmd %u\n", wp.showCmd);
|
||||
ok(wp.ptMinPosition.x == -32000 && wp.ptMinPosition.y == -32000,
|
||||
"got minimized pos (%d,%d)\n", wp.ptMinPosition.x, wp.ptMinPosition.y);
|
||||
ok(wp.ptMaxPosition.x == -1 && wp.ptMaxPosition.y == -1,
|
||||
"got maximized pos (%d,%d)\n", wp.ptMaxPosition.x, wp.ptMaxPosition.y);
|
||||
ok(EqualRect(&wp.rcNormalPosition, &orig), "got normal pos %s\n",
|
||||
wine_dbgstr_rect(&wp.rcNormalPosition));
|
||||
|
||||
SetWindowPos(hwnd, 0, work_rect.left, work_rect.top, work_rect.right - work_rect.left - 1,
|
||||
work_rect.bottom - work_rect.top, SWP_NOZORDER | SWP_NOACTIVATE);
|
||||
ret = GetWindowPlacement(hwnd, &wp);
|
||||
ok(ret, "failed to get window placement, error %u\n", GetLastError());
|
||||
ok(wp.showCmd == SW_SHOWMAXIMIZED, "got show cmd %u\n", wp.showCmd);
|
||||
ok(wp.ptMinPosition.x == -32000 && wp.ptMinPosition.y == -32000,
|
||||
"got minimized pos (%d,%d)\n", wp.ptMinPosition.x, wp.ptMinPosition.y);
|
||||
ok(wp.ptMaxPosition.x == work_rect.left && wp.ptMaxPosition.y == work_rect.top,
|
||||
"got maximized pos (%d,%d)\n", wp.ptMaxPosition.x, wp.ptMaxPosition.y);
|
||||
ok(EqualRect(&wp.rcNormalPosition, &orig), "got normal pos %s\n",
|
||||
wine_dbgstr_rect(&wp.rcNormalPosition));
|
||||
|
||||
SetWindowPos(hwnd, 0, work_rect.left, work_rect.top, work_rect.right - work_rect.left,
|
||||
work_rect.bottom - work_rect.top - 1, SWP_NOZORDER | SWP_NOACTIVATE);
|
||||
ret = GetWindowPlacement(hwnd, &wp);
|
||||
ok(ret, "failed to get window placement, error %u\n", GetLastError());
|
||||
ok(wp.showCmd == SW_SHOWMAXIMIZED, "got show cmd %u\n", wp.showCmd);
|
||||
ok(wp.ptMinPosition.x == -32000 && wp.ptMinPosition.y == -32000,
|
||||
"got minimized pos (%d,%d)\n", wp.ptMinPosition.x, wp.ptMinPosition.y);
|
||||
ok(wp.ptMaxPosition.x == work_rect.left && wp.ptMaxPosition.y == work_rect.top,
|
||||
"got maximized pos (%d,%d)\n", wp.ptMaxPosition.x, wp.ptMaxPosition.y);
|
||||
ok(EqualRect(&wp.rcNormalPosition, &orig), "got normal pos %s\n",
|
||||
wine_dbgstr_rect(&wp.rcNormalPosition));
|
||||
|
||||
ShowWindow(hwnd, SW_MINIMIZE);
|
||||
|
||||
ret = GetWindowPlacement(hwnd, &wp);
|
||||
|
|
@ -11701,7 +11741,6 @@ todo_wine
|
|||
ok(wp.showCmd == SW_SHOWMINIMIZED, "got show cmd %u\n", wp.showCmd);
|
||||
ok(wp.ptMinPosition.x == -32000 && wp.ptMinPosition.y == -32000,
|
||||
"got minimized pos (%d,%d)\n", wp.ptMinPosition.x, wp.ptMinPosition.y);
|
||||
todo_wine
|
||||
ok(wp.ptMaxPosition.x == -1 && wp.ptMaxPosition.y == -1,
|
||||
"got maximized pos (%d,%d)\n", wp.ptMaxPosition.x, wp.ptMaxPosition.y);
|
||||
ok(EqualRect(&wp.rcNormalPosition, &orig), "got normal pos %s\n",
|
||||
|
|
@ -11714,7 +11753,6 @@ todo_wine
|
|||
ok(wp.showCmd == SW_SHOWMAXIMIZED, "got show cmd %u\n", wp.showCmd);
|
||||
ok(wp.ptMinPosition.x == -32000 && wp.ptMinPosition.y == -32000,
|
||||
"got minimized pos (%d,%d)\n", wp.ptMinPosition.x, wp.ptMinPosition.y);
|
||||
todo_wine
|
||||
ok(wp.ptMaxPosition.x == -1 && wp.ptMaxPosition.y == -1,
|
||||
"got maximized pos (%d,%d)\n", wp.ptMaxPosition.x, wp.ptMaxPosition.y);
|
||||
ok(EqualRect(&wp.rcNormalPosition, &orig), "got normal pos %s\n",
|
||||
|
|
@ -11727,7 +11765,6 @@ todo_wine
|
|||
ok(wp.showCmd == SW_SHOWNORMAL, "got show cmd %u\n", wp.showCmd);
|
||||
ok(wp.ptMinPosition.x == -32000 && wp.ptMinPosition.y == -32000,
|
||||
"got minimized pos (%d,%d)\n", wp.ptMinPosition.x, wp.ptMinPosition.y);
|
||||
todo_wine
|
||||
ok(wp.ptMaxPosition.x == -1 && wp.ptMaxPosition.y == -1,
|
||||
"got maximized pos (%d,%d)\n", wp.ptMaxPosition.x, wp.ptMaxPosition.y);
|
||||
ok(EqualRect(&wp.rcNormalPosition, &orig), "got normal pos %s\n",
|
||||
|
|
@ -11745,7 +11782,6 @@ todo_wine
|
|||
ok(wp.showCmd == SW_SHOWNORMAL, "got show cmd %u\n", wp.showCmd);
|
||||
ok(wp.ptMinPosition.x == 100 && wp.ptMinPosition.y == 100,
|
||||
"got minimized pos (%d,%d)\n", wp.ptMinPosition.x, wp.ptMinPosition.y);
|
||||
todo_wine
|
||||
ok(wp.ptMaxPosition.x == -1 && wp.ptMaxPosition.y == -1,
|
||||
"got maximized pos (%d,%d)\n", wp.ptMaxPosition.x, wp.ptMaxPosition.y);
|
||||
ok(EqualRect(&wp.rcNormalPosition, &orig2), "got normal pos %s\n",
|
||||
|
|
@ -11761,7 +11797,6 @@ todo_wine
|
|||
ok(wp.showCmd == SW_SHOWMINIMIZED, "got show cmd %u\n", wp.showCmd);
|
||||
ok(wp.ptMinPosition.x == -32000 && wp.ptMinPosition.y == -32000,
|
||||
"got minimized pos (%d,%d)\n", wp.ptMinPosition.x, wp.ptMinPosition.y);
|
||||
todo_wine
|
||||
ok(wp.ptMaxPosition.x == -1 && wp.ptMaxPosition.y == -1,
|
||||
"got maximized pos (%d,%d)\n", wp.ptMaxPosition.x, wp.ptMaxPosition.y);
|
||||
ok(EqualRect(&wp.rcNormalPosition, &orig2), "got normal pos %s\n",
|
||||
|
|
@ -11783,7 +11818,6 @@ todo_wine
|
|||
ok(wp.showCmd == SW_SHOWMINIMIZED, "got show cmd %u\n", wp.showCmd);
|
||||
ok(wp.ptMinPosition.x == -32000 && wp.ptMinPosition.y == -32000,
|
||||
"got minimized pos (%d,%d)\n", wp.ptMinPosition.x, wp.ptMinPosition.y);
|
||||
todo_wine
|
||||
ok(wp.ptMaxPosition.x == -1 && wp.ptMaxPosition.y == -1,
|
||||
"got maximized pos (%d,%d)\n", wp.ptMaxPosition.x, wp.ptMaxPosition.y);
|
||||
ok(EqualRect(&wp.rcNormalPosition, &orig), "got normal pos %s\n",
|
||||
|
|
@ -11804,7 +11838,6 @@ todo_wine
|
|||
ok(wp.showCmd == SW_SHOWMAXIMIZED, "got show cmd %u\n", wp.showCmd);
|
||||
ok(wp.ptMinPosition.x == 100 && wp.ptMinPosition.y == 100,
|
||||
"got minimized pos (%d,%d)\n", wp.ptMinPosition.x, wp.ptMinPosition.y);
|
||||
todo_wine
|
||||
ok(wp.ptMaxPosition.x == -1 && wp.ptMaxPosition.y == -1,
|
||||
"got maximized pos (%d,%d)\n", wp.ptMaxPosition.x, wp.ptMaxPosition.y);
|
||||
ok(EqualRect(&wp.rcNormalPosition, &orig), "got normal pos %s\n",
|
||||
|
|
@ -11825,7 +11858,6 @@ todo_wine
|
|||
ok(wp.showCmd == SW_SHOWMINIMIZED, "got show cmd %u\n", wp.showCmd);
|
||||
ok(wp.ptMinPosition.x == -32000 && wp.ptMinPosition.y == -32000,
|
||||
"got minimized pos (%d,%d)\n", wp.ptMinPosition.x, wp.ptMinPosition.y);
|
||||
todo_wine
|
||||
ok(wp.ptMaxPosition.x == -1 && wp.ptMaxPosition.y == -1,
|
||||
"got maximized pos (%d,%d)\n", wp.ptMaxPosition.x, wp.ptMaxPosition.y);
|
||||
ok(EqualRect(&wp.rcNormalPosition, &orig), "got normal pos %s\n",
|
||||
|
|
@ -11839,7 +11871,6 @@ todo_wine
|
|||
ok(wp.showCmd == SW_SHOWMINIMIZED, "got show cmd %u\n", wp.showCmd);
|
||||
ok(wp.ptMinPosition.x == -32000 && wp.ptMinPosition.y == -32000,
|
||||
"got minimized pos (%d,%d)\n", wp.ptMinPosition.x, wp.ptMinPosition.y);
|
||||
todo_wine
|
||||
ok(wp.ptMaxPosition.x == -1 && wp.ptMaxPosition.y == -1,
|
||||
"got maximized pos (%d,%d)\n", wp.ptMaxPosition.x, wp.ptMaxPosition.y);
|
||||
ok(EqualRect(&wp.rcNormalPosition, &orig), "got normal pos %s\n",
|
||||
|
|
@ -11854,7 +11885,6 @@ todo_wine
|
|||
ok(wp.showCmd == SW_NORMAL, "got show cmd %u\n", wp.showCmd);
|
||||
ok(wp.ptMinPosition.x == -32000 && wp.ptMinPosition.y == -32000,
|
||||
"got minimized pos (%d,%d)\n", wp.ptMinPosition.x, wp.ptMinPosition.y);
|
||||
todo_wine
|
||||
ok(wp.ptMaxPosition.x == -1 && wp.ptMaxPosition.y == -1,
|
||||
"got maximized pos (%d,%d)\n", wp.ptMaxPosition.x, wp.ptMaxPosition.y);
|
||||
ok(EqualRect(&wp.rcNormalPosition, &orig), "got normal pos %s\n",
|
||||
|
|
|
|||
|
|
@ -1285,6 +1285,47 @@ UINT WINAPI GetInternalWindowPos( HWND hwnd, LPRECT rectWnd,
|
|||
}
|
||||
|
||||
|
||||
static RECT get_maximized_work_rect( HWND hwnd )
|
||||
{
|
||||
RECT work_rect = { 0 };
|
||||
|
||||
if ((GetWindowLongW( hwnd, GWL_STYLE ) & (WS_MINIMIZE | WS_MAXIMIZE)) == WS_MAXIMIZE)
|
||||
{
|
||||
if (!get_work_rect( hwnd, &work_rect ))
|
||||
work_rect = get_primary_monitor_rect();
|
||||
}
|
||||
return work_rect;
|
||||
}
|
||||
|
||||
|
||||
/*******************************************************************
|
||||
* update_maximized_pos
|
||||
*
|
||||
* For top level windows covering the work area, we might have to
|
||||
* "forget" the maximized position. Windows presumably does this
|
||||
* to avoid situations where the border style changes, which would
|
||||
* lead the window to be outside the screen, or the window gets
|
||||
* reloaded on a different screen, and the "saved" position no
|
||||
* longer applies to it (despite being maximized).
|
||||
*
|
||||
* Some applications (e.g. Imperiums: Greek Wars) depend on this.
|
||||
*/
|
||||
static void update_maximized_pos( WND *wnd, RECT *work_rect )
|
||||
{
|
||||
if (wnd->parent && wnd->parent != GetDesktopWindow())
|
||||
return;
|
||||
|
||||
if (wnd->dwStyle & WS_MAXIMIZE)
|
||||
{
|
||||
if (wnd->window_rect.left <= work_rect->left && wnd->window_rect.top <= work_rect->top &&
|
||||
wnd->window_rect.right >= work_rect->right && wnd->window_rect.bottom >= work_rect->bottom)
|
||||
wnd->max_pos.x = wnd->max_pos.y = -1;
|
||||
}
|
||||
else
|
||||
wnd->max_pos.x = wnd->max_pos.y = -1;
|
||||
}
|
||||
|
||||
|
||||
/***********************************************************************
|
||||
* GetWindowPlacement (USER32.@)
|
||||
*
|
||||
|
|
@ -1293,6 +1334,7 @@ UINT WINAPI GetInternalWindowPos( HWND hwnd, LPRECT rectWnd,
|
|||
*/
|
||||
BOOL WINAPI GetWindowPlacement( HWND hwnd, WINDOWPLACEMENT *wndpl )
|
||||
{
|
||||
RECT work_rect = get_maximized_work_rect( hwnd );
|
||||
WND *pWnd = WIN_GetPtr( hwnd );
|
||||
|
||||
if (!pWnd) return FALSE;
|
||||
|
|
@ -1350,6 +1392,7 @@ BOOL WINAPI GetWindowPlacement( HWND hwnd, WINDOWPLACEMENT *wndpl )
|
|||
{
|
||||
pWnd->normal_rect = pWnd->window_rect;
|
||||
}
|
||||
update_maximized_pos( pWnd, &work_rect );
|
||||
|
||||
wndpl->length = sizeof(*wndpl);
|
||||
if( pWnd->dwStyle & WS_MINIMIZE )
|
||||
|
|
@ -1421,6 +1464,7 @@ static void make_point_onscreen( POINT *pt )
|
|||
static BOOL WINPOS_SetPlacement( HWND hwnd, const WINDOWPLACEMENT *wndpl, UINT flags )
|
||||
{
|
||||
DWORD style;
|
||||
RECT work_rect = get_maximized_work_rect( hwnd );
|
||||
WND *pWnd = WIN_GetPtr( hwnd );
|
||||
WINDOWPLACEMENT wp = *wndpl;
|
||||
|
||||
|
|
@ -1438,7 +1482,11 @@ static BOOL WINPOS_SetPlacement( HWND hwnd, const WINDOWPLACEMENT *wndpl, UINT f
|
|||
if (!pWnd || pWnd == WND_OTHER_PROCESS || pWnd == WND_DESKTOP) return FALSE;
|
||||
|
||||
if (flags & PLACE_MIN) pWnd->min_pos = point_thread_to_win_dpi( hwnd, wp.ptMinPosition );
|
||||
if (flags & PLACE_MAX) pWnd->max_pos = point_thread_to_win_dpi( hwnd, wp.ptMaxPosition );
|
||||
if (flags & PLACE_MAX)
|
||||
{
|
||||
pWnd->max_pos = point_thread_to_win_dpi( hwnd, wp.ptMaxPosition );
|
||||
update_maximized_pos( pWnd, &work_rect );
|
||||
}
|
||||
if (flags & PLACE_RECT) pWnd->normal_rect = rect_thread_to_win_dpi( hwnd, wp.rcNormalPosition );
|
||||
|
||||
style = pWnd->dwStyle;
|
||||
|
|
|
|||
Loading…
Reference in a new issue