[PATCH v2 3/6] winex11: Avoid sending WM_WINDOWPOSCHANGING when applying window manager config.

[Rémi Bernon]

From: Rémi Bernon

It sometimes create a feedback loop, trying to override the window
manager config changes.
---
 dlls/winex11.drv/window.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/dlls/winex11.drv/window.c b/dlls/winex11.drv/window.c
index 370ef36ee23..44330b6213a 100644
--- a/dlls/winex11.drv/window.c
+++ b/dlls/winex11.drv/window.c
@@ -1546,6 +1546,7 @@ static UINT window_update_client_state( struct x11drv_win_data *data )

static UINT window_update_client_config( struct x11drv_win_data *data )
 {
+    static const UINT fullscreen_mask = (1 << NET_WM_STATE_MAXIMIZED) | (1 << NET_WM_STATE_FULLSCREEN);
     UINT old_style = NtUserGetWindowLongW( data->hwnd, GWL_STYLE ), flags;
     RECT rect, old_rect = data->rects.window, new_rect;

@@ -1580,6 +1581,9 @@ static UINT window_update_client_config( struct x11drv_win_data *data )

if ((flags & (SWP_NOSIZE | SWP_NOMOVE)) == (SWP_NOSIZE | SWP_NOMOVE)) return 0;

+    /* avoid event feedback loops from window rect adjustments of maximized / fullscreen windows */
+    if (data->current_state.net_wm_state & fullscreen_mask) flags |= SWP_NOSENDCHANGING;
+
     TRACE( "window %p/%lx config changed %s -> %s, flags %#x\n", data->hwnd, data->whole_window,
            wine_dbgstr_rect(&old_rect), wine_dbgstr_rect(&new_rect), flags );
     return MAKELONG(SC_MOVE, flags);