Add possibility to define data callback to BitmapFetcher

chrome/browser/bitmap_fetcher/bitmap_fetcher.h

Index: chrome/browser/bitmap_fetcher/bitmap_fetcher.h
diff --git a/chrome/browser/bitmap_fetcher/bitmap_fetcher.h b/chrome/browser/bitmap_fetcher/bitmap_fetcher.h
index 11055eaf61a50e8aaf7a051e629858c5435e7de5..0f7f8c1ce4008a3c30786fcf5c950cc3ebca2654 100644
--- a/chrome/browser/bitmap_fetcher/bitmap_fetcher.h
+++ b/chrome/browser/bitmap_fetcher/bitmap_fetcher.h
@@ -29,6 +29,7 @@ class BitmapFetcher : public net::URLFetcherDelegate,
   ~BitmapFetcher() override;
 
   const GURL& url() const { return url_; }
+  net::URLFetcher* url_fetcher() { return url_fetcher_.get(); }
 
   // Start fetching the URL with the fetcher. The delegate is notified

[Peter Kasting, 2015/04/24 23:18:21]

This comment needs updating.  In particular you should motivate why Init()
doesn't automatically Start().

   // asynchronously when done.  Start may be called more than once in some
@@ -36,10 +37,12 @@ class BitmapFetcher : public net::URLFetcherDelegate,
   // already in progress.  Arguments are used to configure the internal fetcher.
   // Values for |load_flags| are defined in net/base/load_flags.h.  In general,
   // |net::LOAD_NORMAL| is appropriate.
-  void Start(net::URLRequestContextGetter* request_context,
-             const std::string& referrer,
-             net::URLRequest::ReferrerPolicy referrer_policy,
-             int load_flags);
+  void Init(net::URLRequestContextGetter* request_context,
+            const std::string& referrer,
+            net::URLRequest::ReferrerPolicy referrer_policy,
+            int load_flags);
+
+  void Start();

[Peter Kasting, 2015/04/24 23:18:21]

You should add a comment here.

 
   // Methods inherited from URLFetcherDelegate