[PATCH v3 5/5] winex11: Lock and flush display in X11DRV_ChangeDisplaySettings.

[Rémi Bernon]

From: Rémi Bernon

---
 dlls/winex11.drv/display.c  | 5 +++++
 dlls/winex11.drv/xrandr.c   | 5 -----
 dlls/winex11.drv/xvidmode.c | 2 +-
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/dlls/winex11.drv/display.c b/dlls/winex11.drv/display.c
index 68073f88955..af984efe7f0 100644
--- a/dlls/winex11.drv/display.c
+++ b/dlls/winex11.drv/display.c
@@ -238,6 +238,8 @@ LONG X11DRV_ChangeDisplaySettings( LPDEVMODEW displays, LPCWSTR primary_name, HW
         if ((ret = x11drv_mode_from_devmode( ids[count], mode, modes + count ))) goto done;
     }

+    XGrabServer( gdi_display );
+
     /* Detach displays first to free up CRTCs */
     TRACE( "Using %s\n", settings_handler.name );
     for (UINT i = 0; !ret && i < count; i++)
@@ -253,6 +255,9 @@ LONG X11DRV_ChangeDisplaySettings( LPDEVMODEW displays, LPCWSTR primary_name, HW
         ret = settings_handler.set_current_mode( ids[i], modes + i );
     }

+    XUngrabServer( gdi_display );
+    XFlush( gdi_display );
+
 done:
     free( modes );
     free( ids );
diff --git a/dlls/winex11.drv/xrandr.c b/dlls/winex11.drv/xrandr.c
index 058bde94701..1bc2c722416 100644
--- a/dlls/winex11.drv/xrandr.c
+++ b/dlls/winex11.drv/xrandr.c
@@ -391,7 +391,6 @@ static LONG xrandr10_set_current_mode( x11drv_settings_id id, const struct x11dr
     if (stat != RRSetConfigSuccess)
         return DISP_CHANGE_FAILED;

-    XFlush( gdi_display );
     return DISP_CHANGE_SUCCESSFUL;
 }

@@ -1582,8 +1581,6 @@ static LONG xrandr14_set_current_mode( x11drv_settings_id id, const struct x11dr
     if (!screen_resources)
         return ret;

-    XGrabServer( gdi_display );
-
     output_info = pXRRGetOutputInfo( gdi_display, screen_resources, output );
     if (!output_info || output_info->connection != RR_Connected)
         goto done;
@@ -1658,8 +1655,6 @@ static LONG xrandr14_set_current_mode( x11drv_settings_id id, const struct x11dr
         ret = DISP_CHANGE_SUCCESSFUL;

done:
-    XUngrabServer( gdi_display );
-    XFlush( gdi_display );
     if (crtc_info)
         pXRRFreeCrtcInfo( crtc_info );
     if (output_info)
diff --git a/dlls/winex11.drv/xvidmode.c b/dlls/winex11.drv/xvidmode.c
index d39116c48ff..23fd09a08f0 100644
--- a/dlls/winex11.drv/xvidmode.c
+++ b/dlls/winex11.drv/xvidmode.c
@@ -223,7 +223,7 @@ static LONG xf86vm_set_current_mode( x11drv_settings_id id, const struct x11drv_
 #else
     XWarpPointer(gdi_display, None, DefaultRootWindow(gdi_display), 0, 0, 0, 0, 0, 0);
 #endif
-    XFlush(gdi_display);
+
     return DISP_CHANGE_SUCCESSFUL;
 }