mus: Change PointerWatcher to observe all pointer events, with moves optional.

ui/views/pointer_watcher.h

Index: ui/views/pointer_watcher.h
diff --git a/ui/views/pointer_watcher.h b/ui/views/pointer_watcher.h
index 5ca10efb4b081548a46c51a77c415cdaf7cf7883..105b349f14391267c68de303ac04672791a401f2 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 {
@@ -29,12 +28,13 @@ class Widget;
 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;
+
+  // Called with mouse and touch events. To reduce IPC traffic from the window
+  // server, move events are not provided unless the app specifically requests
+  // them.

[sadrul, 2016/07/29 20:26:10]

Add the note in the comment above instead.

[riajiang, 2016/08/02 19:59:53]

Done.

+  virtual void OnPointerWatcherEvent(const ui::PointerEvent& event,
+                                     const gfx::Point& location_in_screen,
+                                     Widget* target) = 0;

[sadrul, 2016/07/29 20:26:10]

Let's call this OnPointerEvent

[James Cook, 2016/08/01 16:43:54]

Sadrul, are you sure? That makes it look a lot like EventHandler methods
OnMouseEvent, OnTouchEvent, etc. and there's currently a class 
(ShelfTooltipManager) that is both an EventHandler and a PointerWatcher. I
prefer what it is now, but if you feel strongly I'm OK with OnPointerEvent.

[sadrul, 2016/08/02 16:13:51]

Hm. I agree. Calling this OnPointerEvent could be a bit confusing. I am not a
huge fan of the current name though. How about OnPointerEventObserved()?

[James Cook, 2016/08/02 16:18:41]

sgtm

[riajiang, 2016/08/02 19:59:53]

Done.

 
  protected:
   virtual ~PointerWatcher() {}