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

Unified Diff: components/user_manager/user_manager.cc

Issue 2568973002: chromeos: Replace user image notifications with observer (Closed)
Patch Set: rebase Created 4 years 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
« no previous file with comments | « components/user_manager/user_manager.h ('k') | components/user_manager/user_manager_base.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/user_manager/user_manager.cc
diff --git a/components/user_manager/user_manager.cc b/components/user_manager/user_manager.cc
index b0cd217dd43ca798bc8f93f3b8aaf13f2707cf8a..22b33c0ae9196aeecc602c0c2e660a0016f29a54 100644
--- a/components/user_manager/user_manager.cc
+++ b/components/user_manager/user_manager.cc
@@ -9,13 +9,19 @@
namespace user_manager {
-UserManager* UserManager::instance = NULL;
+UserManager* UserManager::instance = nullptr;
-UserManager::Observer::~Observer() {
-}
+UserManager::Observer::~Observer() = default;
-void UserManager::Observer::LocalStateChanged(UserManager* user_manager) {
-}
+void UserManager::Observer::LocalStateChanged(UserManager* user_manager) {}
+
+void UserManager::Observer::OnUserImageChanged(const User& user) {}
+
+void UserManager::Observer::OnUserProfileImageUpdateFailed(const User& user) {}
+
+void UserManager::Observer::OnUserProfileImageUpdated(
+ const User& user,
+ const gfx::ImageSkia& profile_image) {}
void UserManager::UserSessionStateObserver::ActiveUserChanged(
const User* active_user) {
@@ -57,7 +63,7 @@ bool UserManager::IsInitialized() {
void UserManager::Destroy() {
DCHECK(UserManager::instance == this);
- UserManager::SetInstance(NULL);
+ UserManager::SetInstance(nullptr);
}
// static
« no previous file with comments | « components/user_manager/user_manager.h ('k') | components/user_manager/user_manager_base.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698