Use IOSImageDataFetcherWrapper for favicon

components/favicon/ios/web_favicon_driver.h

Index: components/favicon/ios/web_favicon_driver.h
diff --git a/components/favicon/ios/web_favicon_driver.h b/components/favicon/ios/web_favicon_driver.h
index b5209a1b7a0a5067db20c93bb3e054ad3e18fcef..61bd320851ac4aa749f0f233091b4f16da0c9c14 100644
--- a/components/favicon/ios/web_favicon_driver.h
+++ b/components/favicon/ios/web_favicon_driver.h
@@ -7,9 +7,14 @@
 
 #include "base/macros.h"
 #include "components/favicon/core/favicon_driver_impl.h"
+#import "components/image_fetcher/ios/ios_image_data_fetcher_wrapper.h"

[mef, 2017/02/06 16:36:17]

is this #import needed here if you forward declare it below?

[sdefresne, 2017/02/06 17:33:33]

I think the import is required as the image_fetcher_ field is not a pointer (so
the full type need to be known). The forward-declaration is redundant.

[gambard, 2017/02/07 10:03:33]

Yes, sorry.

 #include "ios/web/public/web_state/web_state_observer.h"
 #include "ios/web/public/web_state/web_state_user_data.h"
 
+namespace image_fetcher {
+class IOSImageDataFetcherWrapper;

[sdefresne, 2017/02/06 17:33:33]

Remove.

[gambard, 2017/02/07 10:03:33]

Done.

+}
+
 namespace web {
 class WebState;
 }
@@ -58,6 +63,9 @@ class WebFaviconDriver : public web::WebStateObserver,
   // The URL passed to FetchFavicon().
   GURL fetch_favicon_url_;
 
+  // Image Fetcher used to fetch favicon.
+  image_fetcher::IOSImageDataFetcherWrapper image_fetcher_;
+
   DISALLOW_COPY_AND_ASSIGN(WebFaviconDriver);
 };