Add UMA reporting on failure to load ppapi plugins.

chrome/plugin/chrome_content_plugin_client.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/plugin/chrome_content_plugin_client.h"
 
+#include <sstream>
+
+#include "base/metrics/histogram.h"
+#include "chrome/common/chrome_constants.h"
+#include "third_party/widevine/cdm/widevine_cdm_common.h"
+
 #if defined(ENABLE_REMOTING)
 #include "base/files/file_path.h"
 #include "base/path_service.h"
 #include "content/public/common/content_paths.h"
 #include "media/base/media.h"
 #if defined(OS_WIN)
 #include "base/logging.h"
 #include "base/native_library.h"
 #elif defined(OS_POSIX) && !defined(OS_MACOSX) && defined(USE_NSS)
 #include "crypto/nss_util.h"
-#endif
-#endif
+#endif  // defined(OS_WIN)
+#endif  // defined(ENABLE_REMOTING)
 
 #if defined(OS_MACOSX)
 #include "base/mac/mac_util.h"
 #include "base/mac/scoped_cftyperef.h"
 #include "base/strings/sys_string_conversions.h"
 #include "grit/chromium_strings.h"
 #include "ui/base/l10n/l10n_util.h"
-#endif
+#endif  // defined(OS_MACOSX)
 
 namespace chrome {
 
 void ChromeContentPluginClient::PreSandboxInitialization() {
 #if defined(ENABLE_REMOTING)
 
   // Load crypto libraries for the Chromoting client plugin.
 #if defined(OS_POSIX) && !defined(OS_MACOSX) && defined(USE_NSS)
   // On platforms where we use system NSS libraries, the .so's must be loaded
   // before the sandbox is initialized.
@@ skipping 23 matching lines @@
   base::mac::ScopedCFTypeRef<CFStringRef> app_name(
       base::SysUTF16ToCFStringRef(
           l10n_util::GetStringUTF16(IDS_SHORT_PLUGIN_APP_NAME)));
   base::mac::ScopedCFTypeRef<CFStringRef> process_name(
       CFStringCreateWithFormat(kCFAllocatorDefault, NULL, CFSTR("%@ (%@)"),
                                cf_plugin_name.get(), app_name.get()));
   base::mac::SetProcessName(process_name);
 #endif
 }
 
+static void ReportPluginLoadUMA(const std::string& plugin_name,
+                                bool is_broker,
+                                int result) {
+  DCHECK_LT(result, content::ContentPluginClient::LOAD_RESULT_MAX);
+
+  std::ostringstream histogram_name;
+  histogram_name << "Plugin." << (is_broker ? "Broker" : "Plugin")
+                 << "LoadResult." << plugin_name;
+
+  // Note: This leaks memory, which is expected behavior.
+  base::HistogramBase* histogram =
+      base::LinearHistogram::FactoryGet(
+          histogram_name.str(),
+          1,
+          content::ContentPluginClient::LOAD_RESULT_MAX,
+          content::ContentPluginClient::LOAD_RESULT_MAX + 1,
+          base::HistogramBase::kUmaTargetedHistogramFlag);
+
+  histogram->Add(result);
+}
+
+void ChromeContentPluginClient::PluginLoaded(const base::FilePath& plugin_path,
+                                             bool is_broker,
+                                             PluginLoadResult result) {
+  base::FilePath::StringType plugin_name = plugin_path.BaseName().value();
+  if (plugin_name == kPepperFlashPluginFilename) {
+    ReportPluginLoadUMA("PepperFlash", is_broker, result);
+  } else if (plugin_name == kWidevineCdmPluginFileName) {
+    ReportPluginLoadUMA("WidevineCDM", is_broker, result);
+  }
+}
+
 }  // namespace chrome