Fix preedit handling on IME switching.

chrome/browser/chromeos/input_method/input_method_manager_impl.cc

Index: chrome/browser/chromeos/input_method/input_method_manager_impl.cc
diff --git a/chrome/browser/chromeos/input_method/input_method_manager_impl.cc b/chrome/browser/chromeos/input_method/input_method_manager_impl.cc
index 455e785d4f406713a5913b297157ea23ba05e006..37fa7986e77446a365e0765bd4384b453c2e6fd7 100644
--- a/chrome/browser/chromeos/input_method/input_method_manager_impl.cc
+++ b/chrome/browser/chromeos/input_method/input_method_manager_impl.cc
@@ -248,7 +248,18 @@ void InputMethodManagerImpl::ChangeInputMethodInternal(
     FOR_EACH_OBSERVER(InputMethodManager::Observer,
                       observers_,
                       InputMethodPropertyChanged(this));
-    ibus_controller_->Reset();
+    // Hack for fixing
+    // https://code.google.com/p/chrome-os-partner/issues/detail?id=12798

[Yusuke Sato, 2012/09/04 17:28:35]

http://crosbug.com/p/12798

[Seigo Nonaka, 2012/09/04 17:47:34]

Done.

+    // We should notify ime switching to ibus-daemon, otherwise

[Yusuke Sato, 2012/09/04 17:28:35]

IME

[Seigo Nonaka, 2012/09/04 17:47:34]

Done.

+    // IBusPreeditFocusMode does not work. To achieve it, change engine to
+    // itself if the next engine is xkb layout.

[Yusuke Sato, 2012/09/04 17:28:35]

XKB

[Seigo Nonaka, 2012/09/04 17:47:34]

Done.

+    const std::string current_input_method_id = current_input_method_.id();
+    if (current_input_method_id.empty() ||

[Yusuke Sato, 2012/09/04 17:28:35]

Q. when can |current_input_method_id| be empty?

[Seigo Nonaka, 2012/09/04 17:47:34]

|current_input_method_id| is empty just after logging in.
And also, some unittest performs with empty.

+        InputMethodUtil::IsKeyboardLayout(current_input_method_id)) {
+      ibus_controller_->Reset();
+    } else {
+      ibus_controller_->ChangeInputMethod(current_input_method_id);
+    }
   } else {
     ibus_controller_->ChangeInputMethod(input_method_id_to_switch);
   }