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

Unified Diff: chrome/browser/ui/webui/signin/user_manager_screen_handler.cc

Issue 671653002: Standardize usage of virtual/override/final in chrome/browser/ui/ (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/ui/webui/signin/user_manager_screen_handler.cc
diff --git a/chrome/browser/ui/webui/signin/user_manager_screen_handler.cc b/chrome/browser/ui/webui/signin/user_manager_screen_handler.cc
index 275770330e12635b3c7cc0a43ed567e916151583..4673f304b7f7564887f0ea9af1e5bd2cdf64dc26 100644
--- a/chrome/browser/ui/webui/signin/user_manager_screen_handler.cc
+++ b/chrome/browser/ui/webui/signin/user_manager_screen_handler.cc
@@ -163,7 +163,7 @@ class UserManagerScreenHandler::ProfileUpdateObserver
profile_manager_->GetProfileInfoCache().AddObserver(this);
}
- virtual ~ProfileUpdateObserver() {
+ ~ProfileUpdateObserver() override {
DCHECK(profile_manager_);
profile_manager_->GetProfileInfoCache().RemoveObserver(this);
}
@@ -172,30 +172,27 @@ class UserManagerScreenHandler::ProfileUpdateObserver
// ProfileInfoCacheObserver implementation:
// If any change has been made to a profile, propagate it to all the
// visible user manager screens.
- virtual void OnProfileAdded(const base::FilePath& profile_path) override {
+ void OnProfileAdded(const base::FilePath& profile_path) override {
user_manager_handler_->SendUserList();
}
- virtual void OnProfileWasRemoved(
- const base::FilePath& profile_path,
- const base::string16& profile_name) override {
+ void OnProfileWasRemoved(const base::FilePath& profile_path,
+ const base::string16& profile_name) override {
// TODO(noms): Change 'SendUserList' to 'removeUser' JS-call when
// UserManager is able to find pod belonging to removed user.
user_manager_handler_->SendUserList();
}
- virtual void OnProfileNameChanged(
- const base::FilePath& profile_path,
- const base::string16& old_profile_name) override {
+ void OnProfileNameChanged(const base::FilePath& profile_path,
+ const base::string16& old_profile_name) override {
user_manager_handler_->SendUserList();
}
- virtual void OnProfileAvatarChanged(
- const base::FilePath& profile_path) override {
+ void OnProfileAvatarChanged(const base::FilePath& profile_path) override {
user_manager_handler_->SendUserList();
}
- virtual void OnProfileSigninRequiredChanged(
+ void OnProfileSigninRequiredChanged(
const base::FilePath& profile_path) override {
user_manager_handler_->SendUserList();
}

Powered by Google App Engine
This is Rietveld 408576698