PDF: Set a referrer when performing a SaveAs operation.

components/pdf/renderer/pepper_pdf_host.cc

Index: components/pdf/renderer/pepper_pdf_host.cc
diff --git a/components/pdf/renderer/pepper_pdf_host.cc b/components/pdf/renderer/pepper_pdf_host.cc
index eea157f86cd1fce64b917985a12804be899a3eb2..11b3b0642363c10ae288233e6edffccb5f72a682 100644
--- a/components/pdf/renderer/pepper_pdf_host.cc
+++ b/components/pdf/renderer/pepper_pdf_host.cc
@@ -153,11 +153,9 @@ int32_t PepperPDFHost::OnHostMsgSaveAs(
     return PP_ERROR_FAILED;
   GURL url = instance->GetPluginURL();
   content::RenderView* render_view = instance->GetRenderView();
-  blink::WebLocalFrame* frame =
-      render_view->GetWebView()->mainFrame()->toWebLocalFrame();
-  content::Referrer referrer = content::Referrer::SanitizeForRequest(
-      url, content::Referrer(frame->document().url(),
-                             frame->document().referrerPolicy()));
+  content::Referrer referrer;
+  referrer.url = url;

[jochen (gone - plz use gerrit), 2015/12/09 15:41:33]

can you still use the document's referrerPolicy?

[Lei Zhang, 2015/12/09 17:21:15]

In my limited testing, the document's referrer policy is always 1, AKA
WebReferrerPolicyDefault, which is the same as the value set by the Referrer.
But if you really prefer the document's policy, let me know.

[jochen (gone - plz use gerrit), 2015/12/10 15:34:05]

hum, yeah.

so in the future, we'll be able to set the referrer policy via http headers. But
I guess the document() is still just going to be the extension.

can you set the policy explicitly to WebReferrerPolicyDefault then? That'll make
it easier to spot this place in the future

[Lei Zhang, 2015/12/10 21:57:38]

Done.

+  referrer = content::Referrer::SanitizeForRequest(url, referrer);
   render_view->Send(
       new PDFHostMsg_PDFSaveURLAs(render_view->GetRoutingID(), url, referrer));
   return PP_OK;