Index: content/browser/ppapi_plugin_process_host.cc |
=================================================================== |
--- content/browser/ppapi_plugin_process_host.cc (revision 239266) |
+++ content/browser/ppapi_plugin_process_host.cc (working copy) |
@@ -117,6 +117,7 @@ |
const base::FilePath& profile_data_directory) { |
PpapiPluginProcessHost* plugin_host = new PpapiPluginProcessHost( |
info, profile_data_directory); |
+ DCHECK(plugin_host); |
if (plugin_host->Init(info)) |
return plugin_host; |
@@ -253,8 +254,10 @@ |
} |
std::string channel_id = process_->GetHost()->CreateChannel(); |
- if (channel_id.empty()) |
+ if (channel_id.empty()) { |
+ VLOG(1) << "Could not create pepper host channel."; |
return false; |
+ } |
const CommandLine& browser_command_line = *CommandLine::ForCurrentProcess(); |
CommandLine::StringType plugin_launcher = |
@@ -267,8 +270,10 @@ |
int flags = ChildProcessHost::CHILD_NORMAL; |
#endif |
base::FilePath exe_path = ChildProcessHost::GetChildPath(flags); |
- if (exe_path.empty()) |
+ if (exe_path.empty()) { |
+ VLOG(1) << "Pepper plugin exe path is empty."; |
return false; |
+ } |
CommandLine* cmd_line = new CommandLine(exe_path); |
cmd_line->AppendSwitchASCII(switches::kProcessType, |
@@ -357,10 +362,12 @@ |
} |
void PpapiPluginProcessHost::OnProcessLaunched() { |
+ VLOG(2) << "ppapi plugin process launched."; |
host_impl_->set_plugin_process_handle(process_->GetHandle()); |
} |
void PpapiPluginProcessHost::OnProcessCrashed(int exit_code) { |
+ VLOG(1) << "ppapi plugin process crashed."; |
PluginServiceImpl::GetInstance()->RegisterPluginCrash(plugin_path_); |
} |
@@ -391,8 +398,8 @@ |
// Called when the browser <--> plugin channel has an error. This normally |
// means the plugin has crashed. |
void PpapiPluginProcessHost::OnChannelError() { |
- DVLOG(1) << "PpapiPluginProcessHost" << (is_broker_ ? "[broker]" : "") |
- << "::OnChannelError()"; |
+ VLOG(1) << "PpapiPluginProcessHost" << (is_broker_ ? "[broker]" : "") |
+ << "::OnChannelError()"; |
// We don't need to notify the renderers that were communicating with the |
// plugin since they have their own channels which will go into the error |
// state at the same time. Instead, we just need to notify any renderers |