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

Unified Diff: chrome/browser/sessions/session_restore.cc

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/sessions/session_restore.cc
diff --git a/chrome/browser/sessions/session_restore.cc b/chrome/browser/sessions/session_restore.cc
index bcace9b8a495d5bdd0cf6ffcef406ecb41ee3628..f7f5a1eefff4feb3f8912d4ad044e0430bca4f16 100644
--- a/chrome/browser/sessions/session_restore.cc
+++ b/chrome/browser/sessions/session_restore.cc
@@ -118,7 +118,7 @@ class TabLoader : public content::NotificationObserver,
typedef std::set<RenderWidgetHost*> RenderWidgetHostSet;
explicit TabLoader(base::TimeTicks restore_started);
- virtual ~TabLoader();
+ ~TabLoader() override;
// Loads the next tab. If there are no more tabs to load this deletes itself,
// otherwise |force_load_timer_| is restarted.
@@ -126,12 +126,12 @@ class TabLoader : public content::NotificationObserver,
// NotificationObserver method. Removes the specified tab and loads the next
// tab.
- 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;
// net::NetworkChangeNotifier::ConnectionTypeObserver overrides.
- virtual void OnConnectionTypeChanged(
+ void OnConnectionTypeChanged(
net::NetworkChangeNotifier::ConnectionType type) override;
// Removes the listeners from the specified tab and removes the tab from
@@ -686,7 +686,7 @@ class SessionRestoreImpl : public content::NotificationObserver {
return web_contents;
}
- virtual ~SessionRestoreImpl() {
+ ~SessionRestoreImpl() override {
STLDeleteElements(&windows_);
active_session_restorers->erase(this);
@@ -698,9 +698,9 @@ class SessionRestoreImpl : public content::NotificationObserver {
g_browser_process->ReleaseModule();
}
- 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 {
switch (type) {
case chrome::NOTIFICATION_BROWSER_CLOSED:
delete this;
« no previous file with comments | « chrome/browser/sessions/restore_on_startup_policy_handler.h ('k') | chrome/browser/sessions/session_restore_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698