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

Unified Diff: src/url_canon_unittest.cc

Issue 11367010: Make ResolveRelative work with all hierarchical URLs. (Closed) Base URL: http://git.chromium.org/external/google-url.git@master
Patch Set: Created 8 years 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 | « src/gurl_unittest.cc ('k') | src/url_util.cc » ('j') | src/url_util.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/url_canon_unittest.cc
diff --git a/src/url_canon_unittest.cc b/src/url_canon_unittest.cc
index fbc44b7a0b3630baa65d7cb3a0893ae7b788edc2..0c57f55e1b7b321814b0c8ca2a99a764dacc0c08 100644
--- a/src/url_canon_unittest.cc
+++ b/src/url_canon_unittest.cc
@@ -2036,6 +2036,9 @@ TEST(URLCanonTest, ResolveRelativeURL) {
{"filesystem:http://host/t/path", true, false, "./path2", true, true, true, "filesystem:http://host/t/path2"},
{"filesystem:http://host/t/path/", true, false, "path2", true, true, true, "filesystem:http://host/t/path/path2"},
{"filesystem:http://host/t/path", true, false, "filesystem:http:path2", true, false, false, NULL},
+ // Absolute URLs are still not relative to a non-standard base URL.
+ {"about:blank", false, false, "http://X/A", true, false, true, ""},
+ {"about:blank", false, false, "content://content.Provider/", true, false, true, ""},
};
for (size_t i = 0; i < ARRAYSIZE(rel_cases); i++) {
« no previous file with comments | « src/gurl_unittest.cc ('k') | src/url_util.cc » ('j') | src/url_util.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698