Split FaviconHelper in two: ImageLoadingHelper and FaviconHelper

content/common/icon_messages.h

Index: content/common/icon_messages.h
diff --git a/content/common/icon_messages.h b/content/common/icon_messages.h
index 9c466431991bce69c4531c90a21c586db11aa23e..482917e53ebd9d6811df8ee28b7ed3f6f0843b25 100644
--- a/content/common/icon_messages.h
+++ b/content/common/icon_messages.h
@@ -9,7 +9,6 @@
 #include "ipc/ipc_message.h"
 #include "ipc/ipc_message_macros.h"
 #include "ipc/ipc_param_traits.h"
-#include "third_party/skia/include/core/SkBitmap.h"
 
 #define IPC_MESSAGE_START IconMsgStart
 
@@ -22,24 +21,9 @@ IPC_STRUCT_TRAITS_END()
 
 // Messages sent from the browser to the renderer.
 
-// Requests the renderer to download the specified favicon image, decode it,
-// and send the image data back via IconHostMsg_DidDownloadFavicon.
-IPC_MESSAGE_ROUTED3(IconMsg_DownloadFavicon,
-                    int /* identifier for the request */,
-                    GURL /* URL of the image */,
-                    int /* Preferred favicon size. Passed on to
-                           IconHostMsg_DidDownloadFavicon, unused otherwise */)
-
 // Messages sent from the renderer to the browser.
 
 // Notification that the urls for the favicon of a site has been determined.
 IPC_MESSAGE_ROUTED2(IconHostMsg_UpdateFaviconURL,
                     int32 /* page_id */,
                     std::vector<content::FaviconURL> /* urls of the favicon */)

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

seems too much to have one messages file for one message. especially if we move
the favicon file in content\renderer into renderviewimpl and that this message
is dispatched in WebContentsImpl, this should just move to view_messages

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

moved to view_messages, removed FaviconHelper class.

-
-IPC_MESSAGE_ROUTED4(IconHostMsg_DidDownloadFavicon,
-                    int /* Identifier of the request */,
-                    GURL /* URL of the image */,
-                    int /* Preferred icon size passed to
-                           IconMsg_DownloadFavicon */,
-                    std::vector<SkBitmap> /* image_data */)