[gtk+] Revert "gtkbin: replace internal api"

[Matthias Clasen <matthiasc src gnome org>]

commit f0f9c2aa37893cf5d3f09ce9c8ac36a88d661e37
Author: Matthias Clasen <mclasen redhat com>
Date:   Mon Jun 16 17:31:56 2014 -0400

    Revert "gtkbin: replace internal api"
    
    This reverts commit 0f4751c08e22a4f45d58e2368e4ae82df407d330.
    
    This causes warnings from the notebook and combobox code.

 gtk/gtkassistant.c      |    3 ++-
 gtk/gtkbin.c            |    6 ++++++
 gtk/gtkbin.h            |    3 +++
 gtk/gtkcombobox.c       |   18 +++++++++++++-----
 gtk/gtknotebook.c       |    7 +++----
 gtk/gtkscrolledwindow.c |   12 +++++++++---
 6 files changed, 36 insertions(+), 13 deletions(-)
---
diff --git a/gtk/gtkassistant.c b/gtk/gtkassistant.c
index 6112d30..215a9c8 100644
--- a/gtk/gtkassistant.c
+++ b/gtk/gtkassistant.c
@@ -1434,7 +1434,8 @@ gtk_assistant_add (GtkContainer *container,
    */
   if (!gtk_bin_get_child (GTK_BIN (container)))
     {
-      GTK_CONTAINER_CLASS (gtk_assistant_parent_class)->add (container, page);
+      gtk_widget_set_parent (page, GTK_WIDGET (container));
+      _gtk_bin_set_child (GTK_BIN (container), page);
       return;
     }
 
diff --git a/gtk/gtkbin.c b/gtk/gtkbin.c
index 70de242..9d7a050 100644
--- a/gtk/gtkbin.c
+++ b/gtk/gtkbin.c
@@ -331,3 +331,9 @@ gtk_bin_get_child (GtkBin *bin)
   return bin->priv->child;
 }
 
+void
+_gtk_bin_set_child (GtkBin    *bin,
+                    GtkWidget *widget)
+{
+  bin->priv->child = widget;
+}
diff --git a/gtk/gtkbin.h b/gtk/gtkbin.h
index 464bff7..bca7885 100644
--- a/gtk/gtkbin.h
+++ b/gtk/gtkbin.h
@@ -79,6 +79,9 @@ GType      gtk_bin_get_type  (void) G_GNUC_CONST;
 GDK_AVAILABLE_IN_ALL
 GtkWidget *gtk_bin_get_child (GtkBin *bin);
 
+void       _gtk_bin_set_child (GtkBin    *bin,
+                               GtkWidget *widget);
+
 G_END_DECLS
 
 #endif /* __GTK_BIN_H__ */
