isa/mutter
1
0
Fork 0
mirror of https://github.com/brl/mutter.git synced 2024-12-25 12:32:05 +00:00
Code Issues Packages Projects Releases Wiki Activity

window-actor: Don't pass around client_area

Browse Source 
To solve a performance regression, the three update_*_region
calls are going to be split up, so just call get_client_area_rect
in the methods itself.
This commit is contained in:
Jasper St. Pierre 2014-02-26 19:35:41 -05:00
parent ec2c3e1438
commit 2dc7371944
1 changed files with 19 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

36
src/compositor/meta-window-actor.c
View File

@ -1720,16 +1720,18 @@ build_and_scan_frame_mask (MetaWindowActor *self,
} }
static void static void
meta_window_actor_update_shape_region (MetaWindowActor *self, meta_window_actor_update_shape_region (MetaWindowActor *self)
cairo_rectangle_int_t *client_area)
{ {
MetaWindowActorPrivate *priv = self->priv; MetaWindowActorPrivate *priv = self->priv;
cairo_region_t *region = NULL; cairo_region_t *region = NULL;
cairo_rectangle_int_t client_area;
meta_window_get_client_area_rect (priv->window, &client_area);
if (priv->window->frame != NULL && priv->window->shape_region != NULL) if (priv->window->frame != NULL && priv->window->shape_region != NULL)
{ {
region = cairo_region_copy (priv->window->shape_region); region = cairo_region_copy (priv->window->shape_region);
cairo_region_translate (region, client_area->x, client_area->y); cairo_region_translate (region, client_area.x, client_area.y);
} }
else if (priv->window->shape_region != NULL) else if (priv->window->shape_region != NULL)
{ {
@ -1740,11 +1742,11 @@ meta_window_actor_update_shape_region (MetaWindowActor *self,
/* If we don't have a shape on the server, that means that /* If we don't have a shape on the server, that means that
* we have an implicit shape of one rectangle covering the * we have an implicit shape of one rectangle covering the
* entire window. */ * entire window. */
region = cairo_region_create_rectangle (client_area); region = cairo_region_create_rectangle (&client_area);
} }
if ((priv->window->shape_region != NULL) || (priv->window->frame != NULL)) if ((priv->window->shape_region != NULL) || (priv->window->frame != NULL))
build_and_scan_frame_mask (self, client_area, region); build_and_scan_frame_mask (self, &client_area, region);
g_clear_pointer (&priv->shape_region, cairo_region_destroy); g_clear_pointer (&priv->shape_region, cairo_region_destroy);
priv->shape_region = region; priv->shape_region = region;
@ -1755,20 +1757,23 @@ meta_window_actor_update_shape_region (MetaWindowActor *self,
} }
static void static void
meta_window_actor_update_input_region (MetaWindowActor *self, meta_window_actor_update_input_region (MetaWindowActor *self)
cairo_rectangle_int_t *client_area)
{ {
MetaWindowActorPrivate *priv = self->priv; MetaWindowActorPrivate *priv = self->priv;
cairo_region_t *region = NULL; cairo_region_t *region = NULL;
cairo_rectangle_int_t client_area;
meta_window_get_client_area_rect (priv->window, &client_area);
if (priv->window->frame != NULL) if (priv->window->frame != NULL)
{ {
region = meta_frame_get_frame_bounds (priv->window->frame); region = meta_frame_get_frame_bounds (priv->window->frame);
/* client area is in client window coordinates, so translate the
/* input_region is in client window coordinates, so translate the
* input region into that coordinate system and back */ * input region into that coordinate system and back */
cairo_region_translate (region, -client_area->x, -client_area->y); cairo_region_translate (region, -client_area.x, -client_area.y);
cairo_region_union_rectangle (region, client_area); cairo_region_union_rectangle (region, &client_area);
cairo_region_translate (region, client_area->x, client_area->y); cairo_region_translate (region, client_area.x, client_area.y);
} }
else if (priv->window->shape_region != NULL || else if (priv->window->shape_region != NULL ||
priv->window->input_region != NULL) priv->window->input_region != NULL)
@ -1788,7 +1793,7 @@ meta_window_actor_update_input_region (MetaWindowActor *self,
/* If we don't have a shape on the server, that means that /* If we don't have a shape on the server, that means that
* we have an implicit shape of one rectangle covering the * we have an implicit shape of one rectangle covering the
* entire window. */ * entire window. */
region = cairo_region_create_rectangle (client_area); region = cairo_region_create_rectangle (&client_area);
} }
meta_surface_actor_set_input_region (priv->surface, region); meta_surface_actor_set_input_region (priv->surface, region);
@ -1835,18 +1840,15 @@ static void
check_needs_reshape (MetaWindowActor *self) check_needs_reshape (MetaWindowActor *self)
{ {
MetaWindowActorPrivate *priv = self->priv; MetaWindowActorPrivate *priv = self->priv;
cairo_rectangle_int_t client_area;
if (!priv->needs_reshape) if (!priv->needs_reshape)
return; return;
meta_window_get_client_area_rect (window, &client_area); meta_window_actor_update_shape_region (self);
meta_window_actor_update_shape_region (self, &client_area);
if (priv->window->client_type == META_WINDOW_CLIENT_TYPE_X11) if (priv->window->client_type == META_WINDOW_CLIENT_TYPE_X11)
{ {
meta_window_actor_update_input_region (self, &client_area); meta_window_actor_update_input_region (self);
meta_window_actor_update_opaque_region (self); meta_window_actor_update_opaque_region (self);
} }