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

Side by Side Diff: chrome/browser/ui/views/toolbar/app_menu.h

Issue 2790773002: Cleanup MenuRunner API (Closed)
Patch Set: Rebase Created 3 years, 8 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_UI_VIEWS_TOOLBAR_APP_MENU_H_ 5 #ifndef CHROME_BROWSER_UI_VIEWS_TOOLBAR_APP_MENU_H_
6 #define CHROME_BROWSER_UI_VIEWS_TOOLBAR_APP_MENU_H_ 6 #define CHROME_BROWSER_UI_VIEWS_TOOLBAR_APP_MENU_H_
7 7
8 #include <map> 8 #include <map>
9 #include <memory> 9 #include <memory>
10 #include <utility> 10 #include <utility>
(...skipping 78 matching lines...) Expand 10 before | Expand all | Expand 10 after
89 int GetDragOperations(views::MenuItemView* sender) override; 89 int GetDragOperations(views::MenuItemView* sender) override;
90 int GetMaxWidthForMenu(views::MenuItemView* menu) override; 90 int GetMaxWidthForMenu(views::MenuItemView* menu) override;
91 bool IsItemChecked(int command_id) const override; 91 bool IsItemChecked(int command_id) const override;
92 bool IsCommandEnabled(int command_id) const override; 92 bool IsCommandEnabled(int command_id) const override;
93 void ExecuteCommand(int command_id, int mouse_event_flags) override; 93 void ExecuteCommand(int command_id, int mouse_event_flags) override;
94 bool GetAccelerator(int command_id, 94 bool GetAccelerator(int command_id,
95 ui::Accelerator* accelerator) const override; 95 ui::Accelerator* accelerator) const override;
96 void WillShowMenu(views::MenuItemView* menu) override; 96 void WillShowMenu(views::MenuItemView* menu) override;
97 void WillHideMenu(views::MenuItemView* menu) override; 97 void WillHideMenu(views::MenuItemView* menu) override;
98 bool ShouldCloseOnDragComplete() override; 98 bool ShouldCloseOnDragComplete() override;
99 void OnMenuClosed(views::MenuItemView* menu, 99 void OnMenuClosed(views::MenuItemView* menu) override;
100 views::MenuRunner::RunResult result) override;
101 bool ShouldExecuteCommandWithoutClosingMenu(int command_id, 100 bool ShouldExecuteCommandWithoutClosingMenu(int command_id,
102 const ui::Event& event) override; 101 const ui::Event& event) override;
103 102
104 // bookmarks::BaseBookmarkModelObserver overrides: 103 // bookmarks::BaseBookmarkModelObserver overrides:
105 void BookmarkModelChanged() override; 104 void BookmarkModelChanged() override;
106 105
107 // content::NotificationObserver overrides: 106 // content::NotificationObserver overrides:
108 void Observe(int type, 107 void Observe(int type,
109 const content::NotificationSource& source, 108 const content::NotificationSource& source,
110 const content::NotificationDetails& details) override; 109 const content::NotificationDetails& details) override;
(...skipping 85 matching lines...) Expand 10 before | Expand all | Expand 10 after
196 195
197 base::ObserverList<AppMenuObserver> observer_list_; 196 base::ObserverList<AppMenuObserver> observer_list_;
198 197
199 // Records the time from when menu opens to when the user selects a menu item. 198 // Records the time from when menu opens to when the user selects a menu item.
200 base::ElapsedTimer menu_opened_timer_; 199 base::ElapsedTimer menu_opened_timer_;
201 200
202 DISALLOW_COPY_AND_ASSIGN(AppMenu); 201 DISALLOW_COPY_AND_ASSIGN(AppMenu);
203 }; 202 };
204 203
205 #endif // CHROME_BROWSER_UI_VIEWS_TOOLBAR_APP_MENU_H_ 204 #endif // CHROME_BROWSER_UI_VIEWS_TOOLBAR_APP_MENU_H_
OLDNEW
« no previous file with comments | « chrome/browser/ui/views/task_manager_view.cc ('k') | chrome/browser/ui/views/toolbar/app_menu.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698