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

Unified Diff: src/gurl_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 | « no previous file | src/url_canon_unittest.cc » ('j') | src/url_util.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/gurl_unittest.cc
diff --git a/src/gurl_unittest.cc b/src/gurl_unittest.cc
index f62267ff981627833df4ea3e1b5d70361d2e4498..670d2dffaab6246c1495139e42ac8dc23f4a1d18 100644
--- a/src/gurl_unittest.cc
+++ b/src/gurl_unittest.cc
@@ -197,10 +197,9 @@ TEST(GURLTest, Resolve) {
{"http://www.google.com/blah/bloo?c#d", "../../../hello/./world.html?a#b", true, "http://www.google.com/hello/world.html?a#b"},
{"http://www.google.com/foo#bar", "#com", true, "http://www.google.com/foo#com"},
{"http://www.google.com/", "Https:images.google.com", true, "https://images.google.com/"},
- // Unknown schemes are not standard.
+ // A non-standard base can be replaced with a standard absolute URL.
{"data:blahblah", "http://google.com/", true, "http://google.com/"},
{"data:blahblah", "http:google.com", true, "http://google.com/"},
- {"data:/blahblah", "file.html", false, ""},
// Filesystem URLs have different paths to test.
{"filesystem:http://www.google.com/type/", "foo.html", true, "filesystem:http://www.google.com/type/foo.html"},
{"filesystem:http://www.google.com/type/", "../foo.html", true, "filesystem:http://www.google.com/type/foo.html"},
« no previous file with comments | « no previous file | src/url_canon_unittest.cc » ('j') | src/url_util.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698