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..5f5a50c5994f8440846822dd99dfed067e049fa8 100644
--- a/chrome/common/render_messages.h
+++ b/chrome/common/render_messages.h
@@ -341,3 +341,5 @@ IPC_MESSAGE_CONTROL2(ChromeViewMsg_SetSearchURLs,
 IPC_SYNC_MESSAGE_CONTROL0_1(ChromeViewHostMsg_IsCrashReportingEnabled,
                             bool /* enabled */)
 #endif
+
+IPC_MESSAGE_ROUTED1(ChromeViewHostMsg_DownloadPDF, GURL /* url */)

[Lei Zhang, 2017/07/07 17:45:45]

Can you explain what this does?

[Lei Zhang, 2017/07/07 17:45:45]

Does this belong inside the #if BUILDFLAG(ENABLE_PLUGINS) section?

[tommycli, 2017/07/07 17:55:10]

For this particular one -- no - since Android needs this as well.

Which brings up another point -- we may need to move this to a different class
(other than PluginObserver) if that's not created on Android.

For now I'm okay with deferring making it work on !ENABLE_PLUGINS until a
followup.

[amberwon, 2017/07/07 20:44:22]

This declares the message (ChromeViewHostMsg_DownloadPDF), with a
GURL as an argument.

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

Can you describe what it does, as a comment in the code? Like all the existing
IPC messages above it.

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

Done.