Add unit tests for SetAutoRepeatEnabled and SetAutoRepeatRate.

chrome/browser/chromeos/input_method/xkeyboard_unittest.cc

 // Copyright (c) 2011 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/chromeos/input_method/xkeyboard.h"
 
 #include <algorithm>
 #include <set>
 #include <string>
 
@@ skipping 24 matching lines @@
 namespace {
 
 class TestableXKeyboard : public XKeyboard {
  public:
   explicit TestableXKeyboard(const InputMethodUtil& util) : XKeyboard(util) {
   }
 
   // Change access rights.
   using XKeyboard::CreateFullXkbLayoutName;
   using XKeyboard::ContainsModifierKeyAsReplacement;
+  using XKeyboard::GetAutoRepeatEnabled;
+  using XKeyboard::GetAutoRepeatRate;
 };
 
 class XKeyboardTest : public testing::Test {
  public:
   XKeyboardTest()
       : controller_(IBusController::Create()),
         util_(controller_->GetSupportedInputMethods()),
         ui_thread_(BrowserThread::UI, &message_loop_) {
   }
 
@@ skipping 291 matching lines @@
   EXPECT_TRUE(TestableXKeyboard::ContainsModifierKeyAsReplacement(
       GetMap(kVoidKey, kVoidKey, kCapsLockKey), kCapsLockKey));
   EXPECT_TRUE(TestableXKeyboard::ContainsModifierKeyAsReplacement(
       GetMap(kCapsLockKey, kCapsLockKey, kVoidKey), kCapsLockKey));
   EXPECT_TRUE(TestableXKeyboard::ContainsModifierKeyAsReplacement(
       GetMap(kCapsLockKey, kCapsLockKey, kCapsLockKey), kCapsLockKey));
   EXPECT_TRUE(TestableXKeyboard::ContainsModifierKeyAsReplacement(
       GetMap(kSearchKey, kVoidKey, kVoidKey), kSearchKey));
 }
 
+TEST_F(XKeyboardTest, TestSetAutoRepeatEnabled) {
+  if (!DisplayAvailable()) {

[Zachary Kuznia, 2011/11/08 03:55:17]

Should this fail the test?

[Yusuke Sato, 2011/11/13 23:24:56]

The intension of this check is that to allow developers to run unit_tests
without an X server (e.g. via ssh). Please note that both try bots and waterfall
always have an X server for running browser_tests.

I've added a comment and a LOG.

+    return;
+  }
+  const bool state = TestableXKeyboard::GetAutoRepeatEnabled();
+  TestableXKeyboard::SetAutoRepeatEnabled(!state);
+  EXPECT_EQ(!state, TestableXKeyboard::GetAutoRepeatEnabled());
+  // Restore the initial state.
+  TestableXKeyboard::SetAutoRepeatEnabled(state);
+  EXPECT_EQ(state, TestableXKeyboard::GetAutoRepeatEnabled());
+}
+
+TEST_F(XKeyboardTest, TestSetAutoRepeatRate) {
+  if (!DisplayAvailable()) {

[Zachary Kuznia, 2011/11/08 03:55:17]

ditto

[Yusuke Sato, 2011/11/13 23:24:56]

the same as above.

+    return;
+  }
+  AutoRepeatRate rate;
+  EXPECT_TRUE(TestableXKeyboard::GetAutoRepeatRate(&rate));
+
+  AutoRepeatRate tmp(rate);
+  ++tmp.initial_delay_in_ms;
+  ++tmp.repeat_interval_in_ms;
+  EXPECT_TRUE(TestableXKeyboard::SetAutoRepeatRate(tmp));
+  EXPECT_TRUE(TestableXKeyboard::GetAutoRepeatRate(&tmp));
+  EXPECT_EQ(rate.initial_delay_in_ms + 1, tmp.initial_delay_in_ms);
+  EXPECT_EQ(rate.repeat_interval_in_ms + 1, tmp.repeat_interval_in_ms);
+
+  // Restore the initial state.
+  EXPECT_TRUE(TestableXKeyboard::SetAutoRepeatRate(rate));
+  EXPECT_TRUE(TestableXKeyboard::GetAutoRepeatRate(&tmp));
+  EXPECT_EQ(rate.initial_delay_in_ms, tmp.initial_delay_in_ms);
+  EXPECT_EQ(rate.repeat_interval_in_ms, tmp.repeat_interval_in_ms);
+}
+
 }  // namespace input_method
 }  // namespace chromeos