[gtk+] Remove gtk_widget_set_mapped



commit 846cf681e8602399e172c256a93cd7484708219a
Author: Timm Bäder <mail baedert org>
Date:   Fri Nov 4 10:04:44 2016 +0100

    Remove gtk_widget_set_mapped
    
    Let all the remaining callers chain up.

 docs/reference/gtk/gtk4-sections.txt |    1 -
 gtk/gtkcontainer.c                   |   15 ++-------------
 gtk/gtklayout.c                      |    2 +-
 gtk/gtkshortcutssection.c            |    4 ++--
 gtk/gtktreeview.c                    |    4 +---
 gtk/gtkwidget.c                      |   25 ++-----------------------
 gtk/gtkwidget.h                      |    3 ---
 gtk/gtkwindow.c                      |    4 ++--
 8 files changed, 10 insertions(+), 48 deletions(-)
---
diff --git a/docs/reference/gtk/gtk4-sections.txt b/docs/reference/gtk/gtk4-sections.txt
index 41bbf9b..b28f209 100644
--- a/docs/reference/gtk/gtk4-sections.txt
+++ b/docs/reference/gtk/gtk4-sections.txt
@@ -4782,7 +4782,6 @@ gtk_widget_set_support_multidevice
 gtk_widget_get_support_multidevice
 gtk_widget_set_realized
 gtk_widget_get_realized
-gtk_widget_set_mapped
 gtk_widget_get_mapped
 gtk_widget_device_is_shadowed
 gtk_widget_get_modifier_mask
