WindowSurfaceGLX: swap after resizing the child window.

This reverts ac2582d3 which was a speculative improvement. It turns out that resizing before the swap is important as it lets the driver know that it should resize the default framebuffer, making the following frame have a default framebuffer of the right size. BUG=angleproject:1050 BUG=angleproject:892 Change-Id: I84ec3728226fde9fb208f91c02a3d5a1ab71aef8 Reviewed-on: https://chromium-review.googlesource.com/286252Reviewed-by: Kenneth Russell <kbr@chromium.org> Reviewed-by: Jamie Madill <jmadill@chromium.org> Tested-by: Corentin Wallez <cwallez@chromium.org>

WindowSurfaceGLX: swap after resizing the child window.
bb6a5f93 · Corentin Wallez · e7d8c80a · bb6a5f93
Commit bb6a5f93 authored Jul 16, 2015 by Corentin Wallez
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

WindowSurfaceGLX.cpp src/libANGLE/renderer/gl/glx/WindowSurfaceGLX.cpp +2 -2

No files found.
--- a/src/libANGLE/renderer/gl/glx/WindowSurfaceGLX.cpp
+++ b/src/libANGLE/renderer/gl/glx/WindowSurfaceGLX.cpp
@@ -117,8 +117,6 @@ egl::Error WindowSurfaceGLX::makeCurrent()

 egl::Error WindowSurfaceGLX::swap()
 {
-    mGLX.swapBuffers(mGLXWindow);
-
    //TODO(cwallez) set up our own error handler to see if the call failed
    unsigned int newParentWidth, newParentHeight;
    if (!getWindowDimensions(mParent, &newParentWidth, &newParentHeight))
@@ -137,6 +135,8 @@ egl::Error WindowSurfaceGLX::swap()
        mGLX.waitX();
    }

+    mGLX.swapBuffers(mGLXWindow);
+
    return egl::Error(EGL_SUCCESS);
 }