DescriptionFix a bug where killing pages doesn't yield a sad-tab
The determination of whether or not to put up a sad tab page is made
based on the termination status of the renderer process.
The termination status of the renderer process is checked when
the channel between the browser and the renderer stops working.
There is a race here because there is no reason to believe that the
renderer process has finished terminating at the time that the
browser detects that its channel to the renderer is dead.
If renderer process is believed to be still running at the moment when
the decision to put up the sad tab page is taken, then no sad tab will
be put up, the view of the renderer will be removed (eventually), and
a transparent window will result.
This bug was previously fixed on Linux with with
https://chromiumcodereview.appspot.com/11316261
this patch expands that fix to Mac.
In particular, if the renderer process is known to be dead, wait for the
process to terminate before taking its termination status. To be sure
that the browser does not wait forever on a renderer that is not exiting,
send a kill signal to the renderer before doing the wait.
BUG=167538
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=223526
Patch Set 1 #Patch Set 2 : Clean up whitespace #
Messages
Total messages: 8 (0 generated)
|