diff options
| author | evan@chromium.org <evan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-05-06 00:49:56 +0000 |
|---|---|---|
| committer | evan@chromium.org <evan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-05-06 00:49:56 +0000 |
| commit | 24e5219bf22811b32ca821c35c85f31e2697160a (patch) | |
| tree | 17cf5e509401835dfa7264a50cf9a55d36b10180 /chrome/browser/renderer_host/resource_dispatcher_host_uitest.cc | |
| parent | fe52784b32117de90cc6218aa34485ca0bc5ee59 (diff) | |
| download | chromium_src-24e5219bf22811b32ca821c35c85f31e2697160a.zip chromium_src-24e5219bf22811b32ca821c35c85f31e2697160a.tar.gz chromium_src-24e5219bf22811b32ca821c35c85f31e2697160a.tar.bz2 | |
Uncomment the crash checks in UITest::TearDown since CountFilesCreatedAfter
has been implemented for POSIX environment.
BUG=9833
Review URL: http://codereview.chromium.org/99360
Patch from Albert Bachand <albertb@google.com>.
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@15364 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/renderer_host/resource_dispatcher_host_uitest.cc')
| -rw-r--r-- | chrome/browser/renderer_host/resource_dispatcher_host_uitest.cc | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/chrome/browser/renderer_host/resource_dispatcher_host_uitest.cc b/chrome/browser/renderer_host/resource_dispatcher_host_uitest.cc index f3b5a07..ea12275 100644 --- a/chrome/browser/renderer_host/resource_dispatcher_host_uitest.cc +++ b/chrome/browser/renderer_host/resource_dispatcher_host_uitest.cc @@ -193,7 +193,7 @@ TEST_F(ResourceDispatcherTest, CrossSiteOnunloadCookie) { ASSERT_STREQ("foo", value_result.c_str()); } -#if !defined(OS_MACOSX) +#if defined(OS_WIN) // 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. @@ -202,6 +202,8 @@ TEST_F(ResourceDispatcherTest, CrossSiteOnunloadCookie) { // 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. +// TODO(albertb): We need to disable this on Linux as well since +// crash_service.exe hasn't been ported yet. TEST_F(ResourceDispatcherTest, CrossSiteAfterCrash) { // This test only works in multi-process mode if (in_process_renderer()) |