Commit 9f072b52 authored by mseaborn@chromium.org's avatar mseaborn@chromium.org

NaCl: Remove duplicated logging for the process exit status

This leaves in the logging that happens in the destructor,
~NaClProcessHost(), which also logs non-crash exits.

The duplication was introduced by a bad merge in r112877 ("Simplify
BrowserChildProcessHost in preparation for refactoring it").

BUG=none
TEST=none

Review URL: https://chromiumcodereview.appspot.com/10389229

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@138215 0039d316-1c4b-4281-b951-d872f2087c98
parent a3f3a6b5
...@@ -529,12 +529,6 @@ bool NaClProcessHost::OnMessageReceived(const IPC::Message& msg) { ...@@ -529,12 +529,6 @@ bool NaClProcessHost::OnMessageReceived(const IPC::Message& msg) {
return handled; return handled;
} }
void NaClProcessHost::OnProcessCrashed(int exit_code) {
std::string message = base::StringPrintf(
"NaCl process exited with status %i (0x%x)", exit_code, exit_code);
LOG(ERROR) << message;
}
void NaClProcessHost::OnProcessLaunched() { void NaClProcessHost::OnProcessLaunched() {
if (!StartWithLaunchedProcess()) if (!StartWithLaunchedProcess())
delete this; delete this;
......
...@@ -79,7 +79,6 @@ class NaClProcessHost : public content::BrowserChildProcessHostDelegate { ...@@ -79,7 +79,6 @@ class NaClProcessHost : public content::BrowserChildProcessHostDelegate {
// BrowserChildProcessHostDelegate implementation: // BrowserChildProcessHostDelegate implementation:
virtual bool OnMessageReceived(const IPC::Message& msg) OVERRIDE; virtual bool OnMessageReceived(const IPC::Message& msg) OVERRIDE;
virtual void OnProcessCrashed(int exit_code) OVERRIDE;
virtual void OnProcessLaunched() OVERRIDE; virtual void OnProcessLaunched() OVERRIDE;
void OnResourcesReady(); void OnResourcesReady();
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment