diff options
| author | cpu@chromium.org <cpu@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-09-16 23:07:05 +0000 |
|---|---|---|
| committer | cpu@chromium.org <cpu@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-09-16 23:07:05 +0000 |
| commit | a3a7e2c07a1dcac13b004257982bb174e681693c (patch) | |
| tree | 4eaaaff74dd1ca3c96d93c15d1b9123bbdd77e83 /content/common/child_process_host.cc | |
| parent | c0bcb01f4d832c493e320198ac31f31cc067fc5d (diff) | |
| download | chromium_src-a3a7e2c07a1dcac13b004257982bb174e681693c.zip chromium_src-a3a7e2c07a1dcac13b004257982bb174e681693c.tar.gz chromium_src-a3a7e2c07a1dcac13b004257982bb174e681693c.tar.bz2 | |
Trying to reland r101435 : Better account crashes for the BrowserChildProcessHost
original code review:
http://codereview.chromium.org/7888070
In this change we remove renderer_host piece, which I landed already and fixes the
problem which was a missing break (line 170, see patch set 2).
TBR=jam
BUG=96059
TEST=see bug
Review URL: http://codereview.chromium.org/7919016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101595 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'content/common/child_process_host.cc')
| -rw-r--r-- | content/common/child_process_host.cc | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/content/common/child_process_host.cc b/content/common/child_process_host.cc index bcaca6d..b8a3b95 100644 --- a/content/common/child_process_host.cc +++ b/content/common/child_process_host.cc @@ -225,6 +225,10 @@ void ChildProcessHost::OnChildDied() { delete this; } +void ChildProcessHost::OnChildDisconnected() { + OnChildDied(); +} + void ChildProcessHost::ShutdownStarted() { } @@ -301,7 +305,7 @@ void ChildProcessHost::ListenerHook::OnChannelError() { host_->filters_[i]->OnChannelError(); // This will delete host_, which will also destroy this! - host_->OnChildDied(); + host_->OnChildDisconnected(); } void ChildProcessHost::ForceShutdown() { |