Cache GAIA profile picture URL

chrome/browser/profiles/profile_downloader_delegate.h

Index: chrome/browser/profiles/profile_downloader_delegate.h
diff --git a/chrome/browser/profiles/profile_downloader_delegate.h b/chrome/browser/profiles/profile_downloader_delegate.h
index 2a71a192bd9ce69b2ab2e47ffeef9bbea7625357..4385665a3489e7093c285058b1e8ef963cf8e0e3 100644
--- a/chrome/browser/profiles/profile_downloader_delegate.h
+++ b/chrome/browser/profiles/profile_downloader_delegate.h
@@ -24,6 +24,10 @@ class ProfileDownloaderDelegate {
   // of the originally uploaded size.
   virtual int GetDesiredImageSideLength() = 0;
 
+  // Returns the cached URL. If the cache URL matches the new image URL
+  // the image will not be downloaded.

[Ivan Korotkov, 2011/11/30 14:35:44]

Please document what to return when there is no cached URL (empty string?)

[sail, 2011/11/30 19:00:23]

Done.

+  virtual std::string GetCachedPictureURL() = 0;

[Ivan Korotkov, 2011/11/30 14:35:44]

I think that this and the previous getter should be "const". For
GetBrowserProfile it probably makes sense to not be const since it returns
non-const Profile*.

[sail, 2011/11/30 19:00:23]

Done.

+
   // Returns the browser profile associated with this download request.
   virtual Profile* GetBrowserProfile() = 0;