Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(149)

Side by Side Diff: ui/views/widget/desktop_aura/x11_pointer_grab.cc

Issue 821803002: Use XGrabPointer instead of XChangeActivatePointerGrab() to change the cursor during a pointer grab (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@fix_capture
Patch Set: Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "base/logging.h"
6 #include "ui/views/widget/desktop_aura/x11_pointer_grab.h"
7
8 #include <X11/Xlib.h>
9
10 namespace views {
11
12 namespace {
13
14 // The grab window. None if there are no active pointer grabs.
15 XID g_grab_window = None;
16
17 // The "owner events" parameter used to grab the pointer.
18 bool g_owner_events = false;
19
20 } // namespace
21
22 int GrabPointer(XID window, bool owner_events, ::Cursor cursor) {
23 int event_mask = PointerMotionMask | ButtonReleaseMask | ButtonPressMask;
24 int result = XGrabPointer(gfx::GetXDisplay(), window, owner_events,
25 event_mask, GrabModeAsync, GrabModeAsync, None,
26 cursor, CurrentTime);
27 if (result == GrabSuccess) {
28 g_grab_window = window;
29 g_owner_events = owner_events;
30 }
31 return result;
32 }
33
34 void ChangeActivePointerGrabCursor(::Cursor cursor) {
35 DCHECK(g_grab_window != None);
36 GrabPointer(g_grab_window, g_owner_events, cursor);
37 }
38
39 void UngrabPointer() {
40 g_grab_window = None;
41 XUngrabPointer(gfx::GetXDisplay(), CurrentTime);
42 }
43
44 } // namespace views
OLDNEW
« no previous file with comments | « ui/views/widget/desktop_aura/x11_pointer_grab.h ('k') | ui/views/widget/desktop_aura/x11_whole_screen_move_loop.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698