Enable keyboard accelerators while a menu is open

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 a7f48e7d7b0116e29422230682df4d830209cdc4..aeb86730d0975abc0d890543ccd8073aabbab53e 100644
--- a/ash/accelerators/accelerator_table.cc
+++ b/ash/accelerators/accelerator_table.cc
@@ -472,4 +472,33 @@ const AcceleratorAction kActionsNeedingWindow[] = {
 
 const size_t kActionsNeedingWindowLength = arraysize(kActionsNeedingWindow);
 
+const AcceleratorAction kActionsKeepingMenuOpen[] = {
+    MEDIA_NEXT_TRACK,
+    MEDIA_PLAY_PAUSE,
+    MEDIA_PREV_TRACK,
+    NEXT_IME,
+    PREVIOUS_IME,
+    PRINT_UI_HIERARCHIES,
+    SWITCH_IME,
+    TAKE_PARTIAL_SCREENSHOT,
+    TAKE_SCREENSHOT,
+#if defined(OS_CHROMEOS)
+    BRIGHTNESS_DOWN,
+    BRIGHTNESS_UP,
+    DISABLE_CAPS_LOCK,
+    DISABLE_GPU_WATCHDOG,

[pkotwicz, 2015/09/10 19:13:00]

Did you get UX feedback on the accelerators which should be on this list?
DISABLE_GPU_WATCHDOG is a weird accelerator to have in the list.

[afakhry, 2015/09/10 20:57:13]

I just spoke to Albert, we can definitely take UX feedback, however, we think
that we can proceed without it for now, especially that the list is short and
adding or removing stuff from it later is not a complicated thing. 
I agree about the DISABLE_GPU_WATCHDOG being weird. I'm not sure why it ended up
being on this list. I removed it. Thanks for pointing it out!

+    KEYBOARD_BRIGHTNESS_DOWN,
+    KEYBOARD_BRIGHTNESS_UP,
+    SILENCE_SPOKEN_FEEDBACK,
+    TOGGLE_CAPS_LOCK,
+    TOGGLE_SPOKEN_FEEDBACK,
+    TOGGLE_WIFI,
+    VOLUME_DOWN,
+    VOLUME_MUTE,
+    VOLUME_UP,
+#endif  // defined(OS_CHROMEOS)
+};
+
+const size_t kActionsKeepingMenuOpenLength = arraysize(kActionsKeepingMenuOpen);
+
 }  // namespace ash