Remove the deprecated Alt+Shift shortcut completely

ash/common/accelerators/accelerator_table.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 "ash/common/accelerators/accelerator_table.h"
 
 #include "ash/common/strings/grit/ash_strings.h"
 #include "base/macros.h"
 
 namespace ash {
@@ skipping 184 matching lines @@
 //    - The IDs of the localized old and new shortcut text to be used to fill
 //      the notification text. Also found in |ash_chromeos_strings.grdp|.
 //    - {true or false} whether the deprecated accelerator is still enabled (we
 //      don't disable a deprecated accelerator abruptly).
 // 5- Don't forget to update the keyboard overlay. Find 'shortcut' in the file
 //    keyboard_overlay_data.js.
 #if defined(OS_CHROMEOS)
 
 const AcceleratorData kDeprecatedAccelerators[] = {
     {true, ui::VKEY_L, ui::EF_SHIFT_DOWN | ui::EF_CONTROL_DOWN, LOCK_SCREEN},
-    {true, ui::VKEY_ESCAPE, ui::EF_SHIFT_DOWN, SHOW_TASK_MANAGER},
-    {false, ui::VKEY_LMENU, ui::EF_SHIFT_DOWN, NEXT_IME},
-    {false, ui::VKEY_MENU, ui::EF_SHIFT_DOWN, NEXT_IME},
-    {false, ui::VKEY_RMENU, ui::EF_SHIFT_DOWN, NEXT_IME},
-    {false, ui::VKEY_LSHIFT, ui::EF_ALT_DOWN, NEXT_IME},
-    {false, ui::VKEY_SHIFT, ui::EF_ALT_DOWN, NEXT_IME},
-    {false, ui::VKEY_RSHIFT, ui::EF_ALT_DOWN, NEXT_IME},
-};
+    {true, ui::VKEY_ESCAPE, ui::EF_SHIFT_DOWN, SHOW_TASK_MANAGER}};
 
 const size_t kDeprecatedAcceleratorsLength = arraysize(kDeprecatedAccelerators);
 
 const DeprecatedAcceleratorData kDeprecatedAcceleratorsData[] = {
     {
         LOCK_SCREEN, "Ash.Accelerators.Deprecated.LockScreen",
         IDS_DEPRECATED_LOCK_SCREEN_MSG, IDS_SHORTCUT_LOCK_SCREEN_OLD,
         IDS_SHORTCUT_LOCK_SCREEN_NEW,
         false  // Old accelerator was disabled in M56.
     },
     {SHOW_TASK_MANAGER, "Ash.Accelerators.Deprecated.ShowTaskManager",
      IDS_DEPRECATED_SHOW_TASK_MANAGER_MSG, IDS_SHORTCUT_TASK_MANAGER_OLD,
-     IDS_SHORTCUT_TASK_MANAGER_NEW, true},
-    {
-        NEXT_IME, "Ash.Accelerators.Deprecated.NextIME",

[James Cook, 2017/01/26 22:24:52]

Does this need to be marked obsolete in histograms.xml?

[afakhry, 2017/01/27 03:35:06]

Done.

-        IDS_DEPRECATED_NEXT_IME_MSG, IDS_SHORTCUT_NEXT_IME_OLD,
-        IDS_SHORTCUT_NEXT_IME_NEW,
-        false  // Old accelerator has been disabled.
-    }};
+     IDS_SHORTCUT_TASK_MANAGER_NEW, true}};
 
 const size_t kDeprecatedAcceleratorsDataLength =
     arraysize(kDeprecatedAcceleratorsData);
 
 #endif  // defined(OS_CHROMEOS)
 
 const AcceleratorData kDebugAcceleratorData[] = {
 #if defined(OS_CHROMEOS)
     {true, ui::VKEY_N, kDebugModifier, TOGGLE_WIFI},
     {true, ui::VKEY_O, kDebugModifier, DEBUG_SHOW_TOAST},
@@ skipping 298 matching lines @@
     TOGGLE_WIFI,
     VOLUME_DOWN,
     VOLUME_MUTE,
     VOLUME_UP,
 #endif  // defined(OS_CHROMEOS)
 };
 
 const size_t kActionsKeepingMenuOpenLength = arraysize(kActionsKeepingMenuOpen);
 
 }  // namespace ash