Merge lp:~achiang/appmenu-gtk/memleaks-787736-780602 into lp:appmenu-gtk/0.4

[Alex Chiang]

Cherrypick r158 from trunk to help fix nm-applet in Precise.

[Charles Kerr (charlesk)]

Alex, you'll need to pull in r160 as well

[lp:~achiang/appmenu-gtk/memleaks-787736-780602]

154. By Alex Chiang on 2012-11-28

Backport r160 from trunk to eliminate noisy debug statements

[Charles Kerr (charlesk)]

review: Approve

=== modified file 'src/bridge.c'
--- src/bridge.c	2012-04-02 19:12:37 +0000
+++ src/bridge.c	2012-11-28 18:08:20 +0000
@@ -85,21 +85,13 @@
 
   GDBusProxy       *appmenuproxy;
   gboolean          online;
+
+  GSList           *rebuild_list;
+  guint             rebuild_timer_id;
 };
 
-typedef struct _RecurseContext
-{
-  AppMenuBridge    *bridge;
-  AppWindowContext *context;
-
-  gint count;
-  gboolean previous;
-  DbusmenuMenuitem *stack[30];
-} RecurseContext;
-
 G_DEFINE_DYNAMIC_TYPE(AppMenuBridge, app_menu_bridge, UBUNTU_TYPE_MENU_PROXY)
 
-static GHashTable *rebuild_ids = NULL;
 static GDBusNodeInfo *      registrar_node_info = NULL;
 static GDBusInterfaceInfo * registrar_interface_info = NULL;
 
@@ -213,28 +205,58 @@
   g_free (context);
 }
 
+
+/* a widget that was queued for rebuild has been destroyed,
+   so remove it from our queue */
+static void
+rebuild_list_widget_destroyed (gpointer gbridge, GObject * dead_object)
+{
+  AppMenuBridge * bridge = APP_MENU_BRIDGE (gbridge);
+  AppMenuBridgePrivate * p = bridge->priv;
+
+  p->rebuild_list = g_slist_remove (p->rebuild_list, dead_object);
+}
+
+static void
+rebuild_list_clear (AppMenuBridge * bridge)
+{
+  GSList * l;
+  AppMenuBridgePrivate * p = bridge->priv;
+
+  /* unref the widgets and blow away the list */
+  for (l=p->rebuild_list; l!=NULL; l=l->next)
+    g_object_weak_unref (l->data, rebuild_list_widget_destroyed, bridge);
+  g_slist_free (p->rebuild_list);
+  p->rebuild_list = NULL;
+
+  /* remove the rebuild timer */
+  if (p->rebuild_timer_id > 0)
+    {
+      g_source_remove (p->rebuild_timer_id);
+      p->rebuild_timer_id = 0;
+    }
+}
+
 static void
 app_menu_bridge_dispose (GObject *object)
 {
   AppMenuBridge *bridge = APP_MENU_BRIDGE (object);
-
-  g_list_foreach (bridge->priv->windows, (GFunc)context_dispose, NULL);
-
-  if (bridge->priv->appmenuproxy)
-    {
-      g_object_unref (bridge->priv->appmenuproxy);
-      bridge->priv->appmenuproxy = NULL;
-    }
+  AppMenuBridgePrivate * p = bridge->priv;
+
+  rebuild_list_clear (bridge);
+  g_list_foreach (p->windows, (GFunc)context_dispose, NULL);
+  g_clear_object (&p->appmenuproxy);
 }
 
 static void
 app_menu_bridge_finalize (GObject *object)
 {
   AppMenuBridge *bridge = APP_MENU_BRIDGE (object);
+  AppMenuBridgePrivate * p = bridge->priv;
 
-  g_list_foreach (bridge->priv->windows, (GFunc)context_free, NULL);
-  g_list_free (bridge->priv->windows);
-  bridge->priv->windows = NULL;
+  g_list_foreach (p->windows, (GFunc)context_free, NULL);
+  g_list_free (p->windows);
+  p->windows = NULL;
 
   G_OBJECT_CLASS (app_menu_bridge_parent_class)->finalize (object);
 }
