Adding new user menu section to the SystemTrayMenu & refactoring of user access

chrome/browser/chromeos/login/user_manager_impl.h

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 6c04dabd2cf971c21cfc71b5dfe88e7437c985e9..4cf7adf97b7352c4c5abf6838fa4b8d8486bb96e 100644
--- a/chrome/browser/chromeos/login/user_manager_impl.h
+++ b/chrome/browser/chromeos/login/user_manager_impl.h
@@ -47,6 +47,7 @@ class UserManagerImpl
   virtual UserImageManager* GetUserImageManager() OVERRIDE;
   virtual const UserList& GetUsers() const OVERRIDE;
   virtual const UserList& GetLoggedInUsers() const OVERRIDE;
+  virtual const UserList& GetLRULoggedInUsers() OVERRIDE;

[Nikita (slow), 2013/05/17 17:00:32]

Please add test for UserManager.

[Mr4D (OOO till 08-26), 2013/05/17 18:45:09]

As stated in my issue header: That will come with the next CL (which will add
the one missing UI element and the unit tests for UI and this).

   virtual void UserLoggedIn(const std::string& email,
                             const std::string& username_hash,
                             bool browser_restart) OVERRIDE;
@@ -251,6 +252,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_;
 
@@ -269,6 +273,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