| commit | 165a7f28c672a899924051df4b29682756d51858 | |
| author | nasko <nasko@chromium.org> | |
| Wed, 1 Oct 2014 20:00:38 +0000 (1 13:00 -0700) | ||
| committer | Commit bot <commit-bot@chromium.org> | |
| Wed, 1 Oct 2014 20:00:59 +0000 (1 20:00 +0000) | ||
| tree | 587d597eb46049800eb97e7e5f22cff5579f9fd2 | treesnapshot (tar.gz zip) |
| parent | 36c8d5e285ab85c424151ae207797d17dc7d3d00 | commitdiff |
Move notifications to after the process has indeed closed.
Renderer process closed notification implies that the process is already gone. In reality, this is not always the case since calling ProcessDied(false) is supposed to tear down the process. It however sends the notification before doing so. This patch moves the notifications to after the process is closed and the IPC channel is torn down.
BUG=
Review URL: https://codereview.chromium.org/613173004
Cr-Commit-Position: refs/heads/master@{#297698}
Renderer process closed notification implies that the process is already gone. In reality, this is not always the case since calling ProcessDied(false) is supposed to tear down the process. It however sends the notification before doing so. This patch moves the notifications to after the process is closed and the IPC channel is torn down.
BUG=
Review URL: https://codereview.chromium.org/613173004
Cr-Commit-Position: refs/heads/master@{#297698}
| content/browser/renderer_host/render_process_host_impl.cc | diffblobblamehistory |