Pass in extra parameters to WebApkBuilder#buildWebApkAsync()

chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc

Index: chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc
diff --git a/chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc b/chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc
index 0615896eb51bb702791a00f4a8b5e9b870ab1db0..53f7556601b4f2517c6d915fdda037e6c7d816eb 100644
--- a/chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc
+++ b/chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc
@@ -126,7 +126,7 @@ void AddToHomescreenDataFetcher::OnDidGetManifest(
         ideal_icon_size_in_dp_,
         minimum_icon_size_in_dp_,
         base::Bind(&AddToHomescreenDataFetcher::OnManifestIconFetched,
-                   this))) {
+                   this, icon_src))) {
     FetchFavicon();
   }
 
@@ -236,34 +236,34 @@ void AddToHomescreenDataFetcher::CreateLauncherIcon(
                                                        &is_generated);
   }
 
+  GURL icon_url = is_generated ? GURL() : bitmap_result.icon_url;
   content::BrowserThread::PostTask(
-      content::BrowserThread::UI,
-      FROM_HERE,
-      base::Bind(&AddToHomescreenDataFetcher::NotifyObserver,
-                 this,
-                 icon_bitmap,
-                 is_generated));
+      content::BrowserThread::UI, FROM_HERE,
+      base::Bind(&AddToHomescreenDataFetcher::NotifyObserver, this, icon_url,
+                 icon_bitmap));
 }
 

[dominickn, 2016/06/16 21:16:36]

Instead of passing icon_url through OnManifestIconFetched() and
NotifyObserver(), can shortcut_icon_url_ be set in CreateLauncherIcon() and
OnManifestIconFetched() since this is all on the same object?

This also makes it clearer that icon_url is plumbed through properly from
FetchFavicon()

-void AddToHomescreenDataFetcher::OnManifestIconFetched(const SkBitmap& icon) {
+void AddToHomescreenDataFetcher::OnManifestIconFetched(const GURL& icon_url,
+                                                       const SkBitmap& icon) {
   if (icon.drawsNothing()) {
     FetchFavicon();
     return;
   }
-  NotifyObserver(icon, false);
+  NotifyObserver(icon_url, icon);
 }
 
-void AddToHomescreenDataFetcher::NotifyObserver(const SkBitmap& bitmap,
-                                                bool is_generated) {
+void AddToHomescreenDataFetcher::NotifyObserver(const GURL& icon_url,
+                                                const SkBitmap& bitmap) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
   if (!web_contents() || !weak_observer_ || is_icon_saved_)
     return;
 
   is_icon_saved_ = true;
+  shortcut_icon_url_ = icon_url;
   shortcut_icon_ = bitmap;
-  shortcut_info_.is_icon_generated = is_generated;
   is_ready_ = true;
-  weak_observer_->OnDataAvailable(shortcut_info_, shortcut_icon_);
+  weak_observer_->OnDataAvailable(shortcut_info_, shortcut_icon_url_,
+                                  shortcut_icon_);
 }
 
 GURL AddToHomescreenDataFetcher::GetShortcutUrl(const GURL& actual_url) {