Avoid two targeting phases in aura client-lib and EventProcessor.

ui/events/event_processor.cc

Index: ui/events/event_processor.cc
diff --git a/ui/events/event_processor.cc b/ui/events/event_processor.cc
index 61d2e72138a8dc51beb04f960b19197d715c750d..18f3a5577f35ea8982cbcb35914b4f166953b9a0 100644
--- a/ui/events/event_processor.cc
+++ b/ui/events/event_processor.cc
@@ -10,11 +10,6 @@
 namespace ui {
 
 EventDispatchDetails EventProcessor::OnEventFromSource(Event* event) {
-  EventTarget* root = GetRootTarget();
-  CHECK(root);
-  EventTargeter* targeter = root->GetEventTargeter();
-  CHECK(targeter);
-
   // If |event| is in the process of being dispatched or has already been
   // dispatched, then dispatch a copy of the event instead.
   bool dispatch_original_event = event->phase() == EP_PREDISPATCH;
@@ -26,9 +21,17 @@ EventDispatchDetails EventProcessor::OnEventFromSource(Event* event) {
   }
 
   OnEventProcessingStarted(event_to_dispatch);
-  EventTarget* target = NULL;
-  if (!event_to_dispatch->handled())
+  EventTarget* target = nullptr;
+  EventTargeter* targeter = nullptr;
+  if (!event_to_dispatch->handled()) {
+    EventTarget* root = GetRootForEvent(event_to_dispatch);
+    DCHECK(root);
+    targeter = root->GetEventTargeter();
+    if (!targeter)
+      targeter = GetDefaultEventTargeter();

[sadrul, 2017/02/13 17:05:42]

Can you add a test in event_processor_unittest.cc that exercises this code?

[riajiang, 2017/02/13 23:30:34]

Added tests for this part of the code in
WindowTreeClientClientTest.InputEventFindTargetAndConversion case 2 and 3. Also
added test for custom WindowTargeter.

+    DCHECK(targeter);
     target = targeter->FindTargetForEvent(root, event_to_dispatch);
+  }
 
   EventDispatchDetails details;
   while (target) {