Revert 173477

chrome/browser/chromeos/input_method/xkeyboard.cc

Index: chrome/browser/chromeos/input_method/xkeyboard.cc
===================================================================
--- chrome/browser/chromeos/input_method/xkeyboard.cc	(revision 173477)
+++ chrome/browser/chromeos/input_method/xkeyboard.cc	(working copy)
@@ -14,10 +14,10 @@
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/process_util.h"
-#include "base/sequenced_task_runner.h"
 #include "base/string_util.h"
 #include "base/stringprintf.h"
 #include "chrome/browser/chromeos/input_method/input_method_util.h"
+#include "content/public/browser/browser_thread.h"
 #include "ui/base/x/x11_util.h"
 
 // These includes conflict with base/tracked_objects.h so must come last.
@@ -25,6 +25,8 @@
 #include <X11/Xlib.h>
 #include <glib.h>
 
+using content::BrowserThread;
+
 namespace chromeos {
 namespace input_method {
 namespace {
@@ -60,9 +62,7 @@
 
 class XKeyboardImpl : public XKeyboard {
  public:
-  explicit XKeyboardImpl(
-      const InputMethodUtil& util,
-      const scoped_refptr<base::SequencedTaskRunner>& default_task_runner);
+  explicit XKeyboardImpl(const InputMethodUtil& util);
   virtual ~XKeyboardImpl() {}
 
   // Overridden from XKeyboard:
@@ -80,8 +80,6 @@
   virtual unsigned int GetNumLockMask() OVERRIDE;
   virtual void GetLockedModifiers(bool* out_caps_lock_enabled,
                                   bool* out_num_lock_enabled) OVERRIDE;
-  virtual bool SetAutoRepeatEnabled(bool enabled) OVERRIDE;
-  virtual bool SetAutoRepeatRate(const AutoRepeatRate& rate) OVERRIDE;
 
  private:
   // This function is used by SetLayout() and RemapModifierKeys(). Calls
@@ -112,16 +110,11 @@
   // A queue for executing setxkbmap one by one.
   std::queue<std::string> execute_queue_;
 
-  scoped_refptr<base::SequencedTaskRunner> default_task_runner_;
-
   DISALLOW_COPY_AND_ASSIGN(XKeyboardImpl);
 };
 
-XKeyboardImpl::XKeyboardImpl(
-    const InputMethodUtil& util,
-    const scoped_refptr<base::SequencedTaskRunner>& default_task_runner)
-    : is_running_on_chrome_os_(base::chromeos::IsRunningOnChromeOS()),
-      default_task_runner_(default_task_runner) {
+XKeyboardImpl::XKeyboardImpl(const InputMethodUtil& util)
+    : is_running_on_chrome_os_(base::chromeos::IsRunningOnChromeOS()) {
   num_lock_mask_ = GetNumLockMask();
 
   // web_input_event_aurax11.cc seems to assume that Mod2Mask is always assigned
@@ -324,14 +317,14 @@
   // TODO(yusukes): Stop special-casing browser_tests and remove this function.
   if (!is_running_on_chrome_os_)
     return true;
-  return default_task_runner_->RunsTasksOnCurrentThread();
+  return BrowserThread::CurrentlyOn(BrowserThread::UI);
 }
 
 // static
 void XKeyboardImpl::OnSetLayoutFinish(pid_t pid,
                                       int status,
                                       XKeyboardImpl* self) {
-  CHECK(self->default_task_runner_->RunsTasksOnCurrentThread());
+  CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   DVLOG(1) << "OnSetLayoutFinish: pid=" << pid;
   if (self->execute_queue_.empty()) {
     DVLOG(1) << "OnSetLayoutFinish: execute_queue_ is empty. "
@@ -342,8 +335,11 @@
   self->MaybeExecuteSetLayoutCommand();
 }
 
-bool XKeyboardImpl::SetAutoRepeatEnabled(bool enabled) {
-  CHECK(default_task_runner_->RunsTasksOnCurrentThread());
+}  // namespace
+
+// static
+bool XKeyboard::SetAutoRepeatEnabled(bool enabled) {
+  CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   if (enabled)
     XAutoRepeatOn(ui::GetXDisplay());
   else
@@ -352,8 +348,9 @@
   return true;
 }
 
-bool XKeyboardImpl::SetAutoRepeatRate(const AutoRepeatRate& rate) {
-  CHECK(default_task_runner_->RunsTasksOnCurrentThread());
+// static
+bool XKeyboard::SetAutoRepeatRate(const AutoRepeatRate& rate) {
+  CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   DVLOG(1) << "Set auto-repeat rate to: "
            << rate.initial_delay_in_ms << " ms delay, "
            << rate.repeat_interval_in_ms << " ms interval";
@@ -366,8 +363,6 @@
   return true;
 }
 
-}  // namespace
-
 // static
 bool XKeyboard::GetAutoRepeatEnabledForTesting() {
   XKeyboardState state = {};
@@ -388,10 +383,8 @@
 }
 
 // static
-XKeyboard* XKeyboard::Create(
-    const InputMethodUtil& util,
-    const scoped_refptr<base::SequencedTaskRunner>& default_task_runner) {
-  return new XKeyboardImpl(util, default_task_runner);
+XKeyboard* XKeyboard::Create(const InputMethodUtil& util) {
+  return new XKeyboardImpl(util);
 }
 
 }  // namespace input_method