[clutter] actor: Use internal add/remove child in reparent()

[Emmanuele Bassi <ebassi src gnome org>]

commit c760657903b8d1599e25e54fcca01a5f546eb9aa
Author: Emmanuele Bassi <ebassi linux intel com>
Date:   Fri Jan 20 12:02:49 2012 +0000

    actor: Use internal add/remove child in reparent()
    
    Do not use the public API; reparent() used to use unparent() and
    set_parent(), so we need to maintain the old behaviour.

 clutter/clutter-actor.c |   29 ++++++++++++++++++-----------
 1 files changed, 18 insertions(+), 11 deletions(-)
---
diff --git a/clutter/clutter-actor.c b/clutter/clutter-actor.c
index ab9edfb..7fa24c5 100644
--- a/clutter/clutter-actor.c
+++ b/clutter/clutter-actor.c
@@ -10448,24 +10448,31 @@ clutter_actor_reparent (ClutterActor *self,
 
       g_object_ref (self);
 
-      /* go through the Container implementation if this is a regular
-       * child and not an internal one
-       */
-      if (!CLUTTER_ACTOR_IS_INTERNAL_CHILD (self))
+      if (old_parent != NULL)
         {
-          ClutterContainer *parent = CLUTTER_CONTAINER (old_parent);
-
-          /* this will have to call unparent() */
-          clutter_container_remove_actor (parent, self);
+         /* go through the Container implementation if this is a regular
+          * child and not an internal one
+          */
+         if (!CLUTTER_ACTOR_IS_INTERNAL_CHILD (self))
+           {
+             ClutterContainer *parent = CLUTTER_CONTAINER (old_parent);
+
+             /* this will have to call unparent() */
+             clutter_container_remove_actor (parent, self);
+           }
+         else
+           clutter_actor_remove_child_internal (old_parent, self,
+                                                REMOVE_CHILD_LEGACY_FLAGS);
         }
-      else
-        clutter_actor_remove_child (old_parent, self);
 
       /* Note, will call set_parent() */
       if (!CLUTTER_ACTOR_IS_INTERNAL_CHILD (self))
         clutter_container_add_actor (CLUTTER_CONTAINER (new_parent), self);
       else
-        clutter_actor_add_child (new_parent, self);
+        clutter_actor_add_child_internal (new_parent, self,
+                                          ADD_CHILD_LEGACY_FLAGS,
+                                          insert_child_at_depth,
+                                          NULL);
 
       /* we emit the ::parent-set signal once */
       g_signal_emit (self, actor_signals[PARENT_SET], 0, old_parent);