Implement DownloadImage directly in WebStateImpl

ios/web/web_state/web_state_impl.h

Index: ios/web/web_state/web_state_impl.h
diff --git a/ios/web/web_state/web_state_impl.h b/ios/web/web_state/web_state_impl.h
index d75eb41e4ad083160e82e18bb5e2cb0b2d43fe28..9400c6c862a0617baf4e2d89d6b474139aebadea 100644
--- a/ios/web/web_state/web_state_impl.h
+++ b/ios/web/web_state/web_state_impl.h
@@ -43,6 +43,7 @@ struct Credential;
 struct FaviconURL;
 struct LoadCommittedDetails;
 class NavigationManager;
+class RawImageFetcher;
 class WebInterstitialImpl;
 class WebStateDelegate;
 class WebStateFacadeDelegate;
@@ -369,6 +370,9 @@ class WebStateImpl : public WebState, public NavigationManagerDelegate {
   // Mojo interface registry for this WebState.
   std::unique_ptr<service_manager::InterfaceRegistry> mojo_interface_registry_;
 
+  // Image Fetcher used to download favicons.

[Eugene But (OOO till 7-30), 2016/11/30 17:13:50]

This can download any image, not only favicons. Right?

[gambard, 2016/12/01 15:14:21]

Yes but for now |downloadImage| is only called for favicon.
Done.

+  std::unique_ptr<RawImageFetcher> image_fetcher_;
+
   DISALLOW_COPY_AND_ASSIGN(WebStateImpl);
 };