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

Unified Diff: chrome/test/base/menu_model_test.h

Issue 2138353002: SimpleMenuModel and ButtonMenuItemModel implement AcceleratorProvider. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@menumodel-accelerator-default
Patch Set: Fix compile on Windows. Created 4 years, 5 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
« no previous file with comments | « no previous file | chrome/test/base/menu_model_test.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/base/menu_model_test.h
diff --git a/chrome/test/base/menu_model_test.h b/chrome/test/base/menu_model_test.h
index f321f4684acda7a3c21b50eb7a6f8a9a844a2926..fd435632f7852e565c7124e1bd5deb7725aad1b0 100644
--- a/chrome/test/base/menu_model_test.h
+++ b/chrome/test/base/menu_model_test.h
@@ -18,16 +18,12 @@ class MenuModelTest {
protected:
// A menu delegate that counts the number of times certain things are called
// to make sure things are hooked up properly.
- class Delegate : public ui::SimpleMenuModel::Delegate,
- public ui::AcceleratorProvider {
+ class Delegate : public ui::SimpleMenuModel::Delegate {
public:
Delegate() : execute_count_(0), enable_count_(0) {}
bool IsCommandIdChecked(int command_id) const override;
bool IsCommandIdEnabled(int command_id) const override;
- bool GetAcceleratorForCommandId(
- int command_id,
- ui::Accelerator* accelerator) const override;
void ExecuteCommand(int command_id, int event_flags) override;
int execute_count_;
« no previous file with comments | « no previous file | chrome/test/base/menu_model_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698