diff --git a/gtk/gtkcontainer.c b/gtk/gtkcontainer.c
index 5797aaf..a68fa87 100644
--- a/gtk/gtkcontainer.c
+++ b/gtk/gtkcontainer.c
@@ -3090,28 +3090,17 @@ gtk_container_map_child (GtkWidget *child,
 static void
 gtk_container_map (GtkWidget *widget)
 {
-  gtk_widget_set_mapped (widget, TRUE);
+  GTK_WIDGET_CLASS (parent_class)->map (widget);
 
   gtk_container_forall (GTK_CONTAINER (widget),
                         gtk_container_map_child,
                         NULL);
-
-  if (_gtk_widget_get_has_window (widget))
-    gdk_window_show (_gtk_widget_get_window (widget));
 }
 
 static void
 gtk_container_unmap (GtkWidget *widget)
 {
-  gtk_widget_set_mapped (widget, FALSE);
-
-  /* hide our window first so user doesn't see all the child windows
-   * vanishing one by one.  (only matters these days if one of the
-   * children has an actual native window instead of client-side
-   * window, e.g. a GtkSocket would)
-   */
-  if (_gtk_widget_get_has_window (widget))
-    gdk_window_hide (_gtk_widget_get_window (widget));
+  GTK_WIDGET_CLASS (parent_class)->unmap (widget);
 
   gtk_container_forall (GTK_CONTAINER (widget),
                         (GtkCallback)gtk_widget_unmap,
diff --git a/gtk/gtklayout.c b/gtk/gtklayout.c
index d43e9c0..fc87e25 100644
--- a/gtk/gtklayout.c
+++ b/gtk/gtklayout.c
@@ -817,7 +817,7 @@ gtk_layout_map (GtkWidget *widget)
   GtkLayoutPrivate *priv = layout->priv;
   GList *tmp_list;
 
-  gtk_widget_set_mapped (widget, TRUE);
+  GTK_WIDGET_CLASS (gtk_layout_parent_class)->map (widget);
 
   tmp_list = priv->children;
   while (tmp_list)
diff --git a/gtk/gtkshortcutssection.c b/gtk/gtkshortcutssection.c
index 6ffca92..38c1294 100644
--- a/gtk/gtkshortcutssection.c
+++ b/gtk/gtkshortcutssection.c
@@ -192,7 +192,7 @@ gtk_shortcuts_section_map (GtkWidget *widget)
   if (self->need_reflow)
     gtk_shortcuts_section_reflow_groups (self);
 
-  gtk_widget_set_mapped (widget, TRUE);
+  GTK_WIDGET_CLASS (gtk_shortcuts_section_parent_class)->map (widget);
 
   map_child (GTK_WIDGET (self->stack));
   map_child (GTK_WIDGET (self->footer));
@@ -203,7 +203,7 @@ gtk_shortcuts_section_unmap (GtkWidget *widget)
 {
   GtkShortcutsSection *self = GTK_SHORTCUTS_SECTION (widget);
 
-  gtk_widget_set_mapped (widget, FALSE);
+  GTK_WIDGET_CLASS (gtk_shortcuts_section_parent_class)->unmap (widget);
 
   gtk_widget_unmap (GTK_WIDGET (self->footer));
   gtk_widget_unmap (GTK_WIDGET (self->stack));
diff --git a/gtk/gtktreeview.c b/gtk/gtktreeview.c
index 0887f5f..9747b63 100644
--- a/gtk/gtktreeview.c
+++ b/gtk/gtktreeview.c
@@ -2272,7 +2272,7 @@ gtk_tree_view_map (GtkWidget *widget)
   GtkTreeView *tree_view = GTK_TREE_VIEW (widget);
   GList *tmp_list;
 
-  gtk_widget_set_mapped (widget, TRUE);
+  GTK_WIDGET_CLASS (gtk_tree_view_parent_class)->map (widget);
 
   tmp_list = tree_view->priv->children;
   while (tmp_list)
@@ -2289,8 +2289,6 @@ gtk_tree_view_map (GtkWidget *widget)
   gdk_window_show (tree_view->priv->bin_window);
 
   gtk_tree_view_map_buttons (tree_view);
-
-  gdk_window_show (gtk_widget_get_window (widget));
 }
 
 static void
diff --git a/gtk/gtkwidget.c b/gtk/gtkwidget.c
index 2335936..f74104b 100644
--- a/gtk/gtkwidget.c
+++ b/gtk/gtkwidget.c
@@ -8295,27 +8295,6 @@ gtk_widget_get_mapped (GtkWidget *widget)
 }
 
 /**
- * gtk_widget_set_mapped:
- * @widget: a #GtkWidget
- * @mapped: %TRUE to mark the widget as mapped
- *
- * Marks the widget as being realized.
- *
- * This function should only ever be called in a derived widget's
- * “map” or “unmap” implementation.
- *
- * Since: 2.20
- */
-void
-gtk_widget_set_mapped (GtkWidget *widget,
-                       gboolean   mapped)
-{
-  g_return_if_fail (GTK_IS_WIDGET (widget));
-
-  widget->priv->mapped = mapped;
-}
-
-/**
  * gtk_widget_set_redraw_on_allocate:
  * @widget: a #GtkWidget
  * @redraw_on_allocate: if %TRUE, the entire widget will be redrawn
@@ -10684,7 +10663,7 @@ gtk_widget_real_map (GtkWidget *widget)
 
   if (!_gtk_widget_get_mapped (widget))
     {
-      gtk_widget_set_mapped (widget, TRUE);
+      priv->mapped = TRUE;
 
       if (_gtk_widget_get_has_window (widget))
        gdk_window_show (priv->window);
@@ -10706,7 +10685,7 @@ gtk_widget_real_unmap (GtkWidget *widget)
 
   if (_gtk_widget_get_mapped (widget))
     {
-      gtk_widget_set_mapped (widget, FALSE);
+      priv->mapped = FALSE;
 
       if (_gtk_widget_get_has_window (widget))
        gdk_window_hide (priv->window);
diff --git a/gtk/gtkwidget.h b/gtk/gtkwidget.h
index f81b4eb..d229183 100644
--- a/gtk/gtkwidget.h
+++ b/gtk/gtkwidget.h
@@ -784,9 +784,6 @@ void                  gtk_widget_set_realized           (GtkWidget    *widget,
 GDK_AVAILABLE_IN_ALL
 gboolean              gtk_widget_get_realized           (GtkWidget    *widget);
 GDK_AVAILABLE_IN_ALL
-void                  gtk_widget_set_mapped             (GtkWidget    *widget,
-                                                         gboolean      mapped);
-GDK_AVAILABLE_IN_ALL
 gboolean              gtk_widget_get_mapped             (GtkWidget    *widget);
 
 GDK_AVAILABLE_IN_ALL
diff --git a/gtk/gtkwindow.c b/gtk/gtkwindow.c
index 6242c0d..4cbca1e 100644
--- a/gtk/gtkwindow.c
+++ b/gtk/gtkwindow.c
@@ -5973,7 +5973,7 @@ gtk_window_map (GtkWidget *widget)
   if (priv->initial_fullscreen_monitor > gdk_display_get_n_monitors (display))
     priv->initial_fullscreen_monitor = -1;
 
-  gtk_widget_set_mapped (widget, TRUE);
+  GTK_WIDGET_CLASS (gtk_window_parent_class)->map (widget);
 
   child = gtk_bin_get_child (&(window->bin));
   if (child != NULL && gtk_widget_get_visible (child))
@@ -6120,7 +6120,7 @@ gtk_window_unmap (GtkWidget *widget)
 
   gdk_window = _gtk_widget_get_window (widget);
 
-  gtk_widget_set_mapped (widget, FALSE);
+  GTK_WIDGET_CLASS (gtk_window_parent_class)->unmap (widget);
   gdk_window_withdraw (gdk_window);
 
   while (priv->configure_request_count > 0)


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]