Move download messages from Renderer to Frame filter.

content/browser/frame_host/render_frame_message_filter.h

Index: content/browser/frame_host/render_frame_message_filter.h
diff --git a/content/browser/frame_host/render_frame_message_filter.h b/content/browser/frame_host/render_frame_message_filter.h
index 16190cf3d670b81531fa0ed06f67894dea6e7d07..84206ecd4fa4930860a0cafb1250f4bb1cf58392 100644
--- a/content/browser/frame_host/render_frame_message_filter.h
+++ b/content/browser/frame_host/render_frame_message_filter.h
@@ -31,6 +31,7 @@ class URLRequestContextGetter;
 namespace content {
 class BrowserContext;
 class PluginServiceImpl;
+struct Referrer;

[Charlie Reis, 2016/05/13 21:50:22]

nit: structs after classes, right?

[asanka, 2016/05/18 16:26:53]

Hm. I've also been sorting by identifier ignoring the class/struct, though I
can't find anything in the style guide that supports this.

[dcheng, 2016/05/24 07:18:08]

I also sort it by identifier, ignoring class name. IMO, this is consistent with
how #include / #imports are sorted by the path.

 class RenderWidgetHelper;
 class ResourceContext;
 struct WebPluginInfo;
@@ -52,6 +53,15 @@ class RenderFrameMessageFilter : public BrowserMessageFilter {
   // BrowserMessageFilter methods:
   bool OnMessageReceived(const IPC::Message& message) override;
 
+ protected:
+  // This method will be overridden by TestSaveImageFromDataURL class for test.
+  virtual void DownloadUrl(int render_view_id,
+                           int render_frame_id,
+                           const GURL& url,
+                           const Referrer& referrer,
+                           const base::string16& suggested_name,
+                           const bool use_prompt) const;
+
  private:
   class OpenChannelToPpapiPluginCallback;
   class OpenChannelToPpapiBrokerCallback;
@@ -85,6 +95,15 @@ class RenderFrameMessageFilter : public BrowserMessageFilter {
   void SendGetCookiesResponse(IPC::Message* reply_msg,
                               const std::string& cookies);
 
+  void OnDownloadUrl(int render_view_id,
+                     int render_frame_id,
+                     const GURL& url,
+                     const Referrer& referrer,
+                     const base::string16& suggested_name);
+  void OnSaveImageFromDataURL(int render_view_id,
+                              int render_frame_id,
+                              const std::string& url_str);
+
   void OnAre3DAPIsBlocked(int render_frame_id,
                           const GURL& top_origin_url,
                           ThreeDAPIType requester,