Do not consume Alt-Shift-Up event for IME switch.

chrome/browser/ui/ash/ime_controller_chromeos.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/ui/ash/ime_controller_chromeos.h"
 
 #include "chromeos/ime/input_method_manager.h"
 #include "ui/base/accelerators/accelerator.h"
 
 bool ImeController::HandleNextIme() {
   chromeos::input_method::InputMethodManager* manager =
       chromeos::input_method::InputMethodManager::Get();
-  return manager->SwitchToNextInputMethod();
+  manager->SwitchToNextInputMethod();
+  return false;
 }
 
 bool ImeController::HandlePreviousIme(const ui::Accelerator& accelerator) {
   chromeos::input_method::InputMethodManager* manager =
       chromeos::input_method::InputMethodManager::Get();
   return manager->SwitchToPreviousInputMethod(accelerator);

[Shu Chen, 2014/06/24 00:48:49]

Why not do the same as HandleNextIme()?

[Seigo Nonaka, 2014/06/24 04:35:01]

HandlePreviousIme is bound to "Ctrl-Space".
If we don't consume Ctrl-Space key event, the whitespace will be inserted into
text field.

On 2014/06/24 00:48:49, Shu Chen wrote:

 }
 
 bool ImeController::HandleSwitchIme(const ui::Accelerator& accelerator) {
   chromeos::input_method::InputMethodManager* manager =
       chromeos::input_method::InputMethodManager::Get();
   return manager->SwitchInputMethod(accelerator);
 }
 
 ui::Accelerator ImeController::RemapAccelerator(
     const ui::Accelerator& accelerator) {
@@ skipping 17 matching lines @@
 }
 
 bool ImeController::UsingFrenchInputMethod() const {
   chromeos::input_method::InputMethodManager* manager =
       chromeos::input_method::InputMethodManager::Get();
   const chromeos::input_method::InputMethodDescriptor& descriptor =
       manager->GetCurrentInputMethod();
   const std::string& layout = descriptor.id();
   return (layout == "xkb:fr::fra" || layout == "xkb:be::fra");
 }