Add GAIA info to profile info cache

chrome/browser/profiles/profile_info_cache.h

 // Copyright (c) 2011 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_
 #pragma once
 
 #include <string>
 #include <vector>
+#include <map>

[Miranda Callahan, 2011/11/17 15:21:10]

nit: alpha order

[sail, 2011/11/21 23:52:47]

Done.

 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "base/file_path.h"
 #include "base/string16.h"
 #include "chrome/browser/profiles/profile_info_interface.h"
 
+namespace base {
+class DictionaryValue;
+}

[Robert Sesek, 2011/11/17 20:42:04]

nit: blank line after

[sail, 2011/11/21 23:52:47]

Done.

 namespace gfx {
 class Image;
 }
 
-namespace base {
-class DictionaryValue;
-}
-
 class PrefService;
 
 
 // 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:
   ProfileInfoCache(PrefService* prefs, const FilePath& user_data_dir);
   virtual ~ProfileInfoCache();
@@ skipping 10 matching lines @@
   // the item being referred to to change out from under you.
   virtual size_t GetIndexOfProfileWithPath(
       const FilePath& profile_path) const OVERRIDE;
   virtual string16 GetNameOfProfileAtIndex(size_t index) const OVERRIDE;
   virtual 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 bool GetBackgroundStatusOfProfileAtIndex(
       size_t index) const OVERRIDE;
+  virtual string16 GetGAIANameOfProfileAtIndex(size_t index) const OVERRIDE;
+  virtual bool GetIsUsingGAIANameOfProfileAtIndex(size_t index) const OVERRIDE;

[Robert Sesek, 2011/11/17 20:42:04]

When you return bool, I don't think you need the Get*. Just IsUsing.

[sail, 2011/11/21 23:52:47]

Done.

+  virtual const gfx::Image& GetGAIAPictureOfProfileAtIndex(
+      size_t index) const OVERRIDE;
+  virtual bool GetIsUsingGAIAPictureOfProfileAtIndex(

[Robert Sesek, 2011/11/17 20:42:04]

Same here.

[sail, 2011/11/21 23:52:47]

Done.

+      size_t index) const OVERRIDE;
 
   size_t GetAvatarIconIndexOfProfileAtIndex(size_t index) const;
 
   void SetNameOfProfileAtIndex(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 SetBackgroundStatusOfProfileAtIndex(size_t index,
                                            bool running_background_apps);
+  void SetGAIANameOfProfileAtIndex(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);
 
   // Returns unique name that can be assigned to a newly created profile.
   string16 ChooseNameForNewProfile(size_t icon_index);
 
+  bool GetHasMigratedToGAIAInfoOfProfileAtIndex(size_t index) const;

[Miranda Callahan, 2011/11/17 15:21:10]

Could you add a comment for these two?

[Robert Sesek, 2011/11/17 20:42:04]

And here.

[sail, 2011/11/21 23:52:47]

Done.

[sail, 2011/11/21 23:52:47]

Done.

+  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 FilePath& GetUserDataDir() const;
 
   // Gets the number of default avatar icons that exist.
   static size_t GetDefaultAvatarIconCount();
   // Gets the resource ID of the default avatar icon at |index|.
@@ skipping 21 matching lines @@
 
   // 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;
 
+  void UpdateSortForProfileIndex(size_t index);

[Miranda Callahan, 2011/11/17 15:21:10]

Comment for this one, too.

[sail, 2011/11/21 23:52:47]

Done.

+
+  void OnGAIAPictureLoaded(FilePath path, gfx::Image* image) const;
+  void OnGAIAPictureSaved(FilePath path, bool success) const;
+
   PrefService* prefs_;
   std::vector<std::string> sorted_keys_;
   FilePath user_data_dir_;
 
+  // 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_;
+
   DISALLOW_COPY_AND_ASSIGN(ProfileInfoCache);
 };
 
 #endif  // CHROME_BROWSER_PROFILES_PROFILE_INFO_CACHE_H_