Refactoring the ownership of ui::InputMethod.

ash/shell.cc

Index: ash/shell.cc
diff --git a/ash/shell.cc b/ash/shell.cc
index fb61b55893a6e7c274829105a75bea1f9966b97d..58330dfb6a1cb9a1a173fbb97313d3039c43ffcf 100644
--- a/ash/shell.cc
+++ b/ash/shell.cc
@@ -108,7 +108,6 @@
 #include "ui/wm/core/accelerator_filter.h"
 #include "ui/wm/core/compound_event_filter.h"
 #include "ui/wm/core/focus_controller.h"
-#include "ui/wm/core/input_method_event_filter.h"
 #include "ui/wm/core/nested_accelerator_controller.h"
 #include "ui/wm/core/shadow_controller.h"
 #include "ui/wm/core/visibility_controller.h"
@@ -680,7 +679,6 @@ Shell::~Shell() {
   speech_feedback_handler_.reset();
 #endif
   RemovePreTargetHandler(overlay_filter_.get());
-  RemovePreTargetHandler(input_method_filter_.get());
   RemovePreTargetHandler(accelerator_filter_.get());
   RemovePreTargetHandler(event_transformation_handler_.get());
   RemovePreTargetHandler(toplevel_window_event_handler_.get());
@@ -739,6 +737,8 @@ Shell::~Shell() {
   // deleted because it's observing video activity through
   // VideoDetectorObserver interface.
   video_activity_notifier_.reset();
+
+  keyboard::KeyboardController::ResetInstance(NULL);

[James Cook, 2015/06/01 22:21:00]

This used to run on non-Chrome OS platforms. Are you sure it's only needed on
Chrome OS?

Also, nullptr

[Shu Chen, 2015/06/02 04:11:16]

Done. Thanks for catching this. Btw, the destruction of KeyboardController needs
to be before the destruction of DisplayController.

 #endif  // defined(OS_CHROMEOS)
   video_detector_.reset();
   high_contrast_controller_.reset();
@@ -809,8 +809,6 @@ Shell::~Shell() {
   new_window_delegate_.reset();
   media_delegate_.reset();
 
-  keyboard::KeyboardController::ResetInstance(NULL);
-
 #if defined(OS_CHROMEOS)
   display_color_manager_.reset();
   if (display_change_observer_)
@@ -935,10 +933,6 @@ void Shell::Init(const ShellInitParams& init_params) {
   AddPreTargetHandler(overlay_filter_.get());
   AddShellObserver(overlay_filter_.get());
 
-  input_method_filter_.reset(new ::wm::InputMethodEventFilter(
-      root_window->GetHost()->GetAcceleratedWidget()));
-  AddPreTargetHandler(input_method_filter_.get());
-
   accelerator_filter_.reset(new ::wm::AcceleratorFilter(
       scoped_ptr< ::wm::AcceleratorDelegate>(new AcceleratorDelegate).Pass(),
       accelerator_controller_->accelerator_history()));
@@ -1110,7 +1104,6 @@ void Shell::InitRootWindow(aura::Window* root_window) {
   DCHECK(drag_drop_controller_.get());
 
   aura::client::SetFocusClient(root_window, focus_client_.get());
-  input_method_filter_->SetInputMethodPropertyInRootWindow(root_window);
   aura::client::SetActivationClient(root_window, activation_client_);
   ::wm::FocusController* focus_controller =
       static_cast< ::wm::FocusController*>(activation_client_);