Remove docked windows entirely in M59.

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 128 matching lines @@
     {true, ui::VKEY_2, ui::EF_ALT_DOWN, LAUNCH_APP_1},
     {true, ui::VKEY_3, ui::EF_ALT_DOWN, LAUNCH_APP_2},
     {true, ui::VKEY_4, ui::EF_ALT_DOWN, LAUNCH_APP_3},
     {true, ui::VKEY_5, ui::EF_ALT_DOWN, LAUNCH_APP_4},
     {true, ui::VKEY_6, ui::EF_ALT_DOWN, LAUNCH_APP_5},
     {true, ui::VKEY_7, ui::EF_ALT_DOWN, LAUNCH_APP_6},
     {true, ui::VKEY_8, ui::EF_ALT_DOWN, LAUNCH_APP_7},
     {true, ui::VKEY_9, ui::EF_ALT_DOWN, LAUNCH_LAST_APP},
 
     // Window management shortcuts.
-    {true, ui::VKEY_OEM_4, ui::EF_ALT_DOWN, WINDOW_CYCLE_SNAP_DOCK_LEFT},
-    {true, ui::VKEY_OEM_6, ui::EF_ALT_DOWN, WINDOW_CYCLE_SNAP_DOCK_RIGHT},
+    {true, ui::VKEY_OEM_4, ui::EF_ALT_DOWN, WINDOW_CYCLE_SNAP_LEFT},
+    {true, ui::VKEY_OEM_6, ui::EF_ALT_DOWN, WINDOW_CYCLE_SNAP_RIGHT},
     {true, ui::VKEY_OEM_MINUS, ui::EF_ALT_DOWN, WINDOW_MINIMIZE},
     {true, ui::VKEY_OEM_PLUS, ui::EF_ALT_DOWN, TOGGLE_MAXIMIZED},
     {true, ui::VKEY_OEM_PLUS, ui::EF_SHIFT_DOWN | ui::EF_ALT_DOWN,
      WINDOW_POSITION_CENTER},
     {true, ui::VKEY_BROWSER_FORWARD, ui::EF_CONTROL_DOWN, FOCUS_NEXT_PANE},
     {true, ui::VKEY_BROWSER_BACK, ui::EF_CONTROL_DOWN, FOCUS_PREVIOUS_PANE},
 
     // Media Player shortcuts.
     {true, ui::VKEY_MEDIA_NEXT_TRACK, ui::EF_NONE, MEDIA_NEXT_TRACK},
     {true, ui::VKEY_MEDIA_PLAY_PAUSE, ui::EF_NONE, MEDIA_PLAY_PAUSE},
@@ skipping 288 matching lines @@
     TAKE_PARTIAL_SCREENSHOT,
     TAKE_SCREENSHOT,
     TAKE_WINDOW_SCREENSHOT,
     UNPIN,
 };
 
 const size_t kActionsAllowedInPinnedModeLength =
     arraysize(kActionsAllowedInPinnedMode);
 
 const AcceleratorAction kActionsNeedingWindow[] = {
-    CYCLE_BACKWARD_MRU,
-    CYCLE_FORWARD_MRU,
-    TOGGLE_OVERVIEW,
-    WINDOW_CYCLE_SNAP_DOCK_LEFT,
-    WINDOW_CYCLE_SNAP_DOCK_RIGHT,
-    WINDOW_MINIMIZE,
-    TOGGLE_FULLSCREEN,
-    TOGGLE_MAXIMIZED,
-    WINDOW_POSITION_CENTER,
+    CYCLE_BACKWARD_MRU,     CYCLE_FORWARD_MRU,       TOGGLE_OVERVIEW,
+    WINDOW_CYCLE_SNAP_LEFT, WINDOW_CYCLE_SNAP_RIGHT, WINDOW_MINIMIZE,
+    TOGGLE_FULLSCREEN,      TOGGLE_MAXIMIZED,        WINDOW_POSITION_CENTER,

[oshima, 2017/02/22 06:49:07]

I think we still want to keep the orignal format.

[afakhry, 2017/02/22 22:04:43]

Done.

     ROTATE_WINDOW,
 };
 
 const size_t kActionsNeedingWindowLength = arraysize(kActionsNeedingWindow);
 
 const AcceleratorAction kActionsKeepingMenuOpen[] = {
     BRIGHTNESS_DOWN,
     BRIGHTNESS_UP,
     DEBUG_TOGGLE_TOUCH_PAD,
     DEBUG_TOGGLE_TOUCH_SCREEN,
@@ skipping 16 matching lines @@
     TOGGLE_SPOKEN_FEEDBACK,
     TOGGLE_WIFI,
     VOLUME_DOWN,
     VOLUME_MUTE,
     VOLUME_UP,
 };
 
 const size_t kActionsKeepingMenuOpenLength = arraysize(kActionsKeepingMenuOpen);
 
 }  // namespace ash