gdk/x11: Invalidate whole surface after size change
authorCarlos Garnacho <carlosg@gnome.org>
Fri, 21 Apr 2023 08:58:19 +0000 (10:58 +0200)
committerCarlos Garnacho <carlosg@gnome.org>
Sat, 29 Apr 2023 09:51:32 +0000 (11:51 +0200)
The Expose events following a ConfigureNotify may arrive at
a time that we did not resize the surface yet, making these
expose events a no-op. Even though gsk/gtk take care of the
window content itself, this might lead to unrendered portions
of the window shadow.

This may be seen with GSK_RENDERER=cairo and GDK_BACKEND=x11,
attempting to tile a window (e.g. gtk4-demo) left or right.
The window will show black rectangles or other artifacts in
the window shadow areas that correspond to the newly painted
portions (as the window needs to expand vertically).

In order to fix this with a similar behavior to Wayland,
consider ourselves the whole surface invalidated after resize,
in order to ensure everything is painted from scratch.

gdk/x11/gdksurface-x11.c

index 284895ce0e654ba1cd3072004fb0e68169bdf606..7a1bb4e8b9145cc89db5a6967a0f61205a4ef34b 100644 (file)
@@ -209,6 +209,8 @@ gdk_x11_surface_update_size (GdkX11Surface *self,
       cairo_surface_set_device_scale (self->cairo_surface, scale, scale);
     }
 
+  gdk_surface_invalidate_rect (surface, NULL);
+
   return TRUE;
 }