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

Unified Diff: components/pdf/renderer/ppb_pdf_impl.cc

Issue 898613004: Sanitize referrers before we create them (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 10 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: components/pdf/renderer/ppb_pdf_impl.cc
diff --git a/components/pdf/renderer/ppb_pdf_impl.cc b/components/pdf/renderer/ppb_pdf_impl.cc
index aa0a33a05747ff4484f839353d301293d8aaf6b3..8a9f66035f520da2d384e96e3f1d67401167e744 100644
--- a/components/pdf/renderer/ppb_pdf_impl.cc
+++ b/components/pdf/renderer/ppb_pdf_impl.cc
@@ -242,8 +242,9 @@ void SaveAs(PP_Instance instance_id) {
content::RenderView* render_view = instance->GetRenderView();
blink::WebLocalFrame* frame =
render_view->GetWebView()->mainFrame()->toWebLocalFrame();
- content::Referrer referrer(frame->document().url(),
- frame->document().referrerPolicy());
+ content::Referrer referrer = content::Referrer::SanitizeForRequest(
+ url, content::Referrer(frame->document().url(),
+ frame->document().referrerPolicy()));
render_view->Send(
new PDFHostMsg_PDFSaveURLAs(render_view->GetRoutingID(), url, referrer));
}

Powered by Google App Engine
This is Rietveld 408576698