Click to Open PDF: Open PDF once View PDF is clicked.

chrome/renderer/plugins/pdf_plugin_placeholder.cc

Index: chrome/renderer/plugins/pdf_plugin_placeholder.cc
diff --git a/chrome/renderer/plugins/pdf_plugin_placeholder.cc b/chrome/renderer/plugins/pdf_plugin_placeholder.cc
index 6e20cfbce38d5d95d2839659ae16a40719d2095a..bf95ca60fa22eb3b08c2493c8727a562bd132491 100644
--- a/chrome/renderer/plugins/pdf_plugin_placeholder.cc
+++ b/chrome/renderer/plugins/pdf_plugin_placeholder.cc
@@ -4,7 +4,9 @@
 
 #include "chrome/renderer/plugins/pdf_plugin_placeholder.h"
 
+#include "chrome/common/render_messages.h"
 #include "chrome/grit/renderer_resources.h"
+#include "content/public/renderer/render_thread.h"
 #include "gin/object_template_builder.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/base/webui/jstemplate_builder.h"
@@ -25,6 +27,8 @@ PDFPluginPlaceholder* PDFPluginPlaceholder::CreatePDFPlaceholder(
       ResourceBundle::GetSharedInstance().GetRawDataResource(
           IDR_PDF_PLUGIN_HTML));
   base::DictionaryValue values;
+  GURL url = params.url;

[Lei Zhang, 2017/07/07 21:08:20]

Can this be written as:

values.SetString("fileName", GURL(url).ExtractFileName()); ?

[amberwon, 2017/07/07 21:23:09]

Yes, it can. Done.

+  values.SetString("fileName", url.ExtractFileName());
   std::string html_data = webui::GetI18nTemplateHtml(template_html, &values);
   return new PDFPluginPlaceholder(render_frame, params, html_data);
 }
@@ -41,5 +45,6 @@ gin::ObjectTemplateBuilder PDFPluginPlaceholder::GetObjectTemplateBuilder(
 }
 
 void PDFPluginPlaceholder::DownloadPDFCallback() {
-  // TODO(amberwon): Implement starting PDF download.
+  content::RenderThread::Get()->Send(
+      new ChromeViewHostMsg_DownloadPDF(routing_id(), GetPluginParams().url));
 }