Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(520)

Unified Diff: content/browser/renderer_host/resource_dispatcher_host_browsertest.cc

Issue 10693147: Merge 145731 - Remember to set did_defer_ to true when deferring the request from (Closed) Base URL: svn://svn.chromium.org/chrome/branches/1180/src/
Patch Set: Created 8 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « content/browser/renderer_host/cross_site_resource_handler.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/renderer_host/resource_dispatcher_host_browsertest.cc
===================================================================
--- content/browser/renderer_host/resource_dispatcher_host_browsertest.cc (revision 146131)
+++ content/browser/renderer_host/resource_dispatcher_host_browsertest.cc (working copy)
@@ -416,6 +416,32 @@
}
IN_PROC_BROWSER_TEST_F(ResourceDispatcherHostBrowserTest,
+ CrossSiteNavigationErrorPage2) {
+ ASSERT_TRUE(test_server()->Start());
+
+ GURL url(test_server()->GetURL("files/title2.html"));
+ CheckTitleTest(url, "Title Of Awesomeness", 1);
+
+ // Navigate to a new cross-site URL that results in an error page.
+ // TODO(creis): If this causes crashes or hangs, it might be for the same
+ // reason as ErrorPageTest::DNSError. See bug 1199491 and
+ // http://crbug.com/22877.
+ GURL failed_url = URLRequestFailedJob::GetMockHttpUrl(
+ net::ERR_NAME_NOT_RESOLVED);
+
+ ui_test_utils::NavigateToURLBlockUntilNavigationsComplete(
+ browser(), failed_url, 2);
+ EXPECT_NE(ASCIIToUTF16("Title Of Awesomeness"),
+ browser()->GetActiveWebContents()->GetTitle());
+
+ // Repeat navigation. We are testing that this completes.
+ ui_test_utils::NavigateToURLBlockUntilNavigationsComplete(
+ browser(), failed_url, 2);
+ EXPECT_NE(ASCIIToUTF16("Title Of Awesomeness"),
+ browser()->GetActiveWebContents()->GetTitle());
+}
+
+IN_PROC_BROWSER_TEST_F(ResourceDispatcherHostBrowserTest,
CrossOriginRedirectBlocked) {
// We expect the following URL requests from this test:
// 1- http://mock.http/cross-origin-redirect-blocked.html
« no previous file with comments | « content/browser/renderer_host/cross_site_resource_handler.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698