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

Unified Diff: content/common/pepper_plugin_registry.cc

Issue 10917160: Add some logging to debug PPAPI permissions. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 3 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
Index: content/common/pepper_plugin_registry.cc
diff --git a/content/common/pepper_plugin_registry.cc b/content/common/pepper_plugin_registry.cc
index 259013b3f5b8b77af165965ce7ac878ea255bb27..edb262200b0e446a589162801c30be297c46ec9f 100644
--- a/content/common/pepper_plugin_registry.cc
+++ b/content/common/pepper_plugin_registry.cc
@@ -6,6 +6,7 @@
#include "base/command_line.h"
#include "base/file_util.h"
+#include "base/logging.h"
#include "base/native_library.h"
#include "base/string_split.h"
#include "base/string_util.h"
@@ -98,6 +99,11 @@ webkit::WebPluginInfo content::PepperPluginInfo::ToWebPluginInfo() const {
info.mime_types = mime_types;
info.pepper_permissions = permissions;
+ // TODO(brettw) bug 147507: remove this logging.
+ LOG(INFO) << "PepperPluginInfo::ToWebPluginInfo \""
+ << UTF16ToUTF8(info.path.LossyDisplayName()) << "\" "
+ << "permissions = " << permissions;
+
return info;
}
@@ -117,6 +123,10 @@ bool MakePepperPluginInfo(const webkit::WebPluginInfo& webplugin_info,
pepper_info->mime_types = webplugin_info.mime_types;
pepper_info->permissions = webplugin_info.pepper_permissions;
+ LOG(INFO) << "PepperPluginInfo::ToWebPluginInfo \""
+ << UTF16ToUTF8(pepper_info->path.LossyDisplayName()) << "\" "
+ << "permissions = " << pepper_info->permissions;
+
return true;
}
@@ -225,6 +235,11 @@ PepperPluginRegistry::PepperPluginRegistry() {
if (current.is_out_of_process)
continue; // Out of process plugins need no special pre-initialization.
+ // TODO(brettw) bug 147507: Remove this logging.
+ LOG(INFO) << "PepperPluginRegistry::PepperPluginRegistry \""
+ << UTF16ToUTF8(current.path.LossyDisplayName()) << "\" "
+ << " permissions =" << current.permissions;
+
scoped_refptr<webkit::ppapi::PluginModule> module =
new webkit::ppapi::PluginModule(current.name, current.path, this,
ppapi::PpapiPermissions(current.permissions));

Powered by Google App Engine
This is Rietveld 408576698