Split FaviconHelper in two: ImageLoadingHelper and FaviconHelper

content/renderer/render_view_impl.h

Index: content/renderer/render_view_impl.h
diff --git a/content/renderer/render_view_impl.h b/content/renderer/render_view_impl.h
index 1842a7e01ab828b09a073529cf8b9d83d48398e0..35fe5bc32901c9d8060c1b20a941aadf23455174 100644
--- a/content/renderer/render_view_impl.h
+++ b/content/renderer/render_view_impl.h
@@ -151,6 +151,7 @@ class DomAutomationController;
 class ExternalPopupMenu;
 class FaviconHelper;
 class GeolocationDispatcher;
+class ImageLoadingHelper;
 class InputTagSpeechDispatcher;
 class JavaBridgeDispatcher;
 class LoadProgressTracker;
@@ -1443,6 +1444,9 @@ class CONTENT_EXPORT RenderViewImpl
   // Helper class to handle favicon changes.
   FaviconHelper* favicon_helper_;
 
+  // Helper class to handle images/favicons loading.
+  ImageLoadingHelper* image_loading_helper_;

[jam, 2013/03/19 20:12:21]

no need to keep a pointer in this class since it's not being used. just create
it like other helpers and it'll delete itself when the RV is gone

[Dmitry Titov, 2013/03/20 01:30:22]

Done.

+
 #if defined(OS_ANDROID)
   // Android Specific ---------------------------------------------------------