Sooper experimental refactoring of the profile info cache.

chrome/browser/profiles/profile_info_cache.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_PROFILES_PROFILE_INFO_CACHE_H_
 #define CHROME_BROWSER_PROFILES_PROFILE_INFO_CACHE_H_
 
 #include <map>
 #include <string>
 #include <vector>
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "base/files/file_path.h"
 #include "base/memory/weak_ptr.h"
 #include "base/observer_list.h"
 #include "base/strings/string16.h"
 #include "chrome/browser/profiles/profile_info_cache_observer.h"
 #include "chrome/browser/profiles/profile_info_interface.h"
+#include "chrome/browser/profiles/profile_info_entry.h"
 
 namespace gfx {
 class Image;
 }
 
 namespace base {
 class DictionaryValue;
 }
 
 class PrefService;
 class PrefRegistrySimple;
 
 // This class saves various information about profiles to local preferences.
 // This cache can be used to display a list of profiles without having to
 // actually load the profiles from disk.
 class ProfileInfoCache : public ProfileInfoInterface,
                          public base::SupportsWeakPtr<ProfileInfoCache> {
  public:
   ProfileInfoCache(PrefService* prefs, const base::FilePath& user_data_dir);
   virtual ~ProfileInfoCache();
 
+
+
+  // ******************* Start new code
+
+  virtual std::vector<ProfileInfoEntry> GetProfilesSortedByName() const OVERRIDE;
+
+  bool GetInfoForProfile(const base::FilePath& path,
+                         ProfileInfoEntry* entry) const;
+
+  void SetInfoForProfile(const ProfileInfoEntry& info);
+
   // This |is_managed| refers to local management (formerly "managed mode"),
   // not enterprise management.
   void AddProfileToCache(const base::FilePath& profile_path,
                          const string16& name,
                          const string16& username,
                          size_t icon_index,
                          const std::string& managed_user_id);
+
   void DeleteProfileFromCache(const base::FilePath& profile_path);
 
+
+  // ********************* End new code
+
+
+
+
+
+
   // ProfileInfoInterface:
   virtual size_t GetNumberOfProfiles() const OVERRIDE;
+
   // Don't cache this value and reuse, because resorting the menu could cause
   // the item being referred to to change out from under you.
-  virtual size_t GetIndexOfProfileWithPath(
-      const base::FilePath& profile_path) const OVERRIDE;
-  virtual string16 GetNameOfProfileAtIndex(size_t index) const OVERRIDE;
-  virtual string16 GetShortcutNameOfProfileAtIndex(size_t index)
-      const OVERRIDE;
-  virtual base::FilePath GetPathOfProfileAtIndex(size_t index) const OVERRIDE;
-  virtual string16 GetUserNameOfProfileAtIndex(size_t index) const OVERRIDE;
-  virtual const gfx::Image& GetAvatarIconOfProfileAtIndex(
-      size_t index) const OVERRIDE;
+  // virtual size_t GetIndexOfProfileWithPath(
+  //     const base::FilePath& profile_path) const OVERRIDE;
+
+
+
+  //virtual string16 GetNameOfProfileAtIndex(size_t index) const OVERRIDE;
+  //virtual string16 GetShortcutNameOfProfileAtIndex(size_t index)
+  //    const OVERRIDE;
+  //virtual base::FilePath GetPathOfProfileAtIndex(size_t index) const OVERRIDE;
+  //virtual string16 GetUserNameOfProfileAtIndex(size_t index) const OVERRIDE;
+  virtual const gfx::Image& GetAvatarIconOfProfile(const base::FilePath& profile_path) const OVERRIDE;
   // Note that a return value of false could mean an error in collection or
   // that there are currently no background apps running. However, the action
   // which results is the same in both cases (thus far).
-  virtual bool GetBackgroundStatusOfProfileAtIndex(
-      size_t index) const OVERRIDE;
-  virtual string16 GetGAIANameOfProfileAtIndex(size_t index) const OVERRIDE;
-  virtual string16 GetGAIAGivenNameOfProfileAtIndex(
-      size_t index) const OVERRIDE;
-  virtual bool IsUsingGAIANameOfProfileAtIndex(size_t index) const OVERRIDE;
+  //virtual bool GetBackgroundStatusOfProfileAtIndex(
+  //    size_t index) const OVERRIDE;
+  //virtual string16 GetGAIANameOfProfileAtIndex(size_t index) const OVERRIDE;
+  //virtual string16 GetGAIAGivenNameOfProfileAtIndex(
+  //    size_t index) const OVERRIDE;
+  //virtual bool IsUsingGAIANameOfProfileAtIndex(size_t index) const OVERRIDE;
   // Returns the GAIA picture for the given profile. This may return NULL
   // if the profile does not have a GAIA picture or if the picture must be
   // loaded from disk.
-  virtual const gfx::Image* GetGAIAPictureOfProfileAtIndex(
-      size_t index) const OVERRIDE;
-  virtual bool IsUsingGAIAPictureOfProfileAtIndex(
-      size_t index) const OVERRIDE;
-  virtual bool ProfileIsManagedAtIndex(size_t index) const OVERRIDE;
-  virtual bool ProfileIsSigninRequiredAtIndex(size_t index) const OVERRIDE;
-  virtual std::string GetManagedUserIdOfProfileAtIndex(size_t index) const
-      OVERRIDE;
-  virtual bool ProfileIsEphemeralAtIndex(size_t index) const OVERRIDE;
 
-  size_t GetAvatarIconIndexOfProfileAtIndex(size_t index) const;
 
-  void SetNameOfProfileAtIndex(size_t index, const string16& name);
-  void SetShortcutNameOfProfileAtIndex(size_t index, const string16& name);
-  void SetUserNameOfProfileAtIndex(size_t index, const string16& user_name);
-  void SetAvatarIconOfProfileAtIndex(size_t index, size_t icon_index);
-  void SetManagedUserIdOfProfileAtIndex(size_t index, const std::string& id);
-  void SetBackgroundStatusOfProfileAtIndex(size_t index,
-                                           bool running_background_apps);
-  void SetGAIANameOfProfileAtIndex(size_t index, const string16& name);
-  void SetGAIAGivenNameOfProfileAtIndex(size_t index, const string16& name);
-  void SetIsUsingGAIANameOfProfileAtIndex(size_t index, bool value);
-  void SetGAIAPictureOfProfileAtIndex(size_t index, const gfx::Image* image);
-  void SetIsUsingGAIAPictureOfProfileAtIndex(size_t index, bool value);
-  void SetProfileSigninRequiredAtIndex(size_t index, bool value);
-  void SetProfileIsEphemeralAtIndex(size_t index, bool value);
+  // *** FROM HEAD
+  // virtual const gfx::Image* GetGAIAPictureOfProfileAtIndex(
+  //     size_t index) const OVERRIDE;
+  // virtual bool IsUsingGAIAPictureOfProfileAtIndex(
+  //     size_t index) const OVERRIDE;
+  // virtual bool ProfileIsManagedAtIndex(size_t index) const OVERRIDE;
+  // virtual bool ProfileIsSigninRequiredAtIndex(size_t index) const OVERRIDE;
+  // virtual std::string GetManagedUserIdOfProfileAtIndex(size_t index) const
+  //     OVERRIDE;
+  // virtual bool ProfileIsEphemeralAtIndex(size_t index) const OVERRIDE;
+
+  // size_t GetAvatarIconIndexOfProfileAtIndex(size_t index) const;
+
+  // void SetNameOfProfileAtIndex(size_t index, const string16& name);
+  // void SetShortcutNameOfProfileAtIndex(size_t index, const string16& name);
+  // void SetUserNameOfProfileAtIndex(size_t index, const string16& user_name);
+  // void SetAvatarIconOfProfileAtIndex(size_t index, size_t icon_index);
+  // void SetManagedUserIdOfProfileAtIndex(size_t index, const std::string& id);
+  // void SetBackgroundStatusOfProfileAtIndex(size_t index,
+  //                                          bool running_background_apps);
+  // void SetGAIANameOfProfileAtIndex(size_t index, const string16& name);
+  // void SetGAIAGivenNameOfProfileAtIndex(size_t index, const string16& name);
+  // void SetIsUsingGAIANameOfProfileAtIndex(size_t index, bool value);
+  // void SetGAIAPictureOfProfileAtIndex(size_t index, const gfx::Image* image);
+  // void SetIsUsingGAIAPictureOfProfileAtIndex(size_t index, bool value);
+  // void SetProfileSigninRequiredAtIndex(size_t index, bool value);
+  // void SetProfileIsEphemeralAtIndex(size_t index, bool value);
+
+  virtual const gfx::Image* GetGAIAPictureOfProfile(const base::FilePath& profile_path) const OVERRIDE;
+  //virtual bool IsUsingGAIAPictureOfProfileAtIndex(
+  //    size_t index) const OVERRIDE;
+  //virtual bool ProfileIsManagedAtIndex(size_t index) const OVERRIDE;
+  //virtual bool ProfileIsSigninRequiredAtIndex(size_t index) const OVERRIDE;
+  //virtual std::string GetManagedUserIdOfProfileAtIndex(size_t index) const
+  //    OVERRIDE;
+
+  //size_t GetAvatarIconIndexOfProfileAtIndex(size_t index) const;
+
+  //void SetNameOfProfileAtIndex(size_t index, const string16& name);
+  //void SetShortcutNameOfProfileAtIndex(size_t index, const string16& name);
+  //void SetUserNameOfProfileAtIndex(size_t index, const string16& user_name);
+  //void SetAvatarIconOfProfileAtIndex(size_t index, size_t icon_index);
+  //void SetManagedUserIdOfProfileAtIndex(size_t index, const std::string& id);
+  //void SetBackgroundStatusOfProfileAtIndex(size_t index,
+   //                                        bool running_background_apps);
+  //void SetGAIANameOfProfileAtIndex(size_t index, const string16& name);
+  //void SetGAIAGivenNameOfProfileAtIndex(size_t index, const string16& name);
+  //void SetIsUsingGAIANameOfProfileAtIndex(size_t index, bool value);
+  void SetGAIAPictureOfProfile(const base::FilePath& profile_path, const gfx::Image* image);
+  //void SetIsUsingGAIAPictureOfProfileAtIndex(size_t index, bool value);
+  //void SetProfileSigninRequiredAtIndex(size_t index, bool value);
+
+
+
 
   // Returns unique name that can be assigned to a newly created profile.
   string16 ChooseNameForNewProfile(size_t icon_index) const;
 
   // Checks if the given profile has switched to using GAIA information
   // for the profile name and picture. This pref is used to switch over
   // to GAIA info the first time it is available. Afterwards this pref is
   // checked to prevent clobbering the user's custom settings.
-  bool GetHasMigratedToGAIAInfoOfProfileAtIndex(size_t index) const;
+  //bool GetHasMigratedToGAIAInfoOfProfileAtIndex(size_t index) const;
 
   // Marks the given profile as having switched to using GAIA information
   // for the profile name and picture.
-  void SetHasMigratedToGAIAInfoOfProfileAtIndex(size_t index, bool value);
+  //void SetHasMigratedToGAIAInfoOfProfileAtIndex(size_t index, bool value);
 
   // Returns an avatar icon index that can be assigned to a newly created
   // profile. Note that the icon may not be unique since there are a limited
   // set of default icons.
   size_t ChooseAvatarIconIndexForNewProfile() const;
 
   const base::FilePath& GetUserDataDir() const;
 
   // Gets the number of default avatar icons that exist.
   static size_t GetDefaultAvatarIconCount();
@@ skipping 14 matching lines @@
   // rather than going through the ProfileInfoCache object.
   static std::vector<string16> GetProfileNames();
 
   // Register cache related preferences in Local State.
   static void RegisterPrefs(PrefRegistrySimple* registry);
 
   void AddObserver(ProfileInfoCacheObserver* obs);
   void RemoveObserver(ProfileInfoCacheObserver* obs);
 
  private:
-  const base::DictionaryValue* GetInfoForProfileAtIndex(size_t index) const;
+  //const base::DictionaryValue* GetInfoForProfileAtIndex(size_t index) const;
   // Saves the profile info to a cache and takes ownership of |info|.
   // Currently the only information that is cached is the profile's name,
   // user name, and avatar icon.
-  void SetInfoForProfileAtIndex(size_t index, base::DictionaryValue* info);
+  //void SetInfoForProfileAtIndex(size_t index, base::DictionaryValue* info);
   std::string CacheKeyFromProfilePath(const base::FilePath& profile_path) const;
-  std::vector<std::string>::iterator FindPositionForProfile(
-      const std::string& search_key,
-      const string16& search_name);
+  //std::vector<std::string>::iterator FindPositionForProfile(
+  //    const std::string& search_key,
+  //    const string16& search_name);
 
   // Returns true if the given icon index is not in use by another profie.
   bool IconIndexIsUnique(size_t icon_index) const;
 
   // Tries to find an icon index that satisfies all the given conditions.
   // Returns true if an icon was found, false otherwise.
   bool ChooseAvatarIconIndexForNewProfile(bool allow_generic_icon,
                                           bool must_be_unique,
                                           size_t* out_icon_index) const;
 
   // Updates the position of the profile at the given index so that the list
   // of profiles is still sorted.
-  void UpdateSortForProfileIndex(size_t index);
+  //void UpdateSortForProfileIndex(size_t index);
 
   void OnGAIAPictureLoaded(const base::FilePath& path,
                            gfx::Image** image) const;
   void OnGAIAPictureSaved(const base::FilePath& path, bool* success) const;
 
   PrefService* prefs_;
-  std::vector<std::string> sorted_keys_;
+  //std::vector<std::string> sorted_keys_;
+  std::map<std::string, ProfileInfoEntry> cached_entries_;
   base::FilePath user_data_dir_;
 
   ObserverList<ProfileInfoCacheObserver> observer_list_;
 
   // A cache of gaia profile pictures. This cache is updated lazily so it needs
   // to be mutable.
   mutable std::map<std::string, gfx::Image*> gaia_pictures_;
   // Marks a gaia profile picture as loading. This prevents a picture from
   // loading multiple times.
   mutable std::map<std::string, bool> gaia_pictures_loading_;
 
   DISALLOW_COPY_AND_ASSIGN(ProfileInfoCache);
 };
 
 #endif  // CHROME_BROWSER_PROFILES_PROFILE_INFO_CACHE_H_