PDF: Add UMA to track embedded PDF triggered drive-by downloads.

chrome/renderer/chrome_content_renderer_client.cc

Index: chrome/renderer/chrome_content_renderer_client.cc
diff --git a/chrome/renderer/chrome_content_renderer_client.cc b/chrome/renderer/chrome_content_renderer_client.cc
index 51e8e616838f365f3447c388de5b5f6caa004056..8cd60bae3e0e05095d8ca7bacdd20358577f0ae7 100644
--- a/chrome/renderer/chrome_content_renderer_client.cc
+++ b/chrome/renderer/chrome_content_renderer_client.cc
@@ -146,6 +146,7 @@
 #endif
 
 #if BUILDFLAG(ENABLE_PLUGINS)
+#include "chrome/common/pdf_uma.h"

[tommycli, 2017/05/25 18:16:38]

I'm leaving these usages ENABLE_PLUGINS for now, but will add the android
(NonLoadablePlaceholder) calls in a separate CL, where i'll make this outside of
ENABLE_PLUGINS.

[Lei Zhang, 2017/05/25 21:11:08]

Is that why pdf_uma* are in the general file list in chrome/common/BUILD.gn, and
not in the if (enable_plugins) section?

[tommycli, 2017/05/25 21:20:48]

Yes exactly. And in chrome_download_manager_delegate, we use them even if
plugins is off too.

 #include "chrome/common/plugin_utils.h"
 #include "chrome/renderer/plugins/chrome_plugin_placeholder.h"
 #include "chrome/renderer/plugins/power_saver_info.h"
@@ -824,10 +825,10 @@ WebPlugin* ChromeContentRendererClient::CreatePlugin(
           bool is_main_frame_plugin_document =
               render_frame->IsMainFrame() &&
               render_frame->GetWebFrame()->GetDocument().IsPluginDocument();
-          PluginUMAReporter::ReportPDFLoadStatus(
+          ReportPDFLoadStatus(
               is_main_frame_plugin_document
-                  ? PluginUMAReporter::LOADED_FULL_PAGE_PDF_WITH_PDFIUM
-                  : PluginUMAReporter::LOADED_EMBEDDED_PDF_WITH_PDFIUM);
+                  ? PDFLoadStatus::kLoadedFullPagePdfWithPdfium
+                  : PDFLoadStatus::kLoadedEmbeddedPdfWithPdfium);
         }
 
         // Delay loading plugins if prerendering.
@@ -871,9 +872,8 @@ WebPlugin* ChromeContentRendererClient::CreatePlugin(
         PluginUMAReporter::GetInstance()->ReportPluginDisabled(orig_mime_type,
                                                                url);
         if (info.name == ASCIIToUTF16(ChromeContentClient::kPDFPluginName)) {
-          PluginUMAReporter::ReportPDFLoadStatus(
-              PluginUMAReporter::
-                  SHOWED_DISABLED_PLUGIN_PLACEHOLDER_FOR_EMBEDDED_PDF);
+          ReportPDFLoadStatus(
+              PDFLoadStatus::kShowedDisabledPluginPlaceholderForEmbeddedPdf);
         }
 
         placeholder = create_blocked_plugin(