[layout] Set a back pointer to Box inside the layout

Use the LayoutManager API to set a back pointer to the Box actor
inside the LayoutManager used by the box.

This also allows us to replace the LayoutManager on a Box, since
the LayoutManager will be able to replace all the metadata if
needed.

diff --git a/clutter/clutter-box.c b/clutter/clutter-box.c
index 3c4bac8..f51070b 100644 (file)
--- a/clutter/clutter-box.c
+++ b/clutter/clutter-box.c
@@ -289,17 +289,21 @@ on_layout_changed (ClutterLayoutManager *manager,
   clutter_actor_queue_relayout (self);
 }
 
-static void
+static inline void
 set_layout_manager (ClutterBox           *self,
                     ClutterLayoutManager *manager)
 {
   ClutterBoxPrivate *priv = self->priv;
 
+  if (priv->manager == manager)
+    return;
+
   if (priv->manager != NULL)
     {
       if (priv->changed_id != 0)
         g_signal_handler_disconnect (priv->manager, priv->changed_id);
 
+      clutter_layout_manager_set_container (priv->manager, NULL);
       g_object_unref (priv->manager);
 
       priv->manager = NULL;
@@ -309,11 +313,18 @@ set_layout_manager (ClutterBox           *self,
   if (manager != NULL)
     {
       priv->manager = g_object_ref_sink (manager);
+      clutter_layout_manager_set_container (manager,
+                                            CLUTTER_CONTAINER (self));
+
       priv->changed_id =
         g_signal_connect (priv->manager, "layout-changed",
                           G_CALLBACK (on_layout_changed),
                           self);
     }
+
+  clutter_actor_queue_relayout (CLUTTER_ACTOR (self));
+
+  g_object_notify (G_OBJECT (self), "layout-manager");
 }
 
 static void
@@ -391,7 +402,7 @@ clutter_box_class_init (ClutterBoxClass *klass)
                                "The layout manager used by the box",
                                CLUTTER_TYPE_LAYOUT_MANAGER,
                                CLUTTER_PARAM_READWRITE |
-                               G_PARAM_CONSTRUCT_ONLY);
+                               G_PARAM_CONSTRUCT);
   g_object_class_install_property (gobject_class,
                                    PROP_LAYOUT_MANAGER,
                                    pspec);
@@ -425,6 +436,28 @@ clutter_box_new (ClutterLayoutManager *manager)
 }
 
 /**
+ * clutter_box_set_layout_manager:
+ * @box: a #ClutterBox
+ * @manager: a #ClutterLayoutManager
+ *
+ * Sets the #ClutterLayoutManager for @box
+ *
+ * A #ClutterLayoutManager is a delegate object that controls the
+ * layout of the children of @box
+ *
+ * Since: 1.2
+ */
+void
+clutter_box_set_layout_manager (ClutterBox *box,
+                                ClutterLayoutManager *manager)
+{
+  g_return_if_fail (CLUTTER_IS_BOX (box));
+  g_return_if_fail (manager == NULL || CLUTTER_IS_LAYOUT_MANAGER (manager));
+
+  set_layout_manager (box, manager);
+}
+
+/**
  * clutter_box_get_layout_manager:
  * @box: a #ClutterBox
  *

diff --git a/clutter/clutter-box.h b/clutter/clutter-box.h
index 9b8ad19..4e9ea12 100644 (file)
--- a/clutter/clutter-box.h
+++ b/clutter/clutter-box.h
@@ -38,6 +38,8 @@ GType clutter_box_get_type (void) G_GNUC_CONST;
 
 ClutterActor *        clutter_box_new                (ClutterLayoutManager *manager);
 
+void                  clutter_box_set_layout_manager (ClutterBox           *box,
+                                                      ClutterLayoutManager *manger);
 ClutterLayoutManager *clutter_box_get_layout_manager (ClutterBox           *box);
 
 void                  clutter_box_pack               (ClutterBox           *box,

diff --git a/doc/reference/clutter/clutter-sections.txt b/doc/reference/clutter/clutter-sections.txt
index 21d9c51..0b08233 100644 (file)
--- a/doc/reference/clutter/clutter-sections.txt
+++ b/doc/reference/clutter/clutter-sections.txt
@@ -1806,6 +1806,7 @@ clutter_bin_layout_get_type
 ClutterBox
 ClutterBoxClass
 clutter_box_new
+clutter_box_set_layout_manager
 clutter_box_get_layout_manager
 clutter_box_pack
 clutter_box_packv