Whitelist libwidevinecdmadapter.so for pepper UMA reporting.

chrome/renderer/pepper/pepper_uma_host.cc

 // Copyright 2014 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/renderer/pepper/pepper_uma_host.h"
 
 #include "base/metrics/histogram.h"
 #include "base/sha1.h"
 #include "base/strings/string_number_conversions.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/render_messages.h"
 #include "chrome/renderer/chrome_content_renderer_client.h"
+#include "content/public/renderer/pepper_plugin_instance.h"
 #include "content/public/renderer/render_thread.h"
 #include "content/public/renderer/renderer_ppapi_host.h"
 #include "extensions/common/constants.h"
 #include "extensions/common/extension.h"
 #include "ppapi/c/pp_errors.h"
 #include "ppapi/host/dispatch_host_message.h"
 #include "ppapi/host/host_message_context.h"
 #include "ppapi/host/ppapi_host.h"
 #include "ppapi/proxy/ppapi_messages.h"
 
 namespace {
 
 const char* const kPredefinedAllowedUMAOrigins[] = {
     "6EAED1924DB611B6EEF2A664BD077BE7EAD33B8F",  // see http://crbug.com/317833
     "4EB74897CB187C7633357C2FE832E0AD6A44883A"   // see http://crbug.com/317833
 };
 
 const char* const kWhitelistedHistogramPrefixes[] = {
     "CD190EA2B764EDF0BB97552A638D32072F3CFD41",  // see http://crbug.com/317833
 };
 
+const char* const kWhitelistedPluginFileNames[] = {
+    "libwidevinecdmadapter.so",  // see http://crbug.com/368743

[ddorwin, 2014/05/14 02:41:54]

Could/should we just add a histogram prefix?

[elijahtaylor1, 2014/05/14 17:17:21]

I'm assuming this is chromeos only or something?

[xhwang, 2014/05/14 17:41:28]

This is the name on linux and cros, and I only care about cros actually.

[ddorwin, 2014/05/14 19:41:19]

Should we ifdef it to make this clear? Other platforms would just have an empty
array (that might cause problems for arraysize, though).

[xhwang, 2014/05/14 19:56:32]

Yeah, arraysize doesn't work with empty array. I'll just keep this as is.

+};
+
 std::string HashPrefix(const std::string& histogram) {
   const std::string id_hash =
       base::SHA1HashString(histogram.substr(0, histogram.find('.')));
   DCHECK_EQ(id_hash.length(), base::kSHA1Length);
   return base::HexEncode(id_hash.c_str(), id_hash.length());
 }
 
 }  // namespace
 
 PepperUMAHost::PepperUMAHost(content::RendererPpapiHost* host,
                              PP_Instance instance,
                              PP_Resource resource)
     : ResourceHost(host->GetPpapiHost(), instance, resource),
       document_url_(host->GetDocumentURL(instance)),
       is_plugin_in_process_(host->IsRunningInProcess()) {
+  if (host->GetPluginInstance(instance)) {
+    plugin_file_name_ =
+        host->GetPluginInstance(instance)->GetModulePath().BaseName();
+  }
+
   for (size_t i = 0; i < arraysize(kPredefinedAllowedUMAOrigins); ++i)
     allowed_origins_.insert(kPredefinedAllowedUMAOrigins[i]);
   for (size_t i = 0; i < arraysize(kWhitelistedHistogramPrefixes); ++i)
     allowed_histogram_prefixes_.insert(kWhitelistedHistogramPrefixes[i]);
+  for (size_t i = 0; i < arraysize(kWhitelistedPluginFileNames); ++i)
+    allowed_plugin_file_names_.insert(kWhitelistedPluginFileNames[i]);
 }
 
 PepperUMAHost::~PepperUMAHost() {}
 
 int32_t PepperUMAHost::OnResourceMessageReceived(
     const IPC::Message& msg,
     ppapi::host::HostMessageContext* context) {
   IPC_BEGIN_MESSAGE_MAP(PepperUMAHost, msg)
   PPAPI_DISPATCH_HOST_RESOURCE_CALL(PpapiHostMsg_UMA_HistogramCustomTimes,
                                     OnHistogramCustomTimes);
@@ skipping 16 matching lines @@
   if (is_plugin_in_process_ && histogram.find("NaCl.") == 0) {
     return true;
   }
 
   if (IsPluginWhitelisted() &&
       allowed_histogram_prefixes_.find(HashPrefix(histogram)) !=
           allowed_histogram_prefixes_.end()) {
     return true;
   }
 
+  if (allowed_plugin_file_names_.find(plugin_file_name_.MaybeAsASCII()) !=
+      allowed_plugin_file_names_.end()) {
+    return true;
+  }
+
   LOG(ERROR) << "Host or histogram name is not allowed to use the UMA API.";
   return false;
 }
 
 #define RETURN_IF_BAD_ARGS(_min, _max, _buckets) \
   do {                                           \
     if (_min >= _max || _buckets <= 1)           \
       return PP_ERROR_BADARGUMENT;               \
   } while (0)
 
@@ skipping 74 matching lines @@
     ppapi::host::HostMessageContext* context) {
   if (!IsPluginWhitelisted())
     return PP_ERROR_NOACCESS;
   bool enabled = false;
   content::RenderThread::Get()->Send(
       new ChromeViewHostMsg_IsCrashReportingEnabled(&enabled));
   if (enabled)
     return PP_OK;
   return PP_ERROR_FAILED;
 }