Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(98)

Unified Diff: ui/views/controls/menu/menu_event_filter.h

Issue 1138523006: Enable keyboard accelerators while a menu is open (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ui/views/controls/menu/menu_event_filter.h
diff --git a/ui/views/controls/menu/menu_event_filter.h b/ui/views/controls/menu/menu_event_filter.h
new file mode 100644
index 0000000000000000000000000000000000000000..f7e621861be6d678f6e7220f50c5588c07154a2a
--- /dev/null
+++ b/ui/views/controls/menu/menu_event_filter.h
@@ -0,0 +1,73 @@
+// 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_FILTER_H_
+#define UI_VIEWS_CONTROLS_MENU_MENU_EVENT_FILTER_H_
+
+#include "base/logging.h"
+#include "base/macros.h"
+#include "base/memory/scoped_ptr.h"
+#include "ui/events/event_handler.h"
+#include "ui/views/views_export.h"
+
+namespace aura {
+class Window;
+}
+
+namespace ui {
+class Accelerator;
+}
+
+namespace views {
+
+class MenuController;
+
+// Handles events while a menu is open.
+class VIEWS_EXPORT MenuEventFilter : public ui::EventHandler {
+ public:
+ // Defines a delegate interface for the MenuEventFilter that can handle
+ // accelerators on its behalf.
+ class Delegate {
+ public:
+ Delegate() {}
+ virtual ~Delegate() {}
+
+ // Stores |accelerator| in ui::AcceleratorHistory.
+ virtual void StoreInHistory(const ui::Accelerator& accelerator) = 0;
+
+ // Returns true if there is an action associated with |accelerator| and
+ // the menu should be closed before performing the action.
+ virtual bool ShouldCloseMenuAndRepostAccelerator(
+ const ui::Accelerator& accelerator) const = 0;
+
+ // Processes |accelerator| immediately.
+ virtual void ProcessAcceleratorNow(const ui::Accelerator& accelerator) = 0;
+ };
+
+ // |menu_controller| should be of the active MenuController, it can't be
+ // nullptr. If |delegate| is nullptr, the default delegate will be used.
+ MenuEventFilter(MenuController* menu_controller,
+ MenuEventFilter::Delegate* delegate);
+ ~MenuEventFilter() override;
+
+ static void SetMenuEventFilterDelegate(aura::Window* root_window,
+ MenuEventFilter::Delegate* delegate);
+ static MenuEventFilter::Delegate* GetMenuEventFilterDelegate(
+ aura::Window* root_window);
+
+ // ui::EventHandler:
+ void OnKeyEvent(ui::KeyEvent* event) override;
+ void OnTouchEvent(ui::TouchEvent* event) override;
+
+ private:
+ MenuController* menu_controller_; // Not owned.
+ scoped_ptr<MenuEventFilter::Delegate> default_delegate_;
+ MenuEventFilter::Delegate* filter_delegate_; // Not owned.
+
+ DISALLOW_COPY_AND_ASSIGN(MenuEventFilter);
+};
+
+} // namespace views
+
+#endif // UI_VIEWS_CONTROLS_MENU_MENU_EVENT_FILTER_H_

Powered by Google App Engine
This is Rietveld 408576698