diff options
author | phajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-04-06 13:03:47 +0000 |
---|---|---|
committer | phajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-04-06 13:03:47 +0000 |
commit | e80bea9a23243ddf8f58c0358df8118159800b53 (patch) | |
tree | 43f7da3680f9b6045df619184d629e7347ca3c75 /chrome/browser/nacl_host | |
parent | 9bf97140e3fb566cda9436a5f2ba33df01f21d6d (diff) | |
download | chromium_src-e80bea9a23243ddf8f58c0358df8118159800b53.zip chromium_src-e80bea9a23243ddf8f58c0358df8118159800b53.tar.gz chromium_src-e80bea9a23243ddf8f58c0358df8118159800b53.tar.bz2 |
Make DidProcessCrash a bit more solid and accurate on Windows
I hit the NOTREACHED() inside it while debugging an unrelated problem.
Also, now there is one less special case in this function's contract.
TEST=none
BUG=38048
Review URL: http://codereview.chromium.org/1315009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43706 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/nacl_host')
-rw-r--r-- | chrome/browser/nacl_host/nacl_process_host.cc | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/chrome/browser/nacl_host/nacl_process_host.cc b/chrome/browser/nacl_host/nacl_process_host.cc index 1e3bcff..2648b3e 100644 --- a/chrome/browser/nacl_host/nacl_process_host.cc +++ b/chrome/browser/nacl_host/nacl_process_host.cc @@ -118,10 +118,8 @@ void NaClProcessHost::OnProcessLaunchedByBroker(base::ProcessHandle handle) { } bool NaClProcessHost::DidChildCrash() { - if (running_on_wow64_) { - bool child_exited; - return base::DidProcessCrash(&child_exited, handle()); - } + if (running_on_wow64_) + return base::DidProcessCrash(NULL, handle()); return ChildProcessHost::DidChildCrash(); } |