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

Unified Diff: chrome/browser/prerender/prerender_browsertest.cc

Issue 143183009: When cross-site navigations are cancelled, delete the request being transferred (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: *Really* remove AbortTransfer Created 6 years, 10 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 | « no previous file | content/browser/frame_host/render_frame_host_manager.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/prerender/prerender_browsertest.cc
===================================================================
--- chrome/browser/prerender/prerender_browsertest.cc (revision 250813)
+++ chrome/browser/prerender/prerender_browsertest.cc (working copy)
@@ -3621,6 +3621,27 @@
content::SetBrowserClientForTesting(original_browser_client);
}
+// Checks that URLRequests for prerenders being aborted on cross-process
+// navigation from a server redirect are cleaned up, so they don't keep cache
+// entries locked.
+// See http://crbug.com/341134
+IN_PROC_BROWSER_TEST_F(PrerenderBrowserTest,
+ PrerenderCrossProcessServerRedirectNoHang) {
+ const char kDestPath[] = "files/prerender/prerender_page.html";
+ // Force everything to be a process swap.
+ SwapProcessesContentBrowserClient test_browser_client;
+ content::ContentBrowserClient* original_browser_client =
+ content::SetBrowserClientForTesting(&test_browser_client);
+
+ PrerenderTestURL(CreateServerRedirect(kDestPath), FINAL_STATUS_OPEN_URL, 0);
+
+ ui_test_utils::NavigateToURL(
+ browser(),
+ test_server()->GetURL(kDestPath));
+
+ content::SetBrowserClientForTesting(original_browser_client);
+}
+
// Checks that prerenders are aborted on cross-process navigation from
// a client redirect.
IN_PROC_BROWSER_TEST_F(PrerenderBrowserTest,
« no previous file with comments | « no previous file | content/browser/frame_host/render_frame_host_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698