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

Unified Diff: components/nacl/renderer/plugin/module_ppapi.cc

Issue 1631413002: NaCl cleanup: Remove not-very-useful debug logging in the plugin code (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Rebase Created 4 years, 11 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 | components/nacl/renderer/plugin/plugin.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/nacl/renderer/plugin/module_ppapi.cc
diff --git a/components/nacl/renderer/plugin/module_ppapi.cc b/components/nacl/renderer/plugin/module_ppapi.cc
index 0c5661dd3321605f1c1b5d7facc8c16f84ba4c6c..cb0ae3cd25cabec55cb28d324a1c3ce1847a7284 100644
--- a/components/nacl/renderer/plugin/module_ppapi.cc
+++ b/components/nacl/renderer/plugin/module_ppapi.cc
@@ -16,16 +16,12 @@ namespace plugin {
ModulePpapi::ModulePpapi() : pp::Module(),
init_was_successful_(false),
private_interface_(NULL) {
- MODULE_PRINTF(("ModulePpapi::ModulePpapi (this=%p)\n",
- static_cast<void*>(this)));
}
ModulePpapi::~ModulePpapi() {
if (init_was_successful_) {
NaClNrdAllModulesFini();
}
- MODULE_PRINTF(("ModulePpapi::~ModulePpapi (this=%p)\n",
- static_cast<void*>(this)));
}
bool ModulePpapi::Init() {
@@ -34,8 +30,6 @@ bool ModulePpapi::Init() {
GetBrowserInterface(PPB_NACL_PRIVATE_INTERFACE));
if (NULL == private_interface_) {
- MODULE_PRINTF(("ModulePpapi::Init failed: "
- "GetBrowserInterface returned NULL\n"));
return false;
}
SetNaClInterface(private_interface_);
@@ -57,13 +51,7 @@ bool ModulePpapi::Init() {
}
pp::Instance* ModulePpapi::CreateInstance(PP_Instance pp_instance) {
- MODULE_PRINTF(("ModulePpapi::CreateInstance (pp_instance=%" NACL_PRId32
- ")\n",
- pp_instance));
- Plugin* plugin = new Plugin(pp_instance);
- MODULE_PRINTF(("ModulePpapi::CreateInstance (return %p)\n",
- static_cast<void* >(plugin)));
- return plugin;
+ return new Plugin(pp_instance);
}
} // namespace plugin
@@ -72,7 +60,6 @@ pp::Instance* ModulePpapi::CreateInstance(PP_Instance pp_instance) {
namespace pp {
Module* CreateModule() {
- MODULE_PRINTF(("CreateModule ()\n"));
return new plugin::ModulePpapi();
}
« no previous file with comments | « no previous file | components/nacl/renderer/plugin/plugin.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698