Add a browsertest for bug 75604

content/browser/webkit_browsertest.cc

Index: content/browser/webkit_browsertest.cc
diff --git a/content/browser/webkit_browsertest.cc b/content/browser/webkit_browsertest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..d36a62af7c4e3c312c1905442029bfddfbbb9e51
--- /dev/null
+++ b/content/browser/webkit_browsertest.cc
@@ -0,0 +1,34 @@
+// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/ui/browser.h"
+#include "chrome/test/base/in_process_browser_test.h"
+#include "chrome/test/base/ui_test_utils.h"
+#include "content/browser/net/url_request_abort_on_end_job.h"
+#include "content/browser/tab_contents/tab_contents.h"
+
+typedef InProcessBrowserTest WebKitBrowserTest;
+
+const char kAsyncScriptThatAbortsOnEndPage[] =
+    "files/webkit/async_script_abort_on_end.html";
+
+// This is a browser test because it is hard to reproduce reliably in a
+// layout test without races. http://crbug.com/75604 deals with a request
+// for an async script which gets data in the response and immediately
+// after aborts. This test creates that condition, and it is passed
+// if chrome does not crash.
+
+IN_PROC_BROWSER_TEST_F(WebKitBrowserTest, AbortOnEnd) {
+  ASSERT_TRUE(test_server()->Start());
+  URLRequestAbortOnEndJob::AddUrlHandler();
+  GURL url = test_server()->GetURL(kAsyncScriptThatAbortsOnEndPage);
+
+  ui_test_utils::NavigateToURL(browser(), url);
+
+  // TODO(gavinp): after bug 75604 is fixed in WebKit, update expectations

[Paweł Hajdan Jr., 2011/10/27 08:52:58]

I think it'd be safer strategy to omit this check for now and when the bug is
fixed just insert EXPECT_FALSE. Up to you.

[gavinp, 2011/10/27 12:52:53]

I'd really like to see this test pass as it is on all platforms; this bug has
been notoriously hard to reproduce.  But you're right: I will, after this check
has rolled through, remove it before landing the WebKit change, then add it
inverted after, in order to make gardening that much easier.

+  // so that this test is not expected to crash.
+  TabContents* tab_contents = browser()->GetSelectedTabContents();
+  EXPECT_TRUE(tab_contents->is_crashed());
+}
+