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

Side by Side Diff: components/translate/core/browser/options_menu_model.h

Issue 666133002: Standardize usage of virtual/override/final in components/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 COMPONENTS_TRANSLATE_CORE_BROWSER_OPTIONS_MENU_MODEL_H_ 5 #ifndef COMPONENTS_TRANSLATE_CORE_BROWSER_OPTIONS_MENU_MODEL_H_
6 #define COMPONENTS_TRANSLATE_CORE_BROWSER_OPTIONS_MENU_MODEL_H_ 6 #define COMPONENTS_TRANSLATE_CORE_BROWSER_OPTIONS_MENU_MODEL_H_
7 7
8 #include "ui/base/models/simple_menu_model.h" 8 #include "ui/base/models/simple_menu_model.h"
9 9
10 namespace translate { 10 namespace translate {
11 11
12 class TranslateInfoBarDelegate; 12 class TranslateInfoBarDelegate;
13 13
14 // A menu model that builds the contents of the options menu in the translate 14 // A menu model that builds the contents of the options menu in the translate
15 // infobar. This menu has only one level (no submenus). 15 // infobar. This menu has only one level (no submenus).
16 class OptionsMenuModel : public ui::SimpleMenuModel, 16 class OptionsMenuModel : public ui::SimpleMenuModel,
17 public ui::SimpleMenuModel::Delegate { 17 public ui::SimpleMenuModel::Delegate {
18 public: 18 public:
19 // Command IDs of the items in this menu; exposed for testing. 19 // Command IDs of the items in this menu; exposed for testing.
20 enum CommandID { 20 enum CommandID {
21 ABOUT_TRANSLATE = 0, 21 ABOUT_TRANSLATE = 0,
22 ALWAYS_TRANSLATE, 22 ALWAYS_TRANSLATE,
23 NEVER_TRANSLATE_LANGUAGE, 23 NEVER_TRANSLATE_LANGUAGE,
24 NEVER_TRANSLATE_SITE, 24 NEVER_TRANSLATE_SITE,
25 REPORT_BAD_DETECTION 25 REPORT_BAD_DETECTION
26 }; 26 };
27 27
28 explicit OptionsMenuModel(TranslateInfoBarDelegate* translate_delegate); 28 explicit OptionsMenuModel(TranslateInfoBarDelegate* translate_delegate);
29 virtual ~OptionsMenuModel(); 29 ~OptionsMenuModel() override;
30 30
31 // ui::SimpleMenuModel::Delegate implementation: 31 // ui::SimpleMenuModel::Delegate implementation:
32 virtual bool IsCommandIdChecked(int command_id) const override; 32 bool IsCommandIdChecked(int command_id) const override;
33 virtual bool IsCommandIdEnabled(int command_id) const override; 33 bool IsCommandIdEnabled(int command_id) const override;
34 virtual bool GetAcceleratorForCommandId( 34 bool GetAcceleratorForCommandId(int command_id,
35 int command_id, 35 ui::Accelerator* accelerator) override;
36 ui::Accelerator* accelerator) override; 36 void ExecuteCommand(int command_id, int event_flags) override;
37 virtual void ExecuteCommand(int command_id, int event_flags) override;
38 37
39 private: 38 private:
40 TranslateInfoBarDelegate* translate_infobar_delegate_; 39 TranslateInfoBarDelegate* translate_infobar_delegate_;
41 40
42 DISALLOW_COPY_AND_ASSIGN(OptionsMenuModel); 41 DISALLOW_COPY_AND_ASSIGN(OptionsMenuModel);
43 }; 42 };
44 43
45 } // namespace translate 44 } // namespace translate
46 45
47 #endif // COMPONENTS_TRANSLATE_CORE_BROWSER_OPTIONS_MENU_MODEL_H_ 46 #endif // COMPONENTS_TRANSLATE_CORE_BROWSER_OPTIONS_MENU_MODEL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698