winex11.drv: Always update _NET_WM_STATE in update_net_wm_states().

KWin treats a window covering exactly the whole monitor as maximized when handling its first map request and expects applications to update maximized state later. Wine doesn't know about this added maximized state and expect it unchanged, making the window always maximized as far as KWin is concerned. So always send _NET_WM_STATE updates even if Wine doesn't expect changes to hint KWin that a window should not be maximized. Fix test failures when running the tests introduced by 36b72035 with KWin. Signed-off-by: Zhiyi Zhang <zzhang@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

winex11.drv: Always update _NET_WM_STATE in update_net_wm_states().
c5ec1585 · Zhiyi Zhang · Alexandre Julliard · f04360cf · c5ec1585
Commit c5ec1585 authored Aug 25, 2020 by Zhiyi Zhang Committed by Alexandre Julliard Aug 26, 2020
Hide whitespace changes
Inline Side-by-side

Showing with 0 additions and 2 deletions

window.c dlls/winex11.drv/window.c +0 -2

No files found.
--- a/dlls/winex11.drv/window.c
+++ b/dlls/winex11.drv/window.c
@@ -996,8 +996,6 @@ void update_net_wm_states( struct x11drv_win_data *data )

        for (i = 0; i < NB_NET_WM_STATES; i++)
        {
-            if (!((data->net_wm_state ^ new_state) & (1 << i))) continue;  /* unchanged */
-
            TRACE( "setting wm state %u for window %p/%lx to %u prev %u\n",
                   i, data->hwnd, data->whole_window,
                   (new_state & (1 << i)) != 0, (data->net_wm_state & (1 << i)) != 0 );