]> Pileus Git - ~andy/gtk/commitdiff
Don't unnecessarily clear background twice in no EXPOSE_MASK case
authorAlexander Larsson <alexl@redhat.com>
Sun, 19 Feb 2012 10:55:22 +0000 (11:55 +0100)
committerAlexander Larsson <alexl@redhat.com>
Sun, 19 Feb 2012 10:55:22 +0000 (11:55 +0100)
We already clear in begin_paint, no need to do it again. In fact, this
will get the wrong result if the background has alpha.

gdk/gdkwindow.c

index be7ac1a6ef6ee8639470740337bafe8d5f37ced6..3f541c63f1a5dd4ab402310c8336d3c38102190e 100644 (file)
@@ -3602,17 +3602,6 @@ gdk_window_clear_backing_region_direct (GdkWindow *window,
   cairo_surface_destroy (paint.surface);
 }
 
-
-static void
-gdk_window_clear_region_internal (GdkWindow *window,
-                                 cairo_region_t *region)
-{
-  if (window->paint_stack)
-    gdk_window_clear_backing_region (window, region);
-  else
-    gdk_window_clear_backing_region_direct (window, region);
-}
-
 static void
 gdk_window_drop_cairo_surface (GdkWindow *window)
 {
@@ -3933,7 +3922,7 @@ _gdk_window_process_updates_recurse (GdkWindow *window,
           * piggyback on the implicit paint */
 
          gdk_window_begin_paint_region (window, clipped_expose_region);
-         gdk_window_clear_region_internal (window, clipped_expose_region);
+         /* The actual clear happens in begin_paint_region */
          gdk_window_end_paint (window);
        }
     }