keyboard::SendKeyEvent() doesn't need to send events to InputMethod directly.

ui/keyboard/keyboard_util.cc

Index: ui/keyboard/keyboard_util.cc
diff --git a/ui/keyboard/keyboard_util.cc b/ui/keyboard/keyboard_util.cc
index 069ee13c60979d99bd0af60a0e32abacda80cf8d..4dbe12a0890b4a39091212878ff3d102057f081a 100644
--- a/ui/keyboard/keyboard_util.cc
+++ b/ui/keyboard/keyboard_util.cc
@@ -348,13 +348,9 @@ bool SendKeyEvent(const std::string type,
         code,
         dom_code,
         modifiers);
-    if (input_method) {
-      input_method->DispatchKeyEvent(&event);
-    } else {
-      ui::EventDispatchDetails details =
-          host->event_sink()->OnEventFromSource(&event);
-      CHECK(!details.dispatcher_destroyed);
-    }
+    ui::EventDispatchDetails details =
+        host->event_sink()->OnEventFromSource(&event);
+    CHECK(!details.dispatcher_destroyed);

[sadrul, 2017/06/20 12:56:10]

Just to confirm: is there a difference to whether the pre/post target event
handlers are triggered because of this change?

[Hadi, 2017/06/20 14:41:11]

WindowEventDispatcher sends the key event to InputMethod before trying
pre/post-target event handlers which happens later at
EventDispatcher::ProcessEvent(), so I think the behavior for this doesn't change
because of this CL.

   }
   return true;
 }