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

Unified Diff: content/renderer/render_frame_impl.cc

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: Fixed savable_resources_browsertest.cc to use RenderFrame rather than RenderView. 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
« no previous file with comments | « content/renderer/render_frame_impl.h ('k') | content/renderer/render_view_impl.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index eda764382968862dc1c968cbffb2679c979520a4..9a467ff2070f215536b8aa15b9f09c8b7d3b097c 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -103,6 +103,7 @@
#include "content/renderer/render_widget_fullscreen_pepper.h"
#include "content/renderer/renderer_webapplicationcachehost_impl.h"
#include "content/renderer/renderer_webcolorchooser_impl.h"
+#include "content/renderer/savable_resources.h"
#include "content/renderer/screen_orientation/screen_orientation_dispatcher.h"
#include "content/renderer/shared_worker_repository.h"
#include "content/renderer/skia_benchmarking_extension.h"
@@ -1098,6 +1099,8 @@ bool RenderFrameImpl::OnMessageReceived(const IPC::Message& msg) {
OnTextTrackSettingsChanged)
IPC_MESSAGE_HANDLER(FrameMsg_PostMessageEvent, OnPostMessageEvent)
IPC_MESSAGE_HANDLER(FrameMsg_FailedNavigation, OnFailedNavigation)
+ IPC_MESSAGE_HANDLER(FrameMsg_GetSavableResourceLinks,
+ OnGetSavableResourceLinks)
#if defined(OS_ANDROID)
IPC_MESSAGE_HANDLER(FrameMsg_SelectPopupMenuItems, OnSelectPopupMenuItems)
#elif defined(OS_MACOSX)
@@ -4376,6 +4379,30 @@ WebNavigationPolicy RenderFrameImpl::DecidePolicyForNavigation(
return info.defaultPolicy;
}
+void RenderFrameImpl::OnGetSavableResourceLinks() {
+ std::vector<GURL> resources_list;
+ std::vector<GURL> referrer_urls_list;
+ std::vector<blink::WebReferrerPolicy> referrer_policies_list;
+ SavableResourcesResult result(&resources_list, &referrer_urls_list,
+ &referrer_policies_list);
+
+ if (!GetSavableResourceLinksForFrame(
+ frame_, &result, const_cast<const char**>(GetSavableSchemes()))) {
+ Send(new FrameHostMsg_SavableResourceLinksError(routing_id_));
+ return;
+ }
+
+ std::vector<Referrer> referrers_list;
+ CHECK_EQ(referrer_urls_list.size(), referrer_policies_list.size());
+ for (unsigned i = 0; i < referrer_urls_list.size(); ++i) {
+ referrers_list.push_back(
+ Referrer(referrer_urls_list[i], referrer_policies_list[i]));
+ }
+
+ Send(new FrameHostMsg_SavableResourceLinksResponse(
+ routing_id_, frame_->document().url(), resources_list, referrers_list));
+}
+
void RenderFrameImpl::OpenURL(WebFrame* frame,
const GURL& url,
const Referrer& referrer,
« no previous file with comments | « content/renderer/render_frame_impl.h ('k') | content/renderer/render_view_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698