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

Unified Diff: chrome/browser/renderer_context_menu/render_view_context_menu_browsertest_util.h

Issue 648653003: Standardize usage of virtual/override/final in chrome/browser/ (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/renderer_context_menu/render_view_context_menu_browsertest_util.h
diff --git a/chrome/browser/renderer_context_menu/render_view_context_menu_browsertest_util.h b/chrome/browser/renderer_context_menu/render_view_context_menu_browsertest_util.h
index a0803aed9719fbac7636535b0fe79f36e06d6f86..c8cd6e47f69d85c1afc2d829a5da70838338c5b7 100644
--- a/chrome/browser/renderer_context_menu/render_view_context_menu_browsertest_util.h
+++ b/chrome/browser/renderer_context_menu/render_view_context_menu_browsertest_util.h
@@ -17,12 +17,12 @@ class ContextMenuNotificationObserver : public content::NotificationObserver {
public:
// Wait for a context menu to be shown, and then execute |command_to_execute|.
explicit ContextMenuNotificationObserver(int command_to_execute);
- virtual ~ContextMenuNotificationObserver();
+ ~ContextMenuNotificationObserver() override;
private:
- virtual void Observe(int type,
- const content::NotificationSource& source,
- const content::NotificationDetails& details) override;
+ void Observe(int type,
+ const content::NotificationSource& source,
+ const content::NotificationDetails& details) override;
void ExecuteCommand(RenderViewContextMenu* context_menu);
@@ -40,7 +40,7 @@ class SaveLinkAsContextMenuObserver : public ContextMenuNotificationObserver {
// NotificationService::AllSources().
explicit SaveLinkAsContextMenuObserver(
const content::NotificationSource& source);
- virtual ~SaveLinkAsContextMenuObserver();
+ ~SaveLinkAsContextMenuObserver() override;
// Suggested filename for file downloaded through "Save Link As" option.
base::string16 GetSuggestedFilename();
@@ -49,9 +49,9 @@ class SaveLinkAsContextMenuObserver : public ContextMenuNotificationObserver {
void WaitForMenu();
private:
- virtual void Observe(int type,
- const content::NotificationSource& source,
- const content::NotificationDetails& details) override;
+ void Observe(int type,
+ const content::NotificationSource& source,
+ const content::NotificationDetails& details) override;
void Cancel(RenderViewContextMenu* context_menu);

Powered by Google App Engine
This is Rietveld 408576698