mus: Add watcher for all touch events.

services/ui/ws/window_tree.cc

Index: services/ui/ws/window_tree.cc
diff --git a/services/ui/ws/window_tree.cc b/services/ui/ws/window_tree.cc
index 8a62bdd534534c4c4b111647511e306071068afe..a9a63359544ea389beffef7eaeebf9287ca6f9e4 100644
--- a/services/ui/ws/window_tree.cc
+++ b/services/ui/ws/window_tree.cc
@@ -1165,21 +1165,32 @@ void WindowTree::SetEventObserver(mojom::EventMatcherPtr matcher,
 
   // Do not allow key events to be observed, as a compromised app could register
   // itself as an event observer and spy on keystrokes to another app.
-  if (!matcher->type_matcher) {
+  if (!matcher->type_matcher && !matcher->pointer_kind_matcher) {
     DVLOG(1) << "SetEventObserver must specify an event type.";
     return;
   }
+
   const ui::mojom::EventType event_type_whitelist[] = {
       ui::mojom::EventType::POINTER_CANCEL, ui::mojom::EventType::POINTER_DOWN,
       ui::mojom::EventType::POINTER_MOVE,   ui::mojom::EventType::POINTER_UP,
       ui::mojom::EventType::MOUSE_EXIT,     ui::mojom::EventType::WHEEL,
   };
+  const ui::mojom::PointerKind pointer_kind_whitelist[] = {
+      ui::mojom::PointerKind::MOUSE, ui::mojom::PointerKind::TOUCH,
+  };
+
   bool allowed = false;
-  for (ui::mojom::EventType event_type : event_type_whitelist) {
-    if (matcher->type_matcher->type == event_type) {
-      allowed = true;
-      break;
-    }
+  if (matcher->type_matcher) {
+    auto iter =
+        std::find(std::begin(event_type_whitelist),
+                  std::end(event_type_whitelist), matcher->type_matcher->type);
+    allowed = iter != std::end(event_type_whitelist);
+  }
+  if (matcher->pointer_kind_matcher) {
+    auto iter = std::find(std::begin(pointer_kind_whitelist),
+                          std::end(pointer_kind_whitelist),
+                          matcher->pointer_kind_matcher->pointer_kind);
+    allowed = iter != std::end(pointer_kind_whitelist);

[sadrul, 2016/07/12 19:21:05]

Actually, let's get rid of this whitelist. Just do:

  if (matcher->type_matcher) {
    ..
  }

  if (matcher->pointer_kind_matcher) {
    allowed = pointer_kind == MOUSE || pointer_kind == TOUCH || pointer_kind ==
PEN;
  }

[riajiang, 2016/07/12 20:35:47]

Done.

   }
   if (!allowed) {
     DVLOG(1) << "SetEventObserver event type not allowed";