Fix preedit handling on IME switching.

chrome/browser/chromeos/input_method/input_method_manager_impl.cc

 // 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/chromeos/input_method/input_method_manager_impl.h"
 
 #include <algorithm>  // std::find
 
 #include "base/basictypes.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 230 matching lines @@
       DVLOG(1) << "Can't change the current input method to "
                << input_method_id << " since the engine is not enabled. "
                << "Switch to " << input_method_id_to_switch << " instead.";
     }
   }
 
   if (InputMethodUtil::IsKeyboardLayout(input_method_id_to_switch)) {
     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);
   }
 
   if (current_input_method_.id() != input_method_id_to_switch) {
     const InputMethodDescriptor* descriptor = NULL;
     if (!InputMethodUtil::IsExtensionInputMethod(input_method_id_to_switch)) {
       descriptor =
           util_.GetInputMethodDescriptorFromId(input_method_id_to_switch);
     } else {
@@ skipping 350 matching lines @@
     DVLOG(1) << "Failed to initialize the candidate window controller";
 }
 
 // static
 InputMethodManagerImpl* InputMethodManagerImpl::GetInstanceForTesting() {
   return new InputMethodManagerImpl;
 }
 
 }  // namespace input_method
 }  // namespace chromeos