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

Unified Diff: url/url_canon_unittest.cc

Issue 23835019: Support URL fragment resolution againt non-hierarchical schemes (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix net & webview tag tests Created 7 years, 2 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: url/url_canon_unittest.cc
diff --git a/url/url_canon_unittest.cc b/url/url_canon_unittest.cc
index 1b59a00c89375af73cf178ac1401e1f41321c04a..9ce6d3053be5a6eef165a553a8ed8aedc84b1e95 100644
--- a/url/url_canon_unittest.cc
+++ b/url/url_canon_unittest.cc
@@ -1824,7 +1824,7 @@ TEST(URLCanonTest, CanonicalizePathURL) {
} path_cases[] = {
{"javascript:", "javascript:"},
{"JavaScript:Foo", "javascript:Foo"},
- {":\":This /is interesting;?#", ":\":This /is interesting;?#"},
+ {":\":This /is interesting;?#", ":\":This%20/is%20interesting;?#"},
};
for (size_t i = 0; i < ARRAYSIZE(path_cases); i++) {
@@ -1848,8 +1848,8 @@ TEST(URLCanonTest, CanonicalizePathURL) {
// When we end with a colon at the end, there should be no path.
if (path_cases[i].input[url_len - 1] == ':') {
- EXPECT_EQ(0, out_parsed.path.begin);
- EXPECT_EQ(-1, out_parsed.path.len);
+ EXPECT_EQ(0, out_parsed.GetContent().begin);
+ EXPECT_EQ(-1, out_parsed.GetContent().len);
}
}
}

Powered by Google App Engine
This is Rietveld 408576698