Change behaviour of the Alt-] and Alt-[ keys so that it cycles through SnapLeft/SnapRight to DockLe…

ash/accelerators/accelerator_table.cc

Index: ash/accelerators/accelerator_table.cc
diff --git a/ash/accelerators/accelerator_table.cc b/ash/accelerators/accelerator_table.cc
index a4a0c52b83eaad2cd9605c7e4efacde8b9370895..fe607e0873edac1304aea41581a1967f95c8f2a8 100644
--- a/ash/accelerators/accelerator_table.cc
+++ b/ash/accelerators/accelerator_table.cc
@@ -144,8 +144,8 @@ const AcceleratorData kAcceleratorData[] = {
   { true, ui::VKEY_9, ui::EF_ALT_DOWN, LAUNCH_LAST_APP },
 
   // Window management shortcuts.
-  { true, ui::VKEY_OEM_4, ui::EF_ALT_DOWN, WINDOW_SNAP_LEFT },
-  { true, ui::VKEY_OEM_6, ui::EF_ALT_DOWN, WINDOW_SNAP_RIGHT },
+  { true, ui::VKEY_OEM_4, ui::EF_ALT_DOWN, WINDOW_SNAP_OR_DOCK_LEFT },
+  { true, ui::VKEY_OEM_6, ui::EF_ALT_DOWN, WINDOW_SNAP_OR_DOCK_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,
@@ -423,8 +423,8 @@ const AcceleratorAction kActionsNeedingWindow[] = {
     CYCLE_BACKWARD_MRU,
     CYCLE_FORWARD_MRU,
     TOGGLE_OVERVIEW,
-    WINDOW_SNAP_LEFT,
-    WINDOW_SNAP_RIGHT,
+    WINDOW_SNAP_OR_DOCK_LEFT,

[varkha, 2014/09/29 21:04:34]

Commenting out the next few accelerators would allow them to be handled even
when there is no window in workspace such as when all the windows are docked. I
mention this even though this may not be the right thing to do (I suspect that
the right thing would be to allow BuildWindowListInternal and
MruWindowTracker::BuildWindowList do the right thing for docked windows but this
may get you into figuring out what to do about the F5 overview mode).

[dtapuska, 2014/09/30 14:40:49]

Acknowledged.

+    WINDOW_SNAP_OR_DOCK_RIGHT,
     WINDOW_MINIMIZE,
     TOGGLE_FULLSCREEN,
     TOGGLE_MAXIMIZED,