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

Unified Diff: ui/wm/core/accelerator_filter.h

Issue 727583002: Regression: Search+Key pops up app launcher (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 1 month 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
« no previous file with comments | « ui/views/focus/focus_manager.cc ('k') | ui/wm/core/accelerator_filter.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/wm/core/accelerator_filter.h
diff --git a/ui/wm/core/accelerator_filter.h b/ui/wm/core/accelerator_filter.h
index 44579efa04348c9683404de9ff779ca9e200e53d..79317e11eb1d2cb775d6e7586515026d0690f1ac 100644
--- a/ui/wm/core/accelerator_filter.h
+++ b/ui/wm/core/accelerator_filter.h
@@ -12,6 +12,7 @@
namespace ui {
class Accelerator;
+class AcceleratorHistory;
}
namespace wm {
@@ -21,7 +22,10 @@ class AcceleratorDelegate;
// keyboard accelerators.
class WM_EXPORT AcceleratorFilter : public ui::EventHandler {
public:
- AcceleratorFilter(scoped_ptr<AcceleratorDelegate> delegate);
+ // AcceleratorFilter doesn't own |accelerator_history|, it's owned by
+ // AcceleratorController.
+ AcceleratorFilter(scoped_ptr<AcceleratorDelegate> delegate,
+ ui::AcceleratorHistory* accelerator_history);
~AcceleratorFilter() override;
// Overridden from ui::EventHandler:
@@ -29,6 +33,7 @@ class WM_EXPORT AcceleratorFilter : public ui::EventHandler {
private:
scoped_ptr<AcceleratorDelegate> delegate_;
+ ui::AcceleratorHistory* accelerator_history_;
DISALLOW_COPY_AND_ASSIGN(AcceleratorFilter);
};
« no previous file with comments | « ui/views/focus/focus_manager.cc ('k') | ui/wm/core/accelerator_filter.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698