From 90c25f0cfe95f153464b0272a02ff85a969a41df Mon Sep 17 00:00:00 2001
From: Carlos Garnacho <carlosg@gnome.org>
Date: Tue, 21 Jun 2011 18:28:55 +0200
Subject: [PATCH] display: Use meta_device_grab() for pointer active grabs.

---
 src/core/display.c | 52 +++++++++++++++++++++++-----------------------
 1 file changed, 26 insertions(+), 26 deletions(-)

diff --git a/src/core/display.c b/src/core/display.c
index 0b6d9b7f2..08e9ac959 100644
--- a/src/core/display.c
+++ b/src/core/display.c
@@ -3408,12 +3408,12 @@ meta_display_create_x_cursor (MetaDisplay *display,
   return xcursor;
 }
 
-static Cursor
-xcursor_for_op (MetaDisplay *display,
-                MetaGrabOp   op)
+static MetaCursor
+cursor_for_op (MetaDisplay *display,
+               MetaGrabOp   op)
 {
   MetaCursor cursor = META_CURSOR_DEFAULT;
-  
+
   switch (op)
     {
     case META_GRAB_OP_RESIZING_SE:
@@ -3458,9 +3458,7 @@ xcursor_for_op (MetaDisplay *display,
       break;
     }
 
-  if (cursor == META_CURSOR_DEFAULT)
-    return None;
-  return meta_display_create_x_cursor (display, cursor);
+  return cursor;
 }
 
 void
@@ -3473,9 +3471,9 @@ meta_display_set_grab_op_cursor (MetaDisplay *display,
                                  guint32      timestamp)
 {
   MetaGrabInfo *grab_info;
-  Cursor cursor;
+  MetaCursor cursor;
 
-  cursor = xcursor_for_op (display, op);
+  cursor = cursor_for_op (display, op);
   grab_info = meta_display_get_grab_info (display, device);
 
 #define GRAB_MASK (PointerMotionMask |                          \
@@ -3484,6 +3482,10 @@ meta_display_set_grab_op_cursor (MetaDisplay *display,
 
   if (change_pointer)
     {
+      Cursor xcursor;
+
+      xcursor = meta_display_create_x_cursor (display, cursor);
+
       meta_error_trap_push_with_return (display);
       XChangeActivePointerGrab (display->xdisplay,
                                 GRAB_MASK,
@@ -3500,39 +3502,37 @@ meta_display_set_grab_op_cursor (MetaDisplay *display,
           if (grab_info->grab_have_pointer)
             grab_info->grab_have_pointer = FALSE;
         }
+
+      if (xcursor != None)
+        XFreeCursor (display->xdisplay, xcursor);
     }
   else
     {
       g_assert (screen != NULL);
 
       meta_error_trap_push (display);
-      if (XGrabPointer (display->xdisplay,
-                        grab_xwindow,
-                        False,
-                        GRAB_MASK,
-                        GrabModeAsync, GrabModeAsync,
-                        screen->xroot,
-                        cursor,
-                        timestamp) == GrabSuccess)
+      if (meta_device_grab (grab_info->grab_pointer,
+                            grab_xwindow,
+                            GRAB_MASK,
+                            cursor,
+                            FALSE,
+                            FALSE,
+                            timestamp))
         {
           grab_info->grab_have_pointer = TRUE;
           meta_topic (META_DEBUG_WINDOW_OPS,
-                      "XGrabPointer() returned GrabSuccess time %u\n",
+                      "grabbing pointer succeeded time %u\n",
                       timestamp);
         }
       else
         {
           meta_topic (META_DEBUG_WINDOW_OPS,
-                      "XGrabPointer() failed time %u\n",
+                      "grabbing pointer failed time %u\n",
                       timestamp);
         }
       meta_error_trap_pop (display);
     }
-
 #undef GRAB_MASK
-  
-  if (cursor != None)
-    XFreeCursor (display->xdisplay, cursor);
 }
 
 gboolean
@@ -3619,7 +3619,7 @@ meta_display_begin_grab_op (MetaDisplay *display,
   if (!grab_info->grab_have_pointer)
     {
       meta_topic (META_DEBUG_WINDOW_OPS,
-                  "XGrabPointer() failed\n");
+                  "grabbing pointer failed\n");
       return FALSE;
     }
 
@@ -3642,7 +3642,7 @@ meta_display_begin_grab_op (MetaDisplay *display,
         {
           meta_topic (META_DEBUG_WINDOW_OPS,
                       "grabbing all keys failed, ungrabbing pointer\n");
-          XUngrabPointer (display->xdisplay, timestamp);
+          meta_device_ungrab (grab_info->grab_pointer, timestamp);
           grab_info->grab_have_pointer = FALSE;
           return FALSE;
         }
@@ -3843,7 +3843,7 @@ meta_display_end_grab_op (MetaDisplay *display,
     {
       meta_topic (META_DEBUG_WINDOW_OPS,
                   "Ungrabbing pointer with timestamp %u\n", timestamp);
-      XUngrabPointer (display->xdisplay, timestamp);
+      meta_device_ungrab (grab_info->grab_pointer, timestamp);
     }
 
   if (grab_info->grab_have_keyboard)