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

Unified Diff: chrome/browser/chromeos/login/user_manager.h

Issue 15718003: Add SessionStateObserver with ActiveUserChanged() (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: merge & add SessionStateDelegateChromeos Created 7 years, 6 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/user_manager.h
diff --git a/chrome/browser/chromeos/login/user_manager.h b/chrome/browser/chromeos/login/user_manager.h
index 059d888094aef59defc81dde92422b2c9c12e2e3..3a23115d75acddaed47db255fe96cc7cbb2fe84b 100644
--- a/chrome/browser/chromeos/login/user_manager.h
+++ b/chrome/browser/chromeos/login/user_manager.h
@@ -46,15 +46,20 @@ class UserManager {
virtual ~Observer();
};
- // TODO(nkostylev): Merge with session state refactoring CL.
+ // TODO(nkostylev): Refactor and move this observer out of UserManager.
+ // Observer interface that defines methods used to notify on user session /
+ // active user state changes. Default implementation is empty.
class UserSessionStateObserver {
public:
+ // Called when active user has changed.
+ virtual void ActiveUserChanged(const User* active_user);
+
// Called right before notifying on user change so that those who rely
// on user_id hash would be accessing up-to-date value.
- virtual void ActiveUserHashChanged(const std::string& hash) = 0;
+ virtual void ActiveUserHashChanged(const std::string& hash);
// Called when UserManager finishes restoring user sessions after crash.
- virtual void PendingUserSessionsRestoreFinished() = 0;
+ virtual void PendingUserSessionsRestoreFinished();
protected:
virtual ~UserSessionStateObserver();
« no previous file with comments | « chrome/browser/chromeos/login/crash_restore_browsertest.cc ('k') | chrome/browser/chromeos/login/user_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698