diff --git a/gtk/gtkcombobox.c b/gtk/gtkcombobox.c
index 7ab2708..baeb30d 100644
--- a/gtk/gtkcombobox.c
+++ b/gtk/gtkcombobox.c
@@ -1587,10 +1587,13 @@ gtk_combo_box_add (GtkContainer *container,
   if (priv->cell_view &&
       gtk_widget_get_parent (priv->cell_view))
     {
-      GTK_CONTAINER_CLASS (gtk_combo_box_parent_class)->remove (container, priv->cell_view);
+      gtk_widget_unparent (priv->cell_view);
+      _gtk_bin_set_child (GTK_BIN (container), NULL);
+      gtk_widget_queue_resize (GTK_WIDGET (container));
     }
   
-  GTK_CONTAINER_CLASS (gtk_combo_box_parent_class)->add (container, widget);
+  gtk_widget_set_parent (widget, GTK_WIDGET (container));
+  _gtk_bin_set_child (GTK_BIN (container), widget);
 
   if (priv->cell_view &&
       widget != priv->cell_view)
@@ -1654,11 +1657,14 @@ gtk_combo_box_remove (GtkContainer *container,
   if (widget == priv->cell_view)
     priv->cell_view = NULL;
 
-  GTK_CONTAINER_CLASS (gtk_combo_box_parent_class)->remove (container, widget);
+  gtk_widget_unparent (widget);
+  _gtk_bin_set_child (GTK_BIN (container), NULL);
 
   if (gtk_widget_in_destruction (GTK_WIDGET (combo_box)))
     return;
 
+  gtk_widget_queue_resize (GTK_WIDGET (container));
+
   if (!priv->tree_view)
     appears_as_list = FALSE;
   else
@@ -1676,7 +1682,8 @@ gtk_combo_box_remove (GtkContainer *container,
   if (!priv->cell_view)
     {
       priv->cell_view = gtk_cell_view_new ();
-      GTK_CONTAINER_CLASS (gtk_combo_box_parent_class)->add (container, priv->cell_view);
+      gtk_widget_set_parent (priv->cell_view, GTK_WIDGET (container));
+      _gtk_bin_set_child (GTK_BIN (container), priv->cell_view);
 
       gtk_widget_show (priv->cell_view);
       gtk_cell_view_set_model (GTK_CELL_VIEW (priv->cell_view),
@@ -4810,7 +4817,8 @@ gtk_combo_box_constructor (GType                  type,
   priv->cell_view = gtk_cell_view_new_with_context (priv->area, NULL);
   gtk_cell_view_set_fit_model (GTK_CELL_VIEW (priv->cell_view), TRUE);
   gtk_cell_view_set_model (GTK_CELL_VIEW (priv->cell_view), priv->model);
-  GTK_CONTAINER_CLASS (gtk_combo_box_parent_class)->add (GTK_CONTAINER (combo_box),priv->cell_view);
+  gtk_widget_set_parent (priv->cell_view, GTK_WIDGET (combo_box));
+  _gtk_bin_set_child (GTK_BIN (combo_box), priv->cell_view);
   gtk_widget_show (priv->cell_view);
 
   gtk_combo_box_check_appearance (combo_box);
diff --git a/gtk/gtknotebook.c b/gtk/gtknotebook.c
index 1e48671..4cc472c 100644
--- a/gtk/gtknotebook.c
+++ b/gtk/gtknotebook.c
@@ -3723,8 +3723,7 @@ gtk_notebook_drag_end (GtkWidget      *widget,
   if (priv->detached_tab)
     gtk_notebook_switch_page (notebook, priv->detached_tab);
 
-  GTK_CONTAINER_CLASS (gtk_notebook_parent_class)->remove (GTK_CONTAINER (priv->dnd_window),
-    gtk_bin_get_child (GTK_BIN (priv->dnd_window)));
+  _gtk_bin_set_child (GTK_BIN (priv->dnd_window), NULL);
   gtk_widget_destroy (priv->dnd_window);
   priv->dnd_window = NULL;
 
@@ -6911,8 +6910,8 @@ static void
 gtk_notebook_menu_label_unparent (GtkWidget *widget,
                                   gpointer  data)
 {
-  GTK_CONTAINER_CLASS (gtk_notebook_parent_class)->remove (GTK_CONTAINER (widget),
-    gtk_bin_get_child (GTK_BIN (widget)));
+  gtk_widget_unparent (gtk_bin_get_child (GTK_BIN (widget)));
+  _gtk_bin_set_child (GTK_BIN (widget), NULL);
 }
 
 static void
diff --git a/gtk/gtkscrolledwindow.c b/gtk/gtkscrolledwindow.c
index 6a1edcb..87d3008 100644
--- a/gtk/gtkscrolledwindow.c
+++ b/gtk/gtkscrolledwindow.c
@@ -2727,9 +2727,14 @@ gtk_scrolled_window_add (GtkContainer *container,
 {
   GtkScrolledWindowPrivate *priv;
   GtkScrolledWindow *scrolled_window;
-  GtkWidget *scrollable_child;
+  GtkBin *bin;
+  GtkWidget *child_widget, *scrollable_child;
   GtkAdjustment *hadj, *vadj;
 
+  bin = GTK_BIN (container);
+  child_widget = gtk_bin_get_child (bin);
+  g_return_if_fail (child_widget == NULL);
+
   scrolled_window = GTK_SCROLLED_WINDOW (container);
   priv = scrolled_window->priv;
 
@@ -2760,10 +2765,11 @@ gtk_scrolled_window_add (GtkContainer *container,
       gtk_container_add (GTK_CONTAINER (scrollable_child), child);
     }
 
-  if (gtk_widget_get_realized (GTK_WIDGET (container)))
+  if (gtk_widget_get_realized (GTK_WIDGET (bin)))
     gtk_widget_set_parent_window (scrollable_child, priv->overshoot_window);
 
-  GTK_CONTAINER_CLASS (gtk_scrolled_window_parent_class)->add (container, scrollable_child);
+  _gtk_bin_set_child (bin, scrollable_child);
+  gtk_widget_set_parent (scrollable_child, GTK_WIDGET (bin));
 
   g_object_set (scrollable_child, "hadjustment", hadj, "vadjustment", vadj, NULL);
 }