clutter: Avoid rounding compensation when invalidating 2D actors
authorCarlos Garnacho <carlosg@gnome.org>
Fri, 5 May 2017 12:15:30 +0000 (14:15 +0200)
committerSimon McVittie <smcv@debian.org>
Sat, 27 Apr 2019 23:13:35 +0000 (00:13 +0100)
This allows the redraw clip to be more constrained, so MetaCullable doesn't
end up rendering portions of window shadows, frame and background when a
window invalidates (part of) its contents.

(cherry picked from commit a7df459416dc46321204c98197e4fd84724976e0)

Bug-Ubuntu: https://launchpad.net/bugs/1767648
Bug: https://bugzilla.gnome.org/show_bug.cgi?id=782344
Applied-upstream: 3.30.3, commit:afc91f0804ad1350fa09aa141b93020f28143b48

Gbp-Pq: Name clutter-Avoid-rounding-compensation-when-invalidating-2D-.patch

clutter/clutter/clutter-paint-volume.c

index b48f7f9d6ab63395ef1ee8d01e01e425365ccc51..f3405138baa10ee7b3153104ba375acad96d35f3 100644 (file)
@@ -1166,6 +1166,21 @@ _clutter_paint_volume_get_stage_paint_box (ClutterPaintVolume *pv,
 
   _clutter_paint_volume_get_bounding_box (&projected_pv, box);
 
+  if (pv->is_2d && pv->actor &&
+      clutter_actor_get_z_position (pv->actor) == 0)
+    {
+      /* If the volume/actor are perfectly 2D, take the bounding box as
+       * good. We won't need to add any extra room for sub-pixel positioning
+       * in this case.
+       */
+      clutter_paint_volume_free (&projected_pv);
+      box->x1 = CLUTTER_NEARBYINT (box->x1);
+      box->y1 = CLUTTER_NEARBYINT (box->y1);
+      box->x2 = CLUTTER_NEARBYINT (box->x2);
+      box->y2 = CLUTTER_NEARBYINT (box->y2);
+      return;
+    }
+
   /* The aim here is that for a given rectangle defined with floating point
    * coordinates we want to determine a stable quantized size in pixels
    * that doesn't vary due to the original box's sub-pixel position.