mus: Change PointerWatcher to watch for all events.

ui/views/mus/window_manager_connection.h

Index: ui/views/mus/window_manager_connection.h
diff --git a/ui/views/mus/window_manager_connection.h b/ui/views/mus/window_manager_connection.h
index ea9619ea287a2db3e0047678724d4fb60c73d6ef..8e73a7f7ec52e427feeb7af9e28e24c3be4777eb 100644
--- a/ui/views/mus/window_manager_connection.h
+++ b/ui/views/mus/window_manager_connection.h
@@ -27,8 +27,8 @@ class Connector;
 namespace views {
 class ClipboardMus;
 class NativeWidget;
+class PointerEventWatcher;
 class PointerWatcher;
-class TouchEventWatcher;
 class ScreenMus;
 namespace internal {
 class NativeWidgetDelegate;
@@ -68,8 +68,11 @@ class VIEWS_MUS_EXPORT WindowManagerConnection
   void AddPointerWatcher(PointerWatcher* watcher);

[sadrul, 2016/07/19 00:18:03]

Rename to AddPointerDownWatcher(PointerDownWatcher*)

[riajiang, 2016/07/19 18:46:34]

Will do the renaming in another cl.

   void RemovePointerWatcher(PointerWatcher* watcher);
 
-  void AddTouchEventWatcher(TouchEventWatcher* watcher);
-  void RemoveTouchEventWatcher(TouchEventWatcher* watcher);
+  void AddTouchEventWatcher(PointerEventWatcher* watcher);
+  void RemoveTouchEventWatcher(PointerEventWatcher* watcher);
+
+  void AddMouseEventWatcher(PointerEventWatcher* watcher);
+  void RemoveMouseEventWatcher(PointerEventWatcher* watcher);

[sadrul, 2016/07/19 00:18:03]

Hm, I don't like having two separate methods for install PointerEventWatchers.
Can we instead have a single AddPointerEventWatcher(watcher,
ui::EventPointerType) instead?

[riajiang, 2016/07/19 18:46:34]

Done.

 
   const std::set<ui::Window*>& GetRoots() const;
 
@@ -82,6 +85,7 @@ class VIEWS_MUS_EXPORT WindowManagerConnection
   // Returns true if there is one or more watchers for this client.
   bool HasPointerWatcher();
   bool HasTouchEventWatcher();
+  bool HasMouseEventWatcher();
 
   // ui::WindowTreeClientDelegate:
   void OnEmbed(ui::Window* root) override;
@@ -98,7 +102,8 @@ class VIEWS_MUS_EXPORT WindowManagerConnection
   std::unique_ptr<ui::WindowTreeClient> client_;
   // Must be empty on destruction.
   base::ObserverList<PointerWatcher, true> pointer_watchers_;
-  base::ObserverList<TouchEventWatcher, true> touch_event_watchers_;
+  base::ObserverList<PointerEventWatcher, true> touch_event_watchers_;
+  base::ObserverList<PointerEventWatcher, true> mouse_event_watchers_;
 
   DISALLOW_COPY_AND_ASSIGN(WindowManagerConnection);
 };