diff options
Diffstat (limited to 'chrome/browser/renderer_host/resource_dispatcher_host_uitest.cc')
| -rw-r--r-- | chrome/browser/renderer_host/resource_dispatcher_host_uitest.cc | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/chrome/browser/renderer_host/resource_dispatcher_host_uitest.cc b/chrome/browser/renderer_host/resource_dispatcher_host_uitest.cc index 469dba7..3c22b13 100644 --- a/chrome/browser/renderer_host/resource_dispatcher_host_uitest.cc +++ b/chrome/browser/renderer_host/resource_dispatcher_host_uitest.cc @@ -193,11 +193,11 @@ TEST_F(ResourceDispatcherTest, CrossSiteOnunloadCookie) { // Tests that the onbeforeunload and onunload logic is shortcutted if the old // renderer is gone. In that case, we don't want to wait for the old renderer // to run the handlers. -// TODO(pinkerton): We need to disable this because the crash causes -// the OS CrashReporter process to kick in to analyze the poor dead renderer. -// Unfortunately, if the app isn't stripped of debug symbols, this takes about -// five minutes to complete and isn't conducive to quick turnarounds. As we -// don't currently strip the app on the build bots, this is bad times. +// We need to disable this on Mac because the crash causes the OS CrashReporter +// process to kick in to analyze the poor dead renderer. Unfortunately, if the +// app isn't stripped of debug symbols, this takes about five minutes to +// complete and isn't conducive to quick turnarounds. As we don't currently +// strip the app on the build bots, this is bad times. TEST_F(ResourceDispatcherTest, CrossSiteAfterCrash) { // This test only works in multi-process mode if (in_process_renderer()) |