Add public accounts to UserManager

chrome/browser/chromeos/login/user_manager_impl.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_CHROMEOS_LOGIN_USER_MANAGER_IMPL_H_
 #define CHROME_BROWSER_CHROMEOS_LOGIN_USER_MANAGER_IMPL_H_
 
+#include <set>
 #include <string>
+#include <vector>
 
 #include "base/basictypes.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/singleton.h"
+#include "base/memory/weak_ptr.h"
 #include "base/observer_list.h"
 #include "base/synchronization/lock.h"
+#include "base/values.h"
 #include "chrome/browser/api/sync/profile_sync_service_observer.h"
 #include "chrome/browser/chromeos/login/user.h"
 #include "chrome/browser/chromeos/login/user_image_manager_impl.h"
 #include "chrome/browser/chromeos/login/user_manager.h"
 #include "chrome/browser/chromeos/login/wallpaper_manager.h"
+#include "chrome/browser/chromeos/settings/cros_settings.h"
 #include "chrome/browser/chromeos/settings/device_settings_service.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
 
 class FilePath;
 class PrefService;
 class ProfileSyncService;
 
 namespace chromeos {
 
@@ skipping 31 matching lines @@
       const std::string& username) const OVERRIDE;
   virtual void SaveUserDisplayEmail(const std::string& username,
                                     const std::string& display_email) OVERRIDE;
   virtual std::string GetUserDisplayEmail(
       const std::string& username) const OVERRIDE;
   virtual bool IsCurrentUserOwner() const OVERRIDE;
   virtual bool IsCurrentUserNew() const OVERRIDE;
   virtual bool IsCurrentUserEphemeral() const OVERRIDE;
   virtual bool CanCurrentUserLock() const OVERRIDE;
   virtual bool IsUserLoggedIn() const OVERRIDE;
+  virtual bool IsLoggedInAsRegularUser() const OVERRIDE;
   virtual bool IsLoggedInAsDemoUser() const OVERRIDE;
   virtual bool IsLoggedInAsPublicAccount() const OVERRIDE;
   virtual bool IsLoggedInAsGuest() const OVERRIDE;
   virtual bool IsLoggedInAsStub() const OVERRIDE;
   virtual bool IsSessionStarted() const OVERRIDE;
   virtual bool IsEphemeralUser(const std::string& email) const OVERRIDE;
   virtual void AddObserver(UserManager::Observer* obs) OVERRIDE;
   virtual void RemoveObserver(UserManager::Observer* obs) OVERRIDE;
   virtual void NotifyLocalStateChanged() OVERRIDE;
 
   // content::NotificationObserver implementation.
   virtual void Observe(int type,
                        const content::NotificationSource& source,
                        const content::NotificationDetails& details) OVERRIDE;
 
   // ProfileSyncServiceObserver implementation.
   virtual void OnStateChanged() OVERRIDE;
 
  private:
   friend class UserManagerImplWrapper;
   friend class WallpaperManager;
   friend class UserManagerTest;
 
   UserManagerImpl();
 
+  // Helper function that copies users from |users_list| to |users_vector| and
+  // |users_set|. Duplicates and users already present in |existing_users| are
+  // skipped. The |logged_in_user| is also skipped and the return value
+  // indicates whether that user was found in |users_list|.
+  bool ParseUserList(std::vector<std::string>* users_vector,
+                     std::set<std::string>* users_set,
+                     const ListValue& users_list,

[pastarmovj, 2012/11/28 09:57:16]

nit: move |users_vector| and |users_set| to be last arguments. It seems that
there is weak common preference to have src,dst argument ordering.

[bartfab (slow), 2012/11/28 12:30:17]

Done.

+                     const std::set<std::string>& existing_users,
+                     const std::string& logged_in_user) const;
+
   // Loads |users_| from Local State if the list has not been loaded yet.
   // Subsequent calls have no effect. Must be called on the UI thread.
   void EnsureUsersLoaded();
 
   // Retrieves trusted device policies and removes users from the persistent
   // list if ephemeral users are enabled. Schedules a callback to itself if
   // trusted device policies are not yet available.
   void RetrieveTrustedDevicePolicies();
 
   // Returns true if trusted device policies have successfully been retrieved
@@ skipping 12 matching lines @@
 
   void SetCurrentUserIsOwner(bool is_current_user_owner);
 
   // Updates current user ownership on UI thread.
   void UpdateOwnership(DeviceSettingsService::OwnershipStatus status,
                        bool is_owner);
 
   // Triggers an asynchronous ownership check.
   void CheckOwnership();
 
-  // Removes the user from the persistent list only. Also removes the user's
-  // picture.
-  void RemoveUserFromListInternal(const std::string& email);
+  // Removes data stored or cached outside the user's cryptohome (wallpaper,
+  // avatar, OAuth token status, display name, display email).
+  void RemoveNonCryptohomeData(const std::string& email);
 
-  // List of all known users. User instances are owned by |this| and deleted
-  // when users are removed by |RemoveUserFromListInternal|.
+  // Moves the regular user identified by |email| out of the user list or to its
+  // front, as specified by |move_to_front|.

[pastarmovj, 2012/11/28 09:57:16]

Can you please put some description why is this needed because it sounds like a
very arbitrary operation from this description.

[bartfab (slow), 2012/11/28 12:30:17]

Done.

+  User *UpdateRegularUsers(const std::string& email, bool move_to_front);
+
+  // Replaces the list of device-local users with |local_users_list|. Ensures
+  // that data stored outside theit cryptohomes is removed for any users who are

[pastarmovj, 2012/11/28 09:57:16]

s/theit/their/

[bartfab (slow), 2012/11/28 12:30:17]

Done.

+  // no longer on the list. Returns |true| if the user list has changed.
+  bool UpdateLocalUsers(const base::ListValue& local_users_list);
+
+  // Interface to the signed settings store.
+  CrosSettings* cros_settings_;
+
+  // True if observers have been registered with the signed settings store.
+  bool observing_cros_settings_;
+
+  // True if users have been loaded from prefs already.
+  bool users_loaded_;
+
+  // List of all known users. User instances are owned by |this|. Regular users
+  // are removed by |RemoveUserFromListInternal|, device-local users by
+  // |UpdateLocalUsers|.

[pastarmovj, 2012/11/28 09:57:16]

Write a justification for having this member mutable please.

[bartfab (slow), 2012/11/28 12:30:17]

Actually, there is no justification. This seems to have been modified from
within a const method at some point in the past... but it never is anymore. I
removed the mutable.

   mutable UserList users_;
 
   // The logged-in user. NULL until a user has logged in, then points to one
   // of the User instances in |users_|, the |guest_user_| instance or an
   // ephemeral user instance.
   User* logged_in_user_;
 
   // True if SessionStarted() has been called.
   bool session_started_;
 
@@ skipping 27 matching lines @@
 
   // Profile sync service which is observed to take actions after sync
   // errors appear. NOTE: there is no guarantee that it is the current sync
   // service, so do NOT use it outside |OnStateChanged| method.
   ProfileSyncService* observed_sync_service_;
 
   ObserverList<UserManager::Observer> observer_list_;
 
   scoped_ptr<UserImageManagerImpl> user_image_manager_;
 
+  // Weak pointer factory.
+  base::WeakPtrFactory<UserManagerImpl> weak_ptr_factory_;
+
   DISALLOW_COPY_AND_ASSIGN(UserManagerImpl);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_LOGIN_USER_MANAGER_IMPL_H_