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

Unified Diff: ui/views/widget/desktop_aura/desktop_window_tree_host_x11.cc

Issue 749063003: Fix grabbing capture when the mouse is pressed on Desktop Linux. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years 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 side-by-side diff with in-line comments
Download patch
Index: ui/views/widget/desktop_aura/desktop_window_tree_host_x11.cc
diff --git a/ui/views/widget/desktop_aura/desktop_window_tree_host_x11.cc b/ui/views/widget/desktop_aura/desktop_window_tree_host_x11.cc
index 3c992813568aa97c66c5617ef3a75b1be63f99dd..736129d9f477c4008b734a9378f0f83dcaba3284 100644
--- a/ui/views/widget/desktop_aura/desktop_window_tree_host_x11.cc
+++ b/ui/views/widget/desktop_aura/desktop_window_tree_host_x11.cc
@@ -956,10 +956,7 @@ void DesktopWindowTreeHostX11::SetCapture() {
if (old_capturer)
old_capturer->OnHostLostWindowCapture();
- unsigned int event_mask = PointerMotionMask | ButtonReleaseMask |
- ButtonPressMask;
- XGrabPointer(xdisplay_, xwindow_, True, event_mask, GrabModeAsync,
- GrabModeAsync, None, None, CurrentTime);
+ ui::GrabPointer(xwindow_, true, None);
}
void DesktopWindowTreeHostX11::ReleaseCapture() {
@@ -968,7 +965,7 @@ void DesktopWindowTreeHostX11::ReleaseCapture() {
// the topmost window underneath the mouse so the capture release being
// asynchronous is likely inconsequential.
g_current_capture = NULL;
- XUngrabPointer(xdisplay_, CurrentTime);
+ ui::UngrabPointer();
OnHostLostWindowCapture();
}

Powered by Google App Engine
This is Rietveld 408576698