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

Unified Diff: content/browser/frame_host/navigation_handle_impl_browsertest.cc

Issue 2917133002: Perform redirect checks before OnReceivedRedirect in //net. (Closed)
Patch Set: nasko comments Created 3 years, 6 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
Index: content/browser/frame_host/navigation_handle_impl_browsertest.cc
diff --git a/content/browser/frame_host/navigation_handle_impl_browsertest.cc b/content/browser/frame_host/navigation_handle_impl_browsertest.cc
index e19b89a9d9f5003466912e25758350d9b7ebabbd..4bd9d5089e2c04c98ef85b692257f3051c635151 100644
--- a/content/browser/frame_host/navigation_handle_impl_browsertest.cc
+++ b/content/browser/frame_host/navigation_handle_impl_browsertest.cc
@@ -1307,7 +1307,7 @@ IN_PROC_BROWSER_TEST_F(NavigationHandleImplBrowserTest, ErrorCodeOnRedirect) {
ExecuteScript(shell(), base::StringPrintf("location.href = '%s';",
redirect_url.spec().c_str())));
same_tab_observer.Wait();
- EXPECT_EQ(net::ERR_ABORTED, observer.net_error_code());
+ EXPECT_EQ(net::ERR_UNSAFE_REDIRECT, observer.net_error_code());
}
// This class allows running tests with PlzNavigate enabled, regardless of

Powered by Google App Engine
This is Rietveld 408576698