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

Unified Diff: chrome/browser/chromeos/app_mode/app_session_lifetime.cc

Issue 827293003: Update {virtual,override,final} to follow C++11 style in chrome/browser/chromeos/app_mode. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address comment Created 5 years, 11 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/chromeos/app_mode/app_session_lifetime.cc
diff --git a/chrome/browser/chromeos/app_mode/app_session_lifetime.cc b/chrome/browser/chromeos/app_mode/app_session_lifetime.cc
index f9b019874d1ed4823b1c12e4459f04f097acf9f9..fe40288df2535d8f0a1a2dd87fb364303f7a0de0 100644
--- a/chrome/browser/chromeos/app_mode/app_session_lifetime.cc
+++ b/chrome/browser/chromeos/app_mode/app_session_lifetime.cc
@@ -42,7 +42,7 @@ namespace {
class AppWindowHandler : public AppWindowRegistry::Observer {
public:
AppWindowHandler() : window_registry_(NULL) {}
- virtual ~AppWindowHandler() {}
+ ~AppWindowHandler() override {}
void Init(Profile* profile) {
DCHECK(!window_registry_);
@@ -53,7 +53,7 @@ class AppWindowHandler : public AppWindowRegistry::Observer {
private:
// extensions::AppWindowRegistry::Observer overrides:
- virtual void OnAppWindowRemoved(AppWindow* app_window) override {
+ void OnAppWindowRemoved(AppWindow* app_window) override {
if (window_registry_->app_windows().empty()) {
if (DemoAppLauncher::IsDemoAppSession(
user_manager::UserManager::Get()->GetActiveUser()->email())) {
@@ -87,9 +87,7 @@ class BrowserWindowHandler : public chrome::BrowserListObserver {
BrowserWindowHandler() {
BrowserList::AddObserver(this);
}
- virtual ~BrowserWindowHandler() {
- BrowserList::RemoveObserver(this);
- }
+ ~BrowserWindowHandler() override { BrowserList::RemoveObserver(this); }
private:
void HandleBrowser(Browser* browser) {
@@ -104,7 +102,7 @@ class BrowserWindowHandler : public chrome::BrowserListObserver {
}
// chrome::BrowserListObserver overrides:
- virtual void OnBrowserAdded(Browser* browser) override {
+ void OnBrowserAdded(Browser* browser) override {
base::MessageLoop::current()->PostTask(
FROM_HERE,
base::Bind(&BrowserWindowHandler::HandleBrowser,
« no previous file with comments | « chrome/browser/chromeos/app_mode/app_launch_utils.cc ('k') | chrome/browser/chromeos/app_mode/certificate_manager_dialog.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698