Rebind mus keyboard shortcut for dump ServerWindow hierarchy to Ctrl-Alt-Shift-M

services/ui/ws/window_manager_state.cc

Index: services/ui/ws/window_manager_state.cc
diff --git a/services/ui/ws/window_manager_state.cc b/services/ui/ws/window_manager_state.cc
index 600228dce2b757f73bb0ee80dd0b46d056b7c31b..333156c533f8d066632d464d29708b6f41cadfbe 100644
--- a/services/ui/ws/window_manager_state.cc
+++ b/services/ui/ws/window_manager_state.cc
@@ -367,7 +367,7 @@ void WindowManagerState::AddDebugAccelerators() {
   // Always register the accelerators, even if they only work in debug, so that
   // keyboard behavior is the same in release and debug builds.
   mojom::EventMatcherPtr matcher = CreateKeyMatcher(
-      ui::mojom::KeyboardCode::S, ui::mojom::kEventFlagControlDown |
+      ui::mojom::KeyboardCode::M, ui::mojom::kEventFlagControlDown |

[sky, 2016/08/01 20:43:20]

I think the wm should be able to override debug accelerators, otherwise we get
errors like we're seeing.

I also think debug accelerators should always be processed. So, if the wm binds
an accelerator mus defines both are processed.

                                       ui::mojom::kEventFlagAltDown |
                                       ui::mojom::kEventFlagShiftDown);
   event_dispatcher_.AddAccelerator(kPrintWindowsDebugAcceleratorId,