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

Unified Diff: components/exo/pointer.cc

Issue 2264503003: exo: WMHelperMus: Hook up event handlers for mus+ash (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 4 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | components/exo/wm_helper_mus.h » ('j') | components/exo/wm_helper_mus.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/exo/pointer.cc
diff --git a/components/exo/pointer.cc b/components/exo/pointer.cc
index 750be6157548ad4dd89e9c3598b011f108b8ecd5..8eb41def9853f34552f05c397c086d7bb3cbaf75 100644
--- a/components/exo/pointer.cc
+++ b/components/exo/pointer.cc
@@ -130,6 +130,9 @@ void Pointer::SetStylusDelegate(PointerStylusDelegate* delegate) {
void Pointer::OnMouseEvent(ui::MouseEvent* event) {
Surface* target = GetEffectiveTargetForEvent(event);
+ if (target == surface_)
reveman 2016/08/22 13:41:42 why is this needed?
Peng 2016/08/22 14:32:40 I don't why, but on mus-ash, this surface_ will ge
reveman 2016/08/22 15:44:08 correct this is the surface for the cursor sprite.
Peng 2016/08/22 16:15:14 I think it must be a bug in mus+ash. I filed a bug
+ return;
+
// If target is different than the current pointer focus then we need to
// generate enter and leave events.
if (target != focus_) {
« no previous file with comments | « no previous file | components/exo/wm_helper_mus.h » ('j') | components/exo/wm_helper_mus.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698