Use IOSImageDataFetcherWrapper in BrowserViewController

components/image_fetcher/ios/ios_image_data_fetcher_wrapper.mm

Index: components/image_fetcher/ios/ios_image_data_fetcher_wrapper.mm
diff --git a/components/image_fetcher/ios/ios_image_data_fetcher_wrapper.mm b/components/image_fetcher/ios/ios_image_data_fetcher_wrapper.mm
index 6bf24c690c545a74e944eada8636b97af9fa2669..9e105d262002fc9539a6075e85260c2be7d1199e 100644
--- a/components/image_fetcher/ios/ios_image_data_fetcher_wrapper.mm
+++ b/components/image_fetcher/ios/ios_image_data_fetcher_wrapper.mm
@@ -86,6 +86,15 @@ IOSImageDataFetcherWrapper::~IOSImageDataFetcherWrapper() {}
 void IOSImageDataFetcherWrapper::FetchImageDataWebpDecoded(
     const GURL& image_url,
     IOSImageDataFetcherCallback callback) {
+  IOSImageDataFetcherWrapper::FetchImageDataWebpDecoded(
+      image_url, callback, std::string(),
+      net::URLRequest::CLEAR_REFERRER_ON_TRANSITION_FROM_SECURE_TO_INSECURE);

[Marc Treib, 2017/02/03 14:10:30]

...and a second place where it's baked in.
IMO this is a strong hint that the design with the IosWrapper isn't quite there
yet. Not for this CL, but we should revisit this at some point.

[gambard, 2017/02/03 14:56:47]

I have changed the interface. Maybe it makes more sense now.

[Marc Treib, 2017/02/03 15:06:45]

Yep, I think that's a bit better, thanks!

At some point, we'll still have to re-think the Wrapper: Having a different
ImageDataFetcher interface on iOS makes it impossible to use the same
ImageFetcher implementation across platforms. But that's for another day :)

[gambard, 2017/02/03 15:11:14]

Actually no :)
There are two usages of image fetchers in iOS: those needing access to the raw
data (which will be using this one) and the one needing access to gfx::Image
(which will be using the shared ImageFetcher).
The WebP decoding will be done in the iOS specific ImageDecoder that will be
passed to the ImageFetcher. We discussed it at go/chrome-ios-image-fetchers

+}
+void IOSImageDataFetcherWrapper::FetchImageDataWebpDecoded(

[Marc Treib, 2017/02/03 14:10:30]

nit: empty line before

[gambard, 2017/02/03 14:56:47]

Done.

+    const GURL& image_url,
+    IOSImageDataFetcherCallback callback,
+    const std::string& referrer,
+    net::URLRequest::ReferrerPolicy referrer_policy) {
   DCHECK(callback);
 
   scoped_refptr<base::TaskRunner> task_runner = task_runner_;
@@ -107,7 +116,8 @@ void IOSImageDataFetcherWrapper::FetchImageDataWebpDecoded(
                                          base::Bind(&DecodeWebpImage, data),
                                          base::BindBlockArc(callback));
       });
-  image_data_fetcher_.FetchImageData(image_url, local_callback);
+  image_data_fetcher_.FetchImageData(image_url, local_callback, referrer,
+                                     referrer_policy);
 }
 
 void IOSImageDataFetcherWrapper::SetDataUseServiceName(