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

Unified Diff: chrome/browser/notifications/message_center_notification_manager.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/message_center_notification_manager.h
diff --git a/chrome/browser/notifications/message_center_notification_manager.h b/chrome/browser/notifications/message_center_notification_manager.h
index e3c5929c12afb9aa69a82e2de152fb725fd0f2e8..443c89bab4205c68f5bb81923a40f24d1e4030b8 100644
--- a/chrome/browser/notifications/message_center_notification_manager.h
+++ b/chrome/browser/notifications/message_center_notification_manager.h
@@ -45,33 +45,30 @@ class MessageCenterNotificationManager
message_center::MessageCenter* message_center,
PrefService* local_state,
scoped_ptr<message_center::NotifierSettingsProvider> settings_provider);
- virtual ~MessageCenterNotificationManager();
+ ~MessageCenterNotificationManager() override;
// Registers preferences.
static void RegisterPrefs(PrefRegistrySimple* registry);
// NotificationUIManager
- virtual void Add(const Notification& notification,
- Profile* profile) override;
- virtual bool Update(const Notification& notification,
- Profile* profile) override;
- virtual const Notification* FindById(const std::string& delegate_id,
- ProfileID profile_id) const override;
- virtual bool CancelById(const std::string& delegate_id,
- ProfileID profile_id) override;
- virtual std::set<std::string> GetAllIdsByProfileAndSourceOrigin(
+ void Add(const Notification& notification, Profile* profile) override;
+ bool Update(const Notification& notification, Profile* profile) override;
+ const Notification* FindById(const std::string& delegate_id,
+ ProfileID profile_id) const override;
+ bool CancelById(const std::string& delegate_id,
+ ProfileID profile_id) override;
+ std::set<std::string> GetAllIdsByProfileAndSourceOrigin(
Profile* profile,
const GURL& source) override;
- virtual bool CancelAllBySourceOrigin(const GURL& source_origin) override;
- virtual bool CancelAllByProfile(ProfileID profile_id) override;
- virtual void CancelAll() override;
+ bool CancelAllBySourceOrigin(const GURL& source_origin) override;
+ bool CancelAllByProfile(ProfileID profile_id) override;
+ void CancelAll() override;
// MessageCenterObserver
- virtual void OnNotificationRemoved(const std::string& notification_id,
- bool by_user) override;
- virtual void OnCenterVisibilityChanged(message_center::Visibility) override;
- virtual void OnNotificationUpdated(const std::string& notification_id)
- override;
+ void OnNotificationRemoved(const std::string& notification_id,
+ bool by_user) override;
+ void OnCenterVisibilityChanged(message_center::Visibility) override;
+ void OnNotificationUpdated(const std::string& notification_id) override;
void EnsureMessageCenterClosed();

Powered by Google App Engine
This is Rietveld 408576698