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

Unified Diff: content/browser/browser_side_navigation_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/browser_side_navigation_browsertest.cc
diff --git a/content/browser/browser_side_navigation_browsertest.cc b/content/browser/browser_side_navigation_browsertest.cc
index 3dc6e81145437d0d8e341d21b769356765ed0c16..c3ac0cf3a22549137423bb1691c5fac31ff76498 100644
--- a/content/browser/browser_side_navigation_browsertest.cc
+++ b/content/browser/browser_side_navigation_browsertest.cc
@@ -420,11 +420,14 @@ IN_PROC_BROWSER_TEST_F(BrowserSideNavigationBrowserTest,
// The expectation is that about:blank was loaded and the virtual URL is set
// to the URL that was blocked.
+ //
+ // TODO(nasko): Now that the error commits on the previous URL, the blocked
+ // navigation logic is no longer needed. https://crbug.com/723796
EXPECT_EQ(1, controller.GetLastCommittedEntryIndex());
EXPECT_FALSE(
controller.GetLastCommittedEntry()->GetURL().SchemeIs(url::kDataScheme));
- EXPECT_TRUE(controller.GetLastCommittedEntry()->GetVirtualURL().SchemeIs(
- url::kDataScheme));
+ EXPECT_EQ(redirect_to_blank_url,
+ controller.GetLastCommittedEntry()->GetVirtualURL());
EXPECT_EQ(url::kAboutBlankURL,
controller.GetLastCommittedEntry()->GetURL().spec());
}
« no previous file with comments | « components/error_page/common/localized_error.cc ('k') | content/browser/frame_host/data_url_navigation_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698