Make favicon and manifest icon processing more consistent for add to homescreen.

chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc

 // Copyright 2015 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/android/webapps/add_to_homescreen_data_fetcher.h"
 
 #include <vector>
 
 #include "base/bind.h"
 #include "base/callback.h"
@@ skipping 90 matching lines @@
 
   // Sanitize received_web_app_info.
   WebApplicationInfo web_app_info = received_web_app_info;
   web_app_info.title =
       web_app_info.title.substr(0, chrome::kMaxMetaTagAttributeLength);
   web_app_info.description =
       web_app_info.description.substr(0, chrome::kMaxMetaTagAttributeLength);
 
   // Simply set the user-editable title to be the page's title
   shortcut_info_.user_title = web_app_info.title.empty()
-                                    ? web_contents()->GetTitle()
-                                    : web_app_info.title;
+                                  ? web_contents()->GetTitle()
+                                  : web_app_info.title;
   shortcut_info_.short_name = shortcut_info_.user_title;
   shortcut_info_.name = shortcut_info_.user_title;
 
   if (web_app_info.mobile_capable == WebApplicationInfo::MOBILE_CAPABLE ||
       web_app_info.mobile_capable == WebApplicationInfo::MOBILE_CAPABLE_APPLE) {
     shortcut_info_.display = blink::WebDisplayModeStandalone;
   }
 
   // Record what type of shortcut was added by the user.
   switch (web_app_info.mobile_capable) {
@@ skipping 29 matching lines @@
       base::Bind(&AddToHomescreenDataFetcher::OnDidPerformInstallableCheck,
                  this));
 }
 
 void AddToHomescreenDataFetcher::OnDidPerformInstallableCheck(
     const InstallableData& data) {
   if (!web_contents() || !weak_observer_)
     return;
 
   if (!data.manifest.IsEmpty()) {
-      content::RecordAction(
-          base::UserMetricsAction("webapps.AddShortcut.Manifest"));
-      shortcut_info_.UpdateFromManifest(data.manifest);
-      shortcut_info_.manifest_url = data.manifest_url;
+    content::RecordAction(
+        base::UserMetricsAction("webapps.AddShortcut.Manifest"));
+    shortcut_info_.UpdateFromManifest(data.manifest);
+    shortcut_info_.manifest_url = data.manifest_url;
   }
 
   // Save the splash screen URL for the later download.
   splash_screen_url_ = ManifestIconSelector::FindBestMatchingIcon(
       data.manifest.icons, ideal_splash_image_size_in_dp_,
       minimum_splash_image_size_in_dp_);
 
   weak_observer_->OnUserTitleAvailable(shortcut_info_.user_title);
 
   if (data.icon && !data.icon->drawsNothing()) {
     // TODO(hanxi): implement WebAPK path if shortcut_info_.url has a secure
     // scheme and data.is_installable is true.
-    NotifyObserver(data.icon_url, *(data.icon));
+    shortcut_info_.icon_url = data.icon_url;
+
+    // base::Bind copies the arguments internally, so it is safe to pass it
+    // data.icon (which is not owned by us).
+    content::BrowserThread::GetBlockingPool()
+        ->PostWorkerTaskWithShutdownBehavior(
+            FROM_HERE,
+            base::Bind(
+                &AddToHomescreenDataFetcher::CreateLauncherIconInBackground,
+                this, *(data.icon)),
+            base::SequencedWorkerPool::SKIP_ON_SHUTDOWN);
     return;
   }
 
   FetchFavicon();
 }
 
 bool AddToHomescreenDataFetcher::OnMessageReceived(
     const IPC::Message& message) {
   if (!is_waiting_for_web_application_info_)
     return false;
@@ skipping 36 matching lines @@
           Profile::FromBrowserContext(web_contents()->GetBrowserContext()),
           ServiceAccessType::EXPLICIT_ACCESS);
 
   // Using favicon if its size is not smaller than platform required size,
   // otherwise using the largest icon among all avaliable icons.
   int ideal_icon_size_in_px =
       ideal_icon_size_in_dp_ *
       display::Screen::GetScreen()->GetPrimaryDisplay().device_scale_factor();
   int threshold_to_get_any_largest_icon = ideal_icon_size_in_px - 1;
   favicon_service->GetLargestRawFaviconForPageURL(
-      shortcut_info_.url,
-      icon_types,
-      threshold_to_get_any_largest_icon,
+      shortcut_info_.url, icon_types, threshold_to_get_any_largest_icon,
       base::Bind(&AddToHomescreenDataFetcher::OnFaviconFetched, this),
       &favicon_task_tracker_);
 }
 
 void AddToHomescreenDataFetcher::OnFaviconFetched(
     const favicon_base::FaviconRawBitmapResult& bitmap_result) {
   if (!web_contents() || !weak_observer_ || is_icon_saved_)
     return;
 
   content::BrowserThread::GetBlockingPool()->PostWorkerTaskWithShutdownBehavior(
-      FROM_HERE,
-      base::Bind(&AddToHomescreenDataFetcher::CreateLauncherIconInBackground,
-                 this, shortcut_info_.url, bitmap_result),
+      FROM_HERE, base::Bind(&AddToHomescreenDataFetcher::
+                                CreateLauncherIconFromFaviconInBackground,
+                            this, bitmap_result),
       base::SequencedWorkerPool::SKIP_ON_SHUTDOWN);
 }
 
