[mutter/wip/carlosg/avoid-parent-relayouts: 11/11] clutter: Only redraw affected portions on parent widget on show/hide

[Georges Basile Stavracas Neto <gbsneto src gnome org>]

commit ffaec917e507a8bb500d6fc4fd2e9d9e1856e6f1
Author: Carlos Garnacho <carlosg gnome org>
Date:   Fri Aug 10 21:33:50 2018 +0200

    clutter: Only redraw affected portions on parent widget on show/hide
    
    If we are lucky enough and the parent actor has the CLUTTER_ACTOR_NO_LAYOUT
    flag, we would skip the relayout, but still redraw the parent actor in its
    entirety.
    
    In these cases, we can at least just redraw the area affected by the actor
    being shown/hidden.

 clutter/clutter/clutter-actor.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)
---
diff --git a/clutter/clutter/clutter-actor.c b/clutter/clutter/clutter-actor.c
index 6913f315a..521d4264c 100644
--- a/clutter/clutter/clutter-actor.c
+++ b/clutter/clutter/clutter-actor.c
@@ -1793,7 +1793,7 @@ clutter_actor_show (ClutterActor *self)
   g_object_notify_by_pspec (G_OBJECT (self), obj_props[PROP_VISIBLE]);
 
   if (priv->parent != NULL)
-    clutter_actor_queue_redraw (priv->parent);
+    clutter_actor_queue_redraw (self);
 
   g_object_thaw_notify (G_OBJECT (self));
 }
@@ -1918,8 +1918,10 @@ clutter_actor_hide (ClutterActor *self)
   g_signal_emit (self, actor_signals[HIDE], 0);
   g_object_notify_by_pspec (G_OBJECT (self), obj_props[PROP_VISIBLE]);
 
-  if (priv->parent != NULL)
+  if (priv->parent != NULL && priv->needs_allocation)
     clutter_actor_queue_redraw (priv->parent);
+  else
+    clutter_actor_queue_redraw_on_parent (self);
 
   g_object_thaw_notify (G_OBJECT (self));
 }