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 http://crbug.com/p/12798

[Yusuke Sato, 2012/09/04 17:49:53]

crosbug

please also fix the BUG= line.

[Seigo Nonaka, 2012/09/04 18:08:48]

Opps, thanks.
Done.
On 2012/09/04 17:49:53, Yusuke Sato wrote:

+    // We should notify IME switching to ibus-daemon, otherwise
+    // IBusPreeditFocusMode does not work. To achieve it, change engine to
+    // itself if the next engine is XKB layout.
+    const std::string current_input_method_id = current_input_method_.id();
+    if (current_input_method_id.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