@@ -505,63 +527,51 @@
 	return;
 }
 
-typedef struct _RebuildData {
-  AppMenuBridge *bridge;
-  GtkWidget     *widget;
-} RebuildData;
-
 static gboolean
-do_rebuild (RebuildData *data)
+rebuild_timer_func (gpointer gbridge)
 {
-  if (data->widget != NULL && gtk_widget_is_toplevel (data->widget))
-    {
-      rebuild_window_items (data->bridge,
-                            data->widget);
-    }
-
-  if (data->widget != NULL)
-    {
-      g_object_remove_weak_pointer (G_OBJECT (data->widget), (gpointer*)&data->widget);
-      g_hash_table_remove (rebuild_ids, data->widget);
-    }
-
-  g_free (data);
-
-  return FALSE;
+  GSList * l;
+  AppMenuBridge *bridge = APP_MENU_BRIDGE (gbridge);
+  AppMenuBridgePrivate * p = bridge->priv;
+
+  /* call rebuild_window_items() for each toplevel in our list */
+  for (l=p->rebuild_list; l!=NULL; l=l->next)
+    {
+      GtkWidget * w = GTK_WIDGET(l->data);
+
+      if (gtk_widget_is_toplevel(w))
+        rebuild_window_items (bridge, w);
+    }
+
+  /* cleanup */
+  rebuild_list_clear (bridge);
+  return G_SOURCE_REMOVE;
 }
 
+/* widgets tend to get a lot of changes all at once...
+ + inserting a 100 msec delay here lets us batch up those changes */
 static void
-rebuild (AppMenuBridge *bridge,
-         GtkWidget     *toplevel)
+rebuild (AppMenuBridge * bridge, GtkWidget * toplevel)
 {
-  guint id = 0;
-
-  if (rebuild_ids != NULL)
-    {
-      id = GPOINTER_TO_UINT (g_hash_table_lookup (rebuild_ids, toplevel));
-
-      if (id > 0)
-        {
-          g_source_remove (id);
-          g_hash_table_remove (rebuild_ids, toplevel);
-          id = 0;
-        }
-    }
-
-  RebuildData *data = g_new0 (RebuildData, 1);
-  data->bridge = bridge;
-  data->widget = toplevel;
-
-  id = gdk_threads_add_timeout (100, (GSourceFunc)do_rebuild, data);
-
-  g_object_add_weak_pointer (G_OBJECT (data->widget), (gpointer*)&data->widget);
-
-  if (rebuild_ids == NULL)
-    {
-      rebuild_ids = g_hash_table_new (g_direct_hash, g_direct_equal);
-    }
-
-  g_hash_table_insert (rebuild_ids, toplevel, GUINT_TO_POINTER (id));
+  AppMenuBridgePrivate * p = bridge->priv;
+
+  g_return_if_fail (toplevel != NULL);
+
+  /* ensure there's a rebuild timer ticking */
+  if (p->rebuild_timer_id == 0)
+    {
+      p->rebuild_timer_id = gdk_threads_add_timeout (100,
+                                                     rebuild_timer_func,
+                                                     bridge);
+    }
+
+  /* ensure this widget is in our rebuild list */
+  if (g_slist_find (p->rebuild_list, toplevel) == NULL)
+    {
+      GObject * o = G_OBJECT(toplevel);
+      p->rebuild_list = g_slist_prepend (p->rebuild_list, o);
+      g_object_weak_ref (o, rebuild_list_widget_destroyed, bridge);
+    }
 }
 
 static DbusmenuMenuitem * find_menu_bar (GtkWidget * widget);
@@ -985,9 +995,4 @@
 G_MODULE_EXPORT void
 menu_proxy_module_unload (UbuntuMenuProxyModule *module)
 {
-  if (rebuild_ids)
-    {
-      g_hash_table_destroy (rebuild_ids);
-      rebuild_ids = NULL;
-    }
 }