Enable keyboard accelerators while a menu is open

ui/views/controls/menu/menu_event_handler.h

+// Copyright 2015 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.
+
+#ifndef UI_VIEWS_CONTROLS_MENU_MENU_EVENT_HANDLER_H_
+#define UI_VIEWS_CONTROLS_MENU_MENU_EVENT_HANDLER_H_
+
+#include "base/macros.h"
+#include "ui/events/event_handler.h"
+#include "ui/views/views_export.h"
+
+namespace ui {
+class Accelerator;
+}
+
+namespace views {
+
+// Handles events while a menu is open.
+class VIEWS_EXPORT MenuEventHandler : public ui::EventHandler {

[sky, 2015/09/09 00:01:59]

Why does this code need to be in views? Isn't it ash specific? That's the whole
reason I wanted a function on ViewsDelegate.

[oshima, 2015/09/09 00:11:35]

This handles mnemonics. The mnemonics is a part of view's menu functionality, so
it should be in views, isn't it?

[afakhry, 2015/09/09 00:16:00]

Yes, as oshima mentioned, this has to handle mnemonics (by calling
MenuController::SelectByChar()) as well as menu key handling like navigation
using arrows and Esc to close and so on (by calling
MenuController::OnKeyDown()). These are not ash specific but for anything using
views menus. This handler should replace MenuEventDispatcher.

+ public:
+  MenuEventHandler();
+  ~MenuEventHandler() override;
+
+  // ui::EventHandler:
+  void OnKeyEvent(ui::KeyEvent* event) override;
+  void OnTouchEvent(ui::TouchEvent* event) override;
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(MenuEventHandler);
+};
+
+}  // namespace views
+
+#endif  // UI_VIEWS_CONTROLS_MENU_MENU_EVENT_HANDLER_H_