Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(930)

Unified Diff: content/common/frame_messages.h

Issue 1308113008: OOPIFs: Transitioning Get/Send...SavableResourceLinks away from RenderViewHost. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@page-serialization-test
Patch Set: Addressed most of CR feedback from Nasko. Created 5 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/common/frame_messages.h
diff --git a/content/common/frame_messages.h b/content/common/frame_messages.h
index 28f27db5245bbcb1507885ac71efa58a66915e4e..4176b364812fbb2c7cd585f977011ec099817352 100644
--- a/content/common/frame_messages.h
+++ b/content/common/frame_messages.h
@@ -643,6 +643,10 @@ IPC_MESSAGE_ROUTED4(FrameMsg_FailedNavigation,
bool, /* stale_copy_in_cache */
int /* error_code */)
+// Request to enumerate and return links to all savable resources in the frame
+// (this covers only the immediate frame / doesn't cover subframes).
nasko 2015/09/03 18:17:02 nit: Drop the parenthesis and prefix with Note:
Łukasz Anforowicz 2015/09/03 19:41:24 Done.
+IPC_MESSAGE_ROUTED0(FrameMsg_GetSavableResourceLinks)
+
#if defined(ENABLE_PLUGINS)
// Notifies the renderer of updates to the Plugin Power Saver origin whitelist.
IPC_MESSAGE_ROUTED1(FrameMsg_UpdatePluginContentOriginWhitelist,
@@ -1022,6 +1026,16 @@ IPC_MESSAGE_ROUTED0(FrameHostMsg_DispatchLoad)
IPC_MESSAGE_ROUTED1(FrameHostMsg_RouteMessageEvent,
FrameMsg_PostMessage_Params)
+// Response to FrameMsg_GetSavableResourceLinks.
+IPC_MESSAGE_ROUTED2(FrameHostMsg_SavableResourceLinksResponse,
+ std::vector<GURL> /* savable resource links */,
+ std::vector<content::Referrer> /* referrers */)
+
+// Response to FrameMsg_GetSavableResourceLinks in case the frame contains
+// non-savable content (i.e. from a non-savable schema) or if there were
nasko 2015/09/03 18:17:02 nit: s/schema/scheme/
Łukasz Anforowicz 2015/09/03 19:41:24 Oh man, really? I've been spelling it wrong my wh
nasko 2015/09/04 16:03:21 Well, there is schema too, but it is in other area
+// errors gathering the links.
+IPC_MESSAGE_ROUTED0(FrameHostMsg_SavableResourceLinksError)
+
#if defined(OS_MACOSX) || defined(OS_ANDROID)
// Message to show/hide a popup menu using native controls.

Powered by Google App Engine
This is Rietveld 408576698