-void AddToHomescreenDataFetcher::CreateLauncherIconInBackground(
-    const GURL& page_url,
+void AddToHomescreenDataFetcher::CreateLauncherIconFromFaviconInBackground(
     const favicon_base::FaviconRawBitmapResult& bitmap_result) {
   DCHECK(content::BrowserThread::GetBlockingPool()->RunsTasksOnCurrentThread());
 
-  SkBitmap icon_bitmap;
+  SkBitmap raw_icon;
   if (bitmap_result.is_valid()) {
     gfx::PNGCodec::Decode(bitmap_result.bitmap_data->front(),
-                          bitmap_result.bitmap_data->size(),
-                          &icon_bitmap);
+                          bitmap_result.bitmap_data->size(), &raw_icon);
   }
 
+  shortcut_info_.icon_url = bitmap_result.icon_url;
+  CreateLauncherIconInBackground(raw_icon);
+}
+
+void AddToHomescreenDataFetcher::CreateLauncherIconInBackground(
+    const SkBitmap& raw_icon) {
+  DCHECK(content::BrowserThread::GetBlockingPool()->RunsTasksOnCurrentThread());
+
+  SkBitmap icon;
   bool is_generated = false;
   if (weak_observer_) {
-    icon_bitmap = weak_observer_->FinalizeLauncherIconInBackground(
-        icon_bitmap, page_url, &is_generated);
+    icon = weak_observer_->FinalizeLauncherIconInBackground(
+        raw_icon, shortcut_info_.url, &is_generated);
   }
 
-  GURL icon_url = is_generated ? GURL() : bitmap_result.icon_url;
+  if (is_generated)
+    shortcut_info_.icon_url = GURL();
+
   content::BrowserThread::PostTask(
       content::BrowserThread::UI, FROM_HERE,
-      base::Bind(&AddToHomescreenDataFetcher::NotifyObserver, this, icon_url,
-                 icon_bitmap));
+      base::Bind(&AddToHomescreenDataFetcher::NotifyObserver, this, icon));
 }
 
-void AddToHomescreenDataFetcher::NotifyObserver(const GURL& icon_url,
-                                                const SkBitmap& bitmap) {
+void AddToHomescreenDataFetcher::NotifyObserver(const SkBitmap& icon) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
   if (!web_contents() || !weak_observer_ || is_icon_saved_)
     return;
 
   is_icon_saved_ = true;
-  shortcut_info_.icon_url = icon_url;
-  shortcut_icon_ = bitmap;
+  shortcut_icon_ = icon;
   is_ready_ = true;
   weak_observer_->OnDataAvailable(shortcut_info_, shortcut_icon_);
 }