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

Unified Diff: ui/views/pointer_watcher.h

Issue 2183163002: mus: Change PointerWatcher to observe all pointer events, with moves optional. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: pointer enter/exit; has_pointer_watcher etc 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
Index: ui/views/pointer_watcher.h
diff --git a/ui/views/pointer_watcher.h b/ui/views/pointer_watcher.h
index 5ca10efb4b081548a46c51a77c415cdaf7cf7883..008187180346388f91725772325b51b4bb3c41fc 100644
--- a/ui/views/pointer_watcher.h
+++ b/ui/views/pointer_watcher.h
@@ -13,8 +13,7 @@ class Point;
}
namespace ui {
-class MouseEvent;
-class TouchEvent;
+class PointerEvent;
}
namespace views {
@@ -23,18 +22,18 @@ class Widget;
// An interface for read-only observation of pointer events (in particular, the
// events cannot be marked as handled). Only certain event types are supported.
// The |target| is the top-level widget that will receive the event, if any.
+// To reduce IPC traffic from the window server, move events are not provided
+// unless the app specifically requests them.
// NOTE: On mus this allows observation of events outside of windows owned
// by the current process, in which case the |target| will be null. On mus
// event.target() is always null.
class VIEWS_EXPORT PointerWatcher {
public:
PointerWatcher() {}
- virtual void OnMousePressed(const ui::MouseEvent& event,
- const gfx::Point& location_in_screen,
- Widget* target) = 0;
- virtual void OnTouchPressed(const ui::TouchEvent& event,
- const gfx::Point& location_in_screen,
- Widget* target) = 0;
+
+ virtual void OnPointerEventObserved(const ui::PointerEvent& event,
+ const gfx::Point& location_in_screen,
+ Widget* target) = 0;
protected:
virtual ~PointerWatcher() {}

Powered by Google App Engine
This is Rietveld 408576698