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

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

Issue 14756019: Adding new user menu section to the SystemTrayMenu & refactoring of user access (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: More windows breakages addressed Created 7 years, 7 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_impl.h
diff --git a/chrome/browser/chromeos/login/user_manager_impl.h b/chrome/browser/chromeos/login/user_manager_impl.h
index 1eea1e32dc32cda857f43b24a848d4bbe91d337b..5b220bb3b605dd23b0c51e910850a54f4ad7821f 100644
--- a/chrome/browser/chromeos/login/user_manager_impl.h
+++ b/chrome/browser/chromeos/login/user_manager_impl.h
@@ -48,6 +48,7 @@ class UserManagerImpl
virtual UserImageManager* GetUserImageManager() OVERRIDE;
virtual const UserList& GetUsers() const OVERRIDE;
virtual const UserList& GetLoggedInUsers() const OVERRIDE;
+ virtual const UserList& GetLRULoggedInUsers() OVERRIDE;
virtual void UserLoggedIn(const std::string& email,
const std::string& username_hash,
bool browser_restart) OVERRIDE;
@@ -259,6 +260,9 @@ class UserManagerImpl
// Gets the list of public accounts defined in device settings.
void ReadPublicAccounts(base::ListValue* public_accounts);
+ // Insert |user| at the front of the LRU user list..
+ void SetLRUUser(User* user);
+
// Interface to the signed settings store.
CrosSettings* cros_settings_;
@@ -277,6 +281,14 @@ class UserManagerImpl
// instances in |users_|. Only one of them could be marked as active.
UserList logged_in_users_;
+ // A list of all users that are logged in the current session. In contrast to
+ // |logged_in_users|, the order of this list is least recently used so that
+ // the active user should always be the first one in the list.
+ UserList lru_logged_in_users_;
+
+ // The list which gets reported when the |lru_logged_in_users_| list is empty.
+ UserList temp_single_logged_in_users_;
+
// The logged-in user that is currently active in current session.
// NULL until a user has logged in, then points to one
// of the User instances in |users_|, the |guest_user_| instance or an

Powered by Google App Engine
This is Rietveld 408576698