Fixes the initialization of focus and activation management.

apps/shell/browser/shell_desktop_controller.cc

Index: apps/shell/browser/shell_desktop_controller.cc
diff --git a/apps/shell/browser/shell_desktop_controller.cc b/apps/shell/browser/shell_desktop_controller.cc
index 752b198a9d02867297f6eb6ca8c56751657f313c..396758f6ddd9afaf8aebdffb5a77c6357c921b9e 100644
--- a/apps/shell/browser/shell_desktop_controller.cc
+++ b/apps/shell/browser/shell_desktop_controller.cc
@@ -22,7 +22,6 @@
 #include "ui/wm/core/base_focus_rules.h"
 #include "ui/wm/core/compound_event_filter.h"
 #include "ui/wm/core/cursor_manager.h"
-#include "ui/wm/core/default_activation_client.h"
 #include "ui/wm/core/focus_controller.h"
 #include "ui/wm/core/input_method_event_filter.h"
 #include "ui/wm/core/native_cursor_manager.h"
@@ -231,8 +230,7 @@ void ShellDesktopController::CreateRootWindow() {
   test_screen_.reset(aura::TestScreen::Create());
   // TODO(jamescook): Replace this with a real Screen implementation.
   gfx::Screen::SetScreenInstance(gfx::SCREEN_TYPE_NATIVE, test_screen_.get());
-  // TODO(jamescook): Initialize a real input method.
-  ui::InitializeInputMethodForTesting();

[oshima, 2014/06/03 00:42:53]

can you check if app_shell_browsertests passes with this?

[Jun Mukai, 2014/06/03 00:57:46]

It passes.

+  // TODO(mukai): Set up input method.
 
   // Set up basic pieces of ui::wm.
   gfx::Size size = GetPrimaryDisplaySize();
@@ -253,16 +251,18 @@ void ShellDesktopController::CreateRootWindow() {
 }
 
 void ShellDesktopController::InitWindowManager() {
-  focus_client_.reset(new wm::FocusController(new AppsFocusRules()));
-  aura::client::SetFocusClient(host_->window(), focus_client_.get());
+  wm::FocusController* focus_controller =
+      new wm::FocusController(new AppsFocusRules());
+  aura::client::SetFocusClient(host_->window(), focus_controller);
+  host_->window()->AddPreTargetHandler(focus_controller);
+  aura::client::SetActivationClient(host_->window(), focus_controller);
+  focus_client_.reset(focus_controller);
 
   input_method_filter_.reset(
       new wm::InputMethodEventFilter(host_->GetAcceleratedWidget()));
   input_method_filter_->SetInputMethodPropertyInRootWindow(host_->window());
   root_window_event_filter_->AddHandler(input_method_filter_.get());
 
-  new wm::DefaultActivationClient(host_->window());
-
   capture_client_.reset(
       new aura::client::DefaultCaptureClient(host_->window()));
 
@@ -294,6 +294,12 @@ void ShellDesktopController::DestroyRootWindow() {
     host_->event_processor()->GetRootTarget()->RemovePreTargetHandler(
         user_activity_detector_.get());
   }
+  wm::FocusController* focus_controller =
+      static_cast<wm::FocusController*>(focus_client_.get());
+  if (focus_controller) {
+    host_->window()->RemovePreTargetHandler(focus_controller);
+    aura::client::SetActivationClient(host_->window(), NULL);
+  }
   root_window_event_filter_.reset();
   capture_client_.reset();
   input_method_filter_.reset();
@@ -304,7 +310,6 @@ void ShellDesktopController::DestroyRootWindow() {
 #endif
   user_activity_detector_.reset();
   host_.reset();
-  ui::ShutdownInputMethodForTesting();
 }
 
 gfx::Size ShellDesktopController::GetPrimaryDisplaySize() {