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

Unified Diff: chrome/browser/chromeos/login/easy_unlock/easy_unlock_reauth.cc

Issue 856493004: Update {virtual,override,final} to follow C++11 style in chrome/browser/chromeos/login. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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/login/easy_unlock/easy_unlock_reauth.cc
diff --git a/chrome/browser/chromeos/login/easy_unlock/easy_unlock_reauth.cc b/chrome/browser/chromeos/login/easy_unlock/easy_unlock_reauth.cc
index f130e7469c2437cba738841f6bed8edd49e9b6f7..1882dcb7dc2ec4d860242497517a9203fb5b1da4 100644
--- a/chrome/browser/chromeos/login/easy_unlock/easy_unlock_reauth.cc
+++ b/chrome/browser/chromeos/login/easy_unlock/easy_unlock_reauth.cc
@@ -32,7 +32,7 @@ class ReauthHandler : public content::NotificationObserver,
explicit ReauthHandler(EasyUnlockReauth::UserContextCallback callback)
: callback_(callback) {}
- virtual ~ReauthHandler() {}
+ ~ReauthHandler() override {}
bool Start() {
ScreenLocker* screen_locker = ScreenLocker::default_screen_locker();
@@ -60,9 +60,9 @@ class ReauthHandler : public content::NotificationObserver,
}
// content::NotificationObserver
- 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 {
CHECK(type == chrome::NOTIFICATION_SCREEN_LOCK_STATE_CHANGED);
bool is_screen_locked = *content::Details<bool>(details).ptr();
DCHECK(is_screen_locked);
@@ -88,15 +88,14 @@ class ReauthHandler : public content::NotificationObserver,
}
// chromeos::AuthStatusConsumer:
- virtual void OnAuthSuccess(
- const chromeos::UserContext& user_context) override {
+ void OnAuthSuccess(const chromeos::UserContext& user_context) override {
callback_.Run(user_context);
// Schedule deletion.
base::MessageLoopForUI::current()->PostTask(FROM_HERE,
base::Bind(&EndReauthAttempt));
}
- virtual void OnAuthFailure(const chromeos::AuthFailure& error) override {}
+ void OnAuthFailure(const chromeos::AuthFailure& error) override {}
private:
content::NotificationRegistrar notification_registrar_;

Powered by Google App Engine
This is Rietveld 408576698