From: Jonas Ådahl Date: Wed, 21 Mar 2018 06:01:58 +0000 (+0800) Subject: wayland/glcontext: Sync surface state before swapping buffers X-Git-Tag: archive/raspbian/4.4.1+ds1-2+rpi1^2~18^2~22^2~826^2~1 X-Git-Url: https://dgit.raspbian.org/?a=commitdiff_plain;h=1d5d46ff29c7e5c012bf7dd571304cc54fb95bed;p=gtk4.git wayland/glcontext: Sync surface state before swapping buffers A side effect of eglSwapBuffers* is the EGL implementation calling wl_surface_commit() on the corresponding Wayland surface. Thus, before swapping buffers, we must synchronize the surface state (e.g. opaque region, window geometry, etc) that changed since the last buffer swap. Prior to this commit, this was done after eglSwapBuffers*, causing the surface state to always correspond to the previous buffer state. As of this commit this is now done before swapping the buffers, thus before wl_surface_commit(). --- diff --git a/gdk/wayland/gdkglcontext-wayland.c b/gdk/wayland/gdkglcontext-wayland.c index 1e6f2ca28a..700baa02d6 100644 --- a/gdk/wayland/gdkglcontext-wayland.c +++ b/gdk/wayland/gdkglcontext-wayland.c @@ -223,6 +223,8 @@ gdk_wayland_gl_context_end_frame (GdkDrawContext *draw_context, egl_surface = gdk_wayland_surface_get_egl_surface (surface->impl_surface, context_wayland->egl_config); + gdk_wayland_surface_sync (surface); + if (display_wayland->have_egl_swap_buffers_with_damage && damage != NULL) { int i, j, n_rects = cairo_region_num_rectangles (damage); @@ -243,8 +245,6 @@ gdk_wayland_gl_context_end_frame (GdkDrawContext *draw_context, } else eglSwapBuffers (display_wayland->egl_display, egl_surface); - - gdk_wayland_surface_sync (surface); } static void