Handle Caps Lock short cut (Shift+Search) in ash [part 2 of 2].

chrome/browser/ui/views/aura/caps_lock_handler_browsertest.cc

Index: chrome/browser/ui/views/aura/caps_lock_handler_browsertest.cc
diff --git a/chrome/browser/ui/views/aura/caps_lock_handler_browsertest.cc b/chrome/browser/ui/views/aura/caps_lock_handler_browsertest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..2543b7ae1200892c0364ac0953fb7f0e53c767a8
--- /dev/null
+++ b/chrome/browser/ui/views/aura/caps_lock_handler_browsertest.cc
@@ -0,0 +1,68 @@
+// Copyright (c) 2012 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 "chrome/browser/ui/views/aura/caps_lock_handler.h"
+#include "chrome/test/base/in_process_browser_test.h"
+#include "chrome/test/base/ui_test_utils.h"
+
+#if defined(OS_CHROMEOS)
+#include "chrome/browser/chromeos/input_method/input_method_manager.h"
+#include "chrome/browser/chromeos/input_method/xkeyboard.h"
+#endif
+
+namespace {
+
+class TestableCapsLockHandler : public CapsLockHandler {
+ public:
+  using CapsLockHandler::set_is_running_on_chromeos;
+  using CapsLockHandler::caps_lock_is_on;
+};
+
+#if defined(OS_CHROMEOS)
+class CapsLockHandlerTest : public InProcessBrowserTest {
+ public:
+  CapsLockHandlerTest()
+      : manager_(chromeos::input_method::InputMethodManager::GetInstance()),
+        initial_caps_lock_state_(
+            manager_->GetXKeyboard()->CapsLockIsEnabled()) {
+  }
+
+  virtual void SetUp() OVERRIDE {
+    handler_.reset(new TestableCapsLockHandler);
+    // Force CapsLockHandler::HandleToggleCapsLock() to toggle the lock state.
+    handler_->set_is_running_on_chromeos(true);
+  }
+  virtual void TearDown() OVERRIDE {
+    handler_.reset();
+  }
+
+ protected:
+  chromeos::input_method::InputMethodManager* manager_;
+  const bool initial_caps_lock_state_;
+  scoped_ptr<TestableCapsLockHandler> handler_;
+
+  DISALLOW_COPY_AND_ASSIGN(CapsLockHandlerTest);
+};
+#endif
+
+}  // namespace
+
+// Check if HandleToggleCapsLock() really changes the lock state.
+IN_PROC_BROWSER_TEST_F(CapsLockHandlerTest, TestCapsLock) {
+#if defined(OS_CHROMEOS)
+  EXPECT_EQ(initial_caps_lock_state_, handler_->caps_lock_is_on());
+  EXPECT_TRUE(handler_->HandleToggleCapsLock());
+  EXPECT_EQ(!initial_caps_lock_state_,
+            manager_->GetXKeyboard()->CapsLockIsEnabled());
+  handler_->OnCapsLockChange(!initial_caps_lock_state_);
+  EXPECT_EQ(!initial_caps_lock_state_, handler_->caps_lock_is_on());
+  EXPECT_TRUE(handler_->HandleToggleCapsLock());
+  handler_->OnCapsLockChange(initial_caps_lock_state_);
+  EXPECT_EQ(initial_caps_lock_state_,
+            manager_->GetXKeyboard()->CapsLockIsEnabled());
+  EXPECT_EQ(initial_caps_lock_state_, handler_->caps_lock_is_on());
+  // Restore the original state, just in case.
+  manager_->GetXKeyboard()->SetCapsLockEnabled(initial_caps_lock_state_);

[Daniel Erat, 2012/01/18 15:44:31]

i wish that this were mocked out.  as is, can we end up leaving the caps lock
state modified on the machine where the test is running?  what if the developer
is composing an email message while the test is running?

[Yusuke Sato, 2012/01/23 06:53:49]

Done.

+#endif
+}