Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(12)

Unified Diff: chrome/browser/profiles/profile_info_cache.h

Issue 1038423004: Reland of: Change default code flag to NewAvatarMenu. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: remove unneeded code from ProfileInfoCacheTest Created 5 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/profiles/profile_info_cache.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/profiles/profile_info_cache.h
diff --git a/chrome/browser/profiles/profile_info_cache.h b/chrome/browser/profiles/profile_info_cache.h
index 7b005fc5c5dceedf928ab44af728ef99d2d1a1de..7b935f826ddd24bf630c45d43b1bedade7bcf852 100644
--- a/chrome/browser/profiles/profile_info_cache.h
+++ b/chrome/browser/profiles/profile_info_cache.h
@@ -62,7 +62,7 @@ class ProfileInfoCache : public ProfileInfoInterface,
base::FilePath GetPathOfProfileAtIndex(size_t index) const override;
base::Time GetProfileActiveTimeAtIndex(size_t index) const override;
base::string16 GetUserNameOfProfileAtIndex(size_t index) const override;
- const gfx::Image& GetAvatarIconOfProfileAtIndex(size_t index) const override;
+ const gfx::Image& GetAvatarIconOfProfileAtIndex(size_t index) override;
std::string GetLocalAuthCredentialsOfProfileAtIndex(
size_t index) const override;
// Note that a return value of false could mean an error in collection or
@@ -148,6 +148,11 @@ class ProfileInfoCache : public ProfileInfoInterface,
void AddObserver(ProfileInfoCacheObserver* obs);
void RemoveObserver(ProfileInfoCacheObserver* obs);
+ void set_disable_avatar_download_for_testing(
+ bool disable_avatar_download_for_testing) {
+ disable_avatar_download_for_testing_ = disable_avatar_download_for_testing;
+ }
+
private:
FRIEND_TEST_ALL_PREFIXES(ProfileInfoCacheTest, DownloadHighResAvatarTest);
@@ -224,6 +229,10 @@ class ProfileInfoCache : public ProfileInfoInterface,
std::map<std::string, ProfileAvatarDownloader*>
avatar_images_downloads_in_progress_;
+ // Determines of the ProfileAvatarDownloader should be created and executed
+ // or not. Only set to true for tests.
+ bool disable_avatar_download_for_testing_;
+
DISALLOW_COPY_AND_ASSIGN(ProfileInfoCache);
};
« no previous file with comments | « no previous file | chrome/browser/profiles/profile_info_cache.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698