Don't try to download 'NothingToDownload' placeholder avatar.

chrome/browser/profiles/profile_info_cache_unittest.cc

Index: chrome/browser/profiles/profile_info_cache_unittest.cc
diff --git a/chrome/browser/profiles/profile_info_cache_unittest.cc b/chrome/browser/profiles/profile_info_cache_unittest.cc
index 67055fbe8d687d2119cb9b7b0ce3c12e9f99cdee..87c7a24fcbd1a9da6b68ab2fbc2c7ebdf5d17286 100644
--- a/chrome/browser/profiles/profile_info_cache_unittest.cc
+++ b/chrome/browser/profiles/profile_info_cache_unittest.cc
@@ -614,6 +614,31 @@ TEST_F(ProfileInfoCacheTest, DownloadHighResAvatarTest) {
   EXPECT_FALSE(base::PathExists(icon_path));
 }
 
+TEST_F(ProfileInfoCacheTest, NothingToDownloadHighResAvatarTest) {
+  switches::EnableNewAvatarMenuForTesting(
+      base::CommandLine::ForCurrentProcess());
+
+  // The TestingProfileManager's ProfileInfoCache doesn't download avatars.
+  ProfileInfoCache profile_info_cache(
+      g_browser_process->local_state(),
+      testing_profile_manager_.profile_manager()->user_data_dir());
+
+  const size_t kIconIndex = profiles::GetPlaceholderAvatarIndex();
+
+  EXPECT_EQ(0U, profile_info_cache.GetNumberOfProfiles());
+  base::FilePath path_1 = GetProfilePath("path_1");
+  profile_info_cache.AddProfileToCache(path_1, ASCIIToUTF16("name_1"),
+                                       std::string(), base::string16(),
+                                       kIconIndex, std::string());
+  EXPECT_EQ(1U, profile_info_cache.GetNumberOfProfiles());
+  base::RunLoop().RunUntilIdle();
+
+  // We haven't tried to download any high-res avatars as the specified icon is
+  // just a placeholder.
+  EXPECT_EQ(0U, profile_info_cache.cached_avatar_images_.size());
+  EXPECT_EQ(0U, profile_info_cache.avatar_images_downloads_in_progress_.size());
+}
+
 TEST_F(ProfileInfoCacheTest, MigrateLegacyProfileNamesWithNewAvatarMenu) {
   switches::EnableNewAvatarMenuForTesting(
       base::CommandLine::ForCurrentProcess());