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

chrome/common/render_messages.h

Index: chrome/common/render_messages.h
diff --git a/chrome/common/render_messages.h b/chrome/common/render_messages.h
index 625a4b3a9f4b84d418571580656469758535661f..d554d5c7d3b34948f7b40b0fe90d32fc628a3ccf 100644
--- a/chrome/common/render_messages.h
+++ b/chrome/common/render_messages.h
@@ -341,3 +341,7 @@ IPC_MESSAGE_CONTROL2(ChromeViewMsg_SetSearchURLs,
 IPC_SYNC_MESSAGE_CONTROL0_1(ChromeViewHostMsg_IsCrashReportingEnabled,
                             bool /* enabled */)
 #endif
+
+// Tells the browser to open a PDF file in a new tab. Used when plugins are

[Lei Zhang, 2017/07/07 22:47:58]

plugins are disabled -> there is no PDF viewer available ?

[amberwon, 2017/07/11 18:08:05]

I believe just Chrome's PDF Viewer is disabled, but other PDF
viewers may be available.

[Lei Zhang, 2017/07/12 00:51:37]

If one disables the internal PDF Viewer and installs a Rob's PDF Viewer
extension, [1] then a page that embeds a PDF will still display the PDF, and the
pdf_plugin.html placeholder won't show. 

By plugins, do you really mean PPAPI plugins, or just anything that can be
plugged in, which includes extensions?

[1]
https://chrome.google.com/webstore/detail/pdf-viewer/oemmndcbldboiebfnladdacb...

[amberwon, 2017/07/12 19:12:20]

Yes I mean PPAPI plugins. I changed the comment to reflect the case where a user
has an PDF viewer extension as well.

[Lei Zhang, 2017/07/12 22:57:30]

So what was wrong with my initial suggestion?

[tommycli, 2017/07/12 23:05:15]

I suggested the new version and did not see your original suggestion.

The original suggestion seems just fine as well and probably more concise.

[amberwon, 2017/07/13 20:33:54]

Done.

+// disabled and user clicks to view PDF.
+IPC_MESSAGE_ROUTED1(ChromeViewHostMsg_OpenPDF, GURL /* url */)