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

Unified Diff: chrome/browser/ui/cocoa/renderer_context_menu/render_view_context_menu_mac.mm

Issue 671653002: Standardize usage of virtual/override/final in chrome/browser/ui/ (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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/cocoa/renderer_context_menu/render_view_context_menu_mac.mm
diff --git a/chrome/browser/ui/cocoa/renderer_context_menu/render_view_context_menu_mac.mm b/chrome/browser/ui/cocoa/renderer_context_menu/render_view_context_menu_mac.mm
index de96f73a08aca81cce340e63da3c3b95ebe19fe9..033c0addd3520806ef32bbabd7413ec64ee17f55 100644
--- a/chrome/browser/ui/cocoa/renderer_context_menu/render_view_context_menu_mac.mm
+++ b/chrome/browser/ui/cocoa/renderer_context_menu/render_view_context_menu_mac.mm
@@ -53,20 +53,18 @@ class ToolkitDelegateMac : public RenderViewContextMenuBase::ToolkitDelegate {
public:
explicit ToolkitDelegateMac(RenderViewContextMenuMac* context_menu)
: context_menu_(context_menu) {}
- virtual ~ToolkitDelegateMac() {}
+ ~ToolkitDelegateMac() override {}
private:
// ToolkitDelegate:
- virtual void Init(ui::SimpleMenuModel* menu_model) override {
+ void Init(ui::SimpleMenuModel* menu_model) override {
context_menu_->InitToolkitMenu();
}
- virtual void Cancel() override{
- context_menu_->CancelToolkitMenu();
- }
- virtual void UpdateMenuItem(int command_id,
- bool enabled,
- bool hidden,
- const base::string16& title) override {
+ void Cancel() override { context_menu_->CancelToolkitMenu(); }
+ void UpdateMenuItem(int command_id,
+ bool enabled,
+ bool hidden,
+ const base::string16& title) override {
context_menu_->UpdateToolkitMenuItem(
command_id, enabled, hidden, title);
}

Powered by Google App Engine
This is Rietveld 408576698