Remove InputMethodEventHandler.

ui/aura/window_tree_host.cc

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/aura/window_tree_host.h"
 
 #include "base/threading/thread_task_runner_handle.h"
 #include "base/trace_event/trace_event.h"
 #include "ui/aura/client/capture_client.h"
 #include "ui/aura/client/cursor_client.h"
@@ skipping 174 matching lines @@
   }
   return input_method_;
 }
 
 void WindowTreeHost::SetSharedInputMethod(ui::InputMethod* input_method) {
   DCHECK(!input_method_);
   input_method_ = input_method;
   owned_input_method_ = false;
 }
 
 ui::EventDispatchDetails WindowTreeHost::DispatchKeyEventPostIME(

[sadrul, 2017/05/15 18:14:54]

Should WindowTreeHost::DispatchKeyEventPostIME() be marked 'final'?

[Hadi, 2017/05/16 14:53:59]

Done.

     ui::KeyEvent* event) {
-  return SendEventToSink(event);
+  dispatcher_->set_skip_ime(true);
+  ui::EventDispatchDetails dispatch_details = SendEventToSink(event);
+  dispatcher_->set_skip_ime(false);

[sadrul, 2017/05/15 18:14:54]

Check |details.dispatcher_destroyed| before using |dispatcher_| after call to
SendEventToSink().

Also, it looks like test::DisableIME() would break after the first key event
dispatch. Should you skip this if skip_ime_ was already set when entering this
function?

[Hadi, 2017/05/16 14:53:59]

Done.
If IME is disabled (i.e. skip_ime is previously true), execution shouldn't reach
here. Added a DCHECK() for that.

+  return dispatch_details;
 }
 
 void WindowTreeHost::Show() {
   // Ensure that compositor has been properly initialized, see InitCompositor()
   // and InitHost().
   DCHECK(compositor());
   DCHECK_EQ(compositor()->root_layer(), window()->layer());
   compositor()->SetVisible(true);
   ShowImpl();
   window()->Show();
@@ skipping 135 matching lines @@
   client::CursorClient* cursor_client = client::GetCursorClient(window());
   if (cursor_client) {
     const display::Display& display =
         display::Screen::GetScreen()->GetDisplayNearestWindow(window());
     cursor_client->SetDisplay(display);
   }
   dispatcher()->OnCursorMovedToRootLocation(root_location);
 }
 
 }  // namespace aura