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 a32eec9bcd3393b4b683a8d93e3ce6d0960e448a..9f17fe13648a6a5e4b6ef52ea473e4c5e1409bee 100644
--- a/services/ui/ws/window_tree.cc
+++ b/services/ui/ws/window_tree.cc
@@ -1157,20 +1157,34 @@ 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) {
+    for (ui::mojom::EventType event_type : event_type_whitelist) {
+      if (matcher->type_matcher->type == event_type) {
+        allowed = true;
+        break;
+      }
+    }
+  } else if (matcher->pointer_kind_matcher) {

[sky, 2016/07/06 20:22:02]

Shouldn't this conditional be a standalone if? That is, if a type_matcher is
specified you validate it, and if a pointer_kind_matcher is specified you
validate it too?

[riajiang, 2016/07/08 02:51:40]

Yes, changed it to a standalone if.

+    for (ui::mojom::PointerKind pointer : pointer_kind_whitelist) {
+      if (matcher->pointer_kind_matcher->pointer_kind == pointer) {
+        allowed = true;
+        break;
+      }
     }
   }
   if (!allowed) {