brl/mutter
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

background: Don't bother attaching a paint node if the region is empty

Browse Source
This commit is contained in:
Jasper St. Pierre 2014-08-07 14:59:59 -04:00
parent 6858cb261f
commit b284126d3b
1 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/compositor/meta-background.c
View File

@ -410,8 +410,6 @@ meta_background_paint_content (ClutterContent *content,
if (priv->texture == NULL) if (priv->texture == NULL)
return; return;
node = meta_background_paint_node_new (self, actor);
clutter_actor_get_content_box (actor, &actor_box); clutter_actor_get_content_box (actor, &actor_box);
/* First figure out where on the monitor the texture is supposed to be painted. /* First figure out where on the monitor the texture is supposed to be painted.
@ -444,6 +442,11 @@ meta_background_paint_content (ClutterContent *content,
cairo_region_intersect (paintable_region, clip_region); cairo_region_intersect (paintable_region, clip_region);
} }
if (cairo_region_is_empty (paintable_region))
goto out;
node = meta_background_paint_node_new (self, actor);
/* Finally, split the paintable region up into distinct areas /* Finally, split the paintable region up into distinct areas
* and paint each area one by one * and paint each area one by one
*/ */
@ -466,10 +469,11 @@ meta_background_paint_content (ClutterContent *content,
clutter_paint_node_add_texture_rectangle (node, &texture_rectangle, tx1, ty1, tx2, ty2); clutter_paint_node_add_texture_rectangle (node, &texture_rectangle, tx1, ty1, tx2, ty2);
} }
cairo_region_destroy (paintable_region);
clutter_paint_node_add_child (root, node); clutter_paint_node_add_child (root, node);
clutter_paint_node_unref (node); clutter_paint_node_unref (node);
out:
cairo_region_destroy (paintable_region);
} }
static void static void