From 445e1041110859afe8eaf163035db8a34dc556ac Mon Sep 17 00:00:00 2001 From: "jochen@chromium.org" Date: Sat, 3 Dec 2011 21:03:15 +0000 Subject: Pass along the referrer policy (renderer side) BUG=105028 TEST=none (browser side still missing, does nothing yet) Review URL: http://codereview.chromium.org/8774050 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@112898 0039d316-1c4b-4281-b951-d872f2087c98 --- content/browser/renderer_host/render_view_host.cc | 4 ++-- content/browser/renderer_host/render_view_host.h | 5 +++-- content/browser/renderer_host/render_view_host_delegate.h | 3 ++- content/browser/renderer_host/test_render_view_host.cc | 4 ++-- 4 files changed, 9 insertions(+), 7 deletions(-) (limited to 'content/browser/renderer_host') diff --git a/content/browser/renderer_host/render_view_host.cc b/content/browser/renderer_host/render_view_host.cc index e03b6fa..3b539a3 100644 --- a/content/browser/renderer_host/render_view_host.cc +++ b/content/browser/renderer_host/render_view_host.cc @@ -896,7 +896,7 @@ void RenderViewHost::OnMsgNavigate(const IPC::Message& msg) { // the URL. To prevent this attack, we block the renderer from inserting // banned URLs into the navigation controller in the first place. FilterURL(policy, renderer_id, &validated_params.url); - FilterURL(policy, renderer_id, &validated_params.referrer); + FilterURL(policy, renderer_id, &validated_params.referrer.url); for (std::vector::iterator it(validated_params.redirects.begin()); it != validated_params.redirects.end(); ++it) { FilterURL(policy, renderer_id, &(*it)); @@ -1007,7 +1007,7 @@ void RenderViewHost::OnMsgToggleFullscreen(bool enter_fullscreen) { } void RenderViewHost::OnMsgOpenURL(const GURL& url, - const GURL& referrer, + const content::Referrer& referrer, WindowOpenDisposition disposition, int64 source_frame_id) { GURL validated_url(url); diff --git a/content/browser/renderer_host/render_view_host.h b/content/browser/renderer_host/render_view_host.h index 8d62b8f..1375e67 100644 --- a/content/browser/renderer_host/render_view_host.h +++ b/content/browser/renderer_host/render_view_host.h @@ -52,8 +52,9 @@ class ListValue; } namespace content { -class RenderViewHostObserver; struct FileChooserParams; +struct Referrer; +class RenderViewHostObserver; struct ShowDesktopNotificationHostMsgParams; } @@ -533,7 +534,7 @@ class CONTENT_EXPORT RenderViewHost : public RenderWidgetHost { void OnMsgContextMenu(const ContextMenuParams& params); void OnMsgToggleFullscreen(bool enter_fullscreen); void OnMsgOpenURL(const GURL& url, - const GURL& referrer, + const content::Referrer& referrer, WindowOpenDisposition disposition, int64 source_frame_id); void OnMsgDidContentsPreferredSizeChange(const gfx::Size& new_size); diff --git a/content/browser/renderer_host/render_view_host_delegate.h b/content/browser/renderer_host/render_view_host_delegate.h index e0faaa3..2da5aa0 100644 --- a/content/browser/renderer_host/render_view_host_delegate.h +++ b/content/browser/renderer_host/render_view_host_delegate.h @@ -42,6 +42,7 @@ class ListValue; namespace content { class BrowserContext; struct FileChooserParams; +struct Referrer; struct RendererPreferences; } @@ -273,7 +274,7 @@ class CONTENT_EXPORT RenderViewHostDelegate : public IPC::Channel::Listener { // The page wants to open a URL with the specified disposition. virtual void RequestOpenURL(const GURL& url, - const GURL& referrer, + const content::Referrer& referrer, WindowOpenDisposition disposition, int64 source_frame_id) {} diff --git a/content/browser/renderer_host/test_render_view_host.cc b/content/browser/renderer_host/test_render_view_host.cc index 9647381..3eb8ebd 100644 --- a/content/browser/renderer_host/test_render_view_host.cc +++ b/content/browser/renderer_host/test_render_view_host.cc @@ -26,7 +26,7 @@ void InitNavigateParams(ViewHostMsg_FrameNavigate_Params* params, content::PageTransition transition) { params->page_id = page_id; params->url = url; - params->referrer = GURL(); + params->referrer = content::Referrer(); params->transition = transition; params->redirects = std::vector(); params->should_update_history = false; @@ -87,7 +87,7 @@ void TestRenderViewHost::SendNavigateWithTransition( params.page_id = page_id; params.url = url; - params.referrer = GURL(); + params.referrer = content::Referrer(); params.transition = transition; params.redirects = std::vector(); params.should_update_history = true; -- cgit v1.1