Add possibility to define data callback to BitmapFetcher

chrome/browser/banners/app_banner_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/banners/app_banner_data_fetcher.h"
 
 #include "base/bind.h"
 #include "base/strings/string_util.h"
 #include "chrome/browser/banners/app_banner_metrics.h"
 #include "chrome/browser/banners/app_banner_settings_helper.h"
@@ skipping 176 matching lines @@
   DCHECK(web_contents);
 
   // Begin asynchronously fetching the app icon.  AddRef() is done before the
   // fetch to ensure that the AppBannerDataFetcher isn't deleted before the
   // BitmapFetcher has called OnFetchComplete() (where the references are
   // decremented).
   AddRef();
   Profile* profile =
       Profile::FromBrowserContext(web_contents->GetBrowserContext());
   bitmap_fetcher_.reset(new chrome::BitmapFetcher(image_url, this));
-  bitmap_fetcher_->Start(
+  bitmap_fetcher_->Init(
       profile->GetRequestContext(),
       std::string(),
       net::URLRequest::CLEAR_REFERRER_ON_TRANSITION_FROM_SECURE_TO_INSECURE,
       net::LOAD_NORMAL);
+  bitmap_fetcher_->Start();
   return true;
 }
 
 infobars::InfoBar* AppBannerDataFetcher::CreateBanner(
     const SkBitmap* icon,
     const base::string16& title) {
   content::WebContents* web_contents = GetWebContents();
   DCHECK(web_contents && !web_app_data_.IsEmpty());
 
   // TODO(dfalcantara): Desktop doesn't display app banners, yet.  Just pretend
@@ skipping 131 matching lines @@
   if (!manifest.start_url.is_valid())
     return false;
   if (manifest.name.is_null() && manifest.short_name.is_null())
     return false;
   if (!DoesManifestContainRequiredIcon(manifest))
     return false;
   return true;
 }
 
 }  // namespace banners