Implement manifest icon downloader

chrome/browser/android/banners/app_banner_data_fetcher_android.cc

Index: chrome/browser/android/banners/app_banner_data_fetcher_android.cc
diff --git a/chrome/browser/android/banners/app_banner_data_fetcher_android.cc b/chrome/browser/android/banners/app_banner_data_fetcher_android.cc
index cb96a9e7c3b9e1e5a70e52c080e879e2378ee94c..3be63521169a008e98c0d99cf582cfa3ca65f253 100644
--- a/chrome/browser/android/banners/app_banner_data_fetcher_android.cc
+++ b/chrome/browser/android/banners/app_banner_data_fetcher_android.cc
@@ -35,7 +35,11 @@ bool AppBannerDataFetcherAndroid::ContinueFetching(
   set_app_title(app_title);
   native_app_package_ = app_package;
   native_app_data_.Reset(app_data);
-  return FetchIcon(image_url);
+
+  content::WebContents* web_contents = GetWebContents();
+  DCHECK(web_contents);

[mlamouri (slow - plz ping), 2015/08/05 08:20:16]

nit: I would move the DCHECK() inside FetchAppIcon().

IOW, I would do:
return FetchAppIcon(GetWebContents(), image_url);

[Lalit Maganti, 2015/08/05 14:47:13]

Got rid of web contents all together as I don't even need it.

+
+  return FetchAppIcon(web_contents, image_url);
 }
 
 std::string AppBannerDataFetcherAndroid::GetAppIdentifier() {