Add unit tests for SetAutoRepeatEnabled and SetAutoRepeatRate.

chrome/browser/chromeos/input_method/xkeyboard.cc

Index: chrome/browser/chromeos/input_method/xkeyboard.cc
diff --git a/chrome/browser/chromeos/input_method/xkeyboard.cc b/chrome/browser/chromeos/input_method/xkeyboard.cc
index 7de0a7151c68ad1c428ab41df3e1d6e772b45ad5..f1c5cca84e12932acafb4734bd0023ffe1b08d32 100644
--- a/chrome/browser/chromeos/input_method/xkeyboard.cc
+++ b/chrome/browser/chromeos/input_method/xkeyboard.cc
@@ -353,7 +353,6 @@ bool XKeyboard::SetAutoRepeatEnabled(bool enabled) {
 
 // static
 bool XKeyboard::SetAutoRepeatRate(const AutoRepeatRate& rate) {
-  // TODO(yusukes): write auto tests for the function.
   CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   DLOG(INFO) << "Set auto-repeat rate to: "
              << rate.initial_delay_in_ms << " ms delay, "
@@ -367,6 +366,20 @@ bool XKeyboard::SetAutoRepeatRate(const AutoRepeatRate& rate) {
   return true;
 }
 
+// static
+bool XKeyboard::GetAutoRepeatEnabled() {
+  XKeyboardState state = {};
+  XGetKeyboardControl(ui::GetXDisplay(), &state);
+  return state.global_auto_repeat != AutoRepeatModeOff;
+}
+
+// static
+bool XKeyboard::GetAutoRepeatRate(AutoRepeatRate* out_rate) {
+  return XkbGetAutoRepeatRate(ui::GetXDisplay(), XkbUseCoreKbd,
+                              &(out_rate->initial_delay_in_ms),
+                              &(out_rate->repeat_interval_in_ms)) == True;

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

Why "True" instead of "true"?

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

because Xlib/XKB functions return True/False instead of C++ true/false.

+}
+
 void XKeyboard::SetLockedModifiers(ModifierLockStatus new_caps_lock_status,
                                    ModifierLockStatus new_num_lock_status) {
   CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));