Merge lp:~bregma/libgrip/lp-827958 into lp:libgrip

[Stephen M. Webb]

Registers only those widgets whose toplevel was just mapped in the window-mapped callback.

Previously, an attempt to register ALL pending widgets regardless of toplevel was causing an occasional segfault.

[Stephen M. Webb (bregma)]

A quick re-review of this fix reveals it introduces another bug.

review: Needs Resubmitting

[lp:~bregma/libgrip/lp-827958]

64. By Stephen M. Webb on 2011-10-06

Fixed invalid deletion of all registration requests in window mapping.

65. By Stephen M. Webb on 2011-10-06

Renamed window_mapped_cb to toplevel_mapped_cb to reduce confusion.

Unmerged revisions

=== modified file 'src/gripgesturemanager.c'
--- src/gripgesturemanager.c	2011-09-22 19:55:37 +0000
+++ src/gripgesturemanager.c	2011-10-06 19:27:27 +0000
@@ -1253,7 +1253,7 @@
       GtkWidget *toplevel = gtk_widget_get_toplevel (widget);
       GripRegistrationRequest *req = (GripRegistrationRequest *)user_data;
 
-      if (GTK_IS_WINDOW (toplevel))
+      if (gtk_widget_is_toplevel (toplevel))
         {
           g_signal_handlers_disconnect_by_func (widget,
                                                 G_CALLBACK (toplevel_notify_cb),
@@ -1316,7 +1316,7 @@
 {
   GtkWidget *toplevel = gtk_widget_get_toplevel (widget);
 
-  if (GTK_IS_WINDOW (toplevel))
+  if (gtk_widget_is_toplevel (toplevel))
     {
       register_internal (manager,
                          widget,
@@ -1354,27 +1354,31 @@
 {
   GripGestureManager *manager = (GripGestureManager *)user_data;
   GripGestureManagerPrivate *priv = manager->priv;
-  GList *tmp;
+  GList *request = priv->requests;
 
-  for (tmp = priv->requests; tmp != NULL; tmp = tmp->next)
+  while (request)
     {
-      GripRegistrationRequest *req = tmp->data;
-
-      register_widget (req->manager,
-                       req->widget,
-                       req->gesture_type,
-                       req->device_type,
-                       req->touch_points,
-                       req->callback,
-                       req->user_data,
-                       req->destroy);
-
-      g_free (req);
+      GList *next = request->next;
+      GripRegistrationRequest *req = request->data;
+
+      if (gtk_widget_get_toplevel (req->widget) == widget)
+        {
+	  register_widget (req->manager,
+                           req->widget,
+                           req->gesture_type,
+                           req->device_type,
+                           req->touch_points,
+                           req->callback,
+                           req->user_data,
+                           req->destroy);
+
+          priv->requests = g_list_remove_link (priv->requests, request);
+          g_free (req);
+          g_list_free_1 (request);
+        }
+      request = next;
     }
 
-  g_list_free (priv->requests);
-  priv->requests = NULL;
-
   g_signal_handlers_disconnect_by_func (widget,
                                         window_mapped_cb,
                                         user_data);