Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4577)

Unified Diff: chrome/renderer/pepper/pepper_uma_host.h

Issue 281853003: Whitelist libwidevinecdmadapter.so for pepper UMA reporting. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/renderer/pepper/pepper_uma_host.cc » ('j') | chrome/renderer/pepper/pepper_uma_host.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/renderer/pepper/pepper_uma_host.h
diff --git a/chrome/renderer/pepper/pepper_uma_host.h b/chrome/renderer/pepper/pepper_uma_host.h
index b417b632b5885402f2e02b1e1d0cf30b13c01855..07d2084b988109a05b5e963a5c4992035e4ce87c 100644
--- a/chrome/renderer/pepper/pepper_uma_host.h
+++ b/chrome/renderer/pepper/pepper_uma_host.h
@@ -8,6 +8,7 @@
#include <set>
#include <string>
+#include "base/files/file_path.h"
#include "ppapi/c/pp_instance.h"
#include "ppapi/c/pp_resource.h"
#include "ppapi/host/resource_host.h"
@@ -63,11 +64,14 @@ class PepperUMAHost : public ppapi::host::ResourceHost {
const GURL document_url_;
bool is_plugin_in_process_;
+ base::FilePath plugin_file_name_;
elijahtaylor1 2014/05/14 17:17:21 nit: it's a basename, not a filename
xhwang 2014/05/14 19:56:32 Done.
// Set of origins that can use UMA private APIs from NaCl.
std::set<std::string> allowed_origins_;
// Set of hashed histogram prefixes that can be used from this interface.
std::set<std::string> allowed_histogram_prefixes_;
+ // Set of plugin files names that are allowed to use this interface.
+ std::set<std::string> allowed_plugin_file_names_;
elijahtaylor1 2014/05/14 17:17:21 same comment about basename
bbudge 2014/05/14 19:31:00 nit: base::hash_set
xhwang 2014/05/14 19:56:32 Done.
xhwang 2014/05/14 19:56:32 Done. I keep the above two as std::set because (1)
DISALLOW_COPY_AND_ASSIGN(PepperUMAHost);
};
« no previous file with comments | « no previous file | chrome/renderer/pepper/pepper_uma_host.cc » ('j') | chrome/renderer/pepper/pepper_uma_host.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698