OOPIFs: Transitioning Get/Send...SavableResourceLinks away from RenderViewHost.

content/common/frame_messages.h

Index: content/common/frame_messages.h
diff --git a/content/common/frame_messages.h b/content/common/frame_messages.h
index 28f27db5245bbcb1507885ac71efa58a66915e4e..e8d272ad9820b8bd366d829b25a9e99c1a3921cc 100644
--- a/content/common/frame_messages.h
+++ b/content/common/frame_messages.h
@@ -1022,6 +1022,13 @@ IPC_MESSAGE_ROUTED0(FrameHostMsg_DispatchLoad)
 IPC_MESSAGE_ROUTED1(FrameHostMsg_RouteMessageEvent,
                     FrameMsg_PostMessage_Params)
 
+// Messages for getting savable resource links from frames into the browser.

[nasko, 2015/09/02 23:45:55]

nit: s/into/to/

[Łukasz Anforowicz, 2015/09/03 16:59:57]

N/A after moving the messages (and having a separate comment for each).

+IPC_MESSAGE_ROUTED0(FrameMsg_GetSavableResourceLinks)

[nasko, 2015/09/02 23:45:55]

Please keep FrameMsg messages (which means browser->renderer) with the rest of
the FrameMsg ones. The file is split in two parts and are grouped by direction
of the message. FrameHostMsg are renderer->browser.

[Łukasz Anforowicz, 2015/09/03 16:59:57]

Done.

+IPC_MESSAGE_ROUTED0(FrameHostMsg_NonSavableResponse)

[nasko, 2015/09/02 23:45:55]

Put a comment explaining what this message is about. Same for the one below.

[Łukasz Anforowicz, 2015/09/03 16:59:57]

Done.

+IPC_MESSAGE_ROUTED2(FrameHostMsg_SavableResourceLinksResponse,
+                    std::vector<GURL> /* savable resource links */,
+                    std::vector<content::Referrer> /* referrers */)
+
 #if defined(OS_MACOSX) || defined(OS_ANDROID)
 
 // Message to show/hide a popup menu using native controls.