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

Unified Diff: chrome/browser/notifications/desktop_notification_service.h

Issue 667283002: Standardize usage of virtual/override/final in chrome/browser/notifications (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/notifications/desktop_notification_service.h
diff --git a/chrome/browser/notifications/desktop_notification_service.h b/chrome/browser/notifications/desktop_notification_service.h
index f337c9fbfd0323dfa5b89b1d2111d5bfbf0da076..e323d1a53067e8675b74d1793da64a44ebc86df5 100644
--- a/chrome/browser/notifications/desktop_notification_service.h
+++ b/chrome/browser/notifications/desktop_notification_service.h
@@ -80,7 +80,7 @@ class DesktopNotificationService : public PermissionContextBase
Profile* profile);
explicit DesktopNotificationService(Profile* profile);
- virtual ~DesktopNotificationService();
+ ~DesktopNotificationService() override;
// Requests Web Notification permission for |requesting_frame|. The |callback|
// will be invoked after the user has made a decision.
@@ -133,16 +133,15 @@ class DesktopNotificationService : public PermissionContextBase
#if defined(ENABLE_EXTENSIONS)
// extensions::ExtensionRegistryObserver:
- virtual void OnExtensionUninstalled(
- content::BrowserContext* browser_context,
- const extensions::Extension* extension,
- extensions::UninstallReason reason) override;
+ void OnExtensionUninstalled(content::BrowserContext* browser_context,
+ const extensions::Extension* extension,
+ extensions::UninstallReason reason) override;
#endif
// PermissionContextBase:
- virtual void UpdateContentSetting(const GURL& requesting_origin,
- const GURL& embedder_origin,
- bool allowed) override;
+ void UpdateContentSetting(const GURL& requesting_origin,
+ const GURL& embedder_origin,
+ bool allowed) override;
// The profile which owns this object.
Profile* profile_;

Powered by Google App Engine
This is Rietveld 408576698