[Profiles] Download high-res avatars using the --new-profile-management flag

chrome/browser/profiles/profile_avatar_downloader.cc

Index: chrome/browser/profiles/profile_avatar_downloader.cc
diff --git a/chrome/browser/profiles/profile_avatar_downloader.cc b/chrome/browser/profiles/profile_avatar_downloader.cc
new file mode 100644
index 0000000000000000000000000000000000000000..9ddae6b508ad0b5b2c58553d258c08f2e2bf3bd1
--- /dev/null
+++ b/chrome/browser/profiles/profile_avatar_downloader.cc
@@ -0,0 +1,51 @@
+// Copyright 2014 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.
+
+#include "chrome/browser/profiles/profile_avatar_downloader.h"
+
+#include "base/files/file_path.h"
+#include "chrome/browser/browser_process.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/profiles/profile_avatar_icon_util.h"
+#include "chrome/browser/profiles/profile_info_cache.h"
+
+namespace {
+const char kHighResAvatarDownloadUrlPrefix[] =
+    "http://www.gstatic.com/chrome/profile_avatars/";
+}
+
+ProfileAvatarDownloader::ProfileAvatarDownloader(
+    size_t icon_index,
+    ProfileInfoCache* cache)
+    : icon_index_(icon_index),
+      cache_(cache) {
+  GURL url(std::string(kHighResAvatarDownloadUrlPrefix) +
+           profiles::GetDefaultAvatarIconFileNameAtIndex(icon_index));
+  fetcher_.reset(new chrome::BitmapFetcher(url, this));
+}
+
+ProfileAvatarDownloader::~ProfileAvatarDownloader() {
+}
+
+void ProfileAvatarDownloader::Start() {
+  // In unit tests, there is no g_browser_process, so we do not have a

[msw, 2014/04/30 19:46:21]

nit: This comment would seem to indicate that g_browser_process is NULL (not a
TestingBrowserProcess that returns a NULL URLRequestContextGetter), can you
rephrase the comment or handle the NULL URLRequestContextGetter if that's really
the case?

[noms (inactive), 2014/04/30 20:02:19]

Updated the comment.
On 2014/04/30 19:46:21, msw wrote:

+  // request context.
+  net::URLRequestContextGetter* request_context =
+      g_browser_process->system_request_context();
+  if (request_context)
+    fetcher_->Start(request_context);
+}
+
+// BitmapFetcherDelegate overrides.
+void ProfileAvatarDownloader::OnFetchComplete(const GURL url,
+                                              const SkBitmap* bitmap) {
+  if (!bitmap || !cache_)
+    return;
+
+  // Decode the downloaded bitmap. Ownership of the image is taken by |cache_|.
+  gfx::Image image = gfx::Image::CreateFrom1xBitmap(*bitmap);
+  cache_->SaveAvatarImageAtPath(&image,
+      profiles::GetDefaultAvatarIconFileNameAtIndex(icon_index_),
+      profiles::GetPathOfHighResAvatarAtIndex(icon_index_));
+}