Make ResolveRelative work with all hierarchical URLs.

src/gurl_unittest.cc

 // Copyright 2007 Google Inc. All Rights Reserved.
 // Author: brettw@google.com (Brett Wilson)
 
 #include "googleurl/src/gurl.h"
 #include "googleurl/src/url_canon.h"
 #include "googleurl/src/url_test_utils.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 // Some implementations of base/basictypes.h may define ARRAYSIZE.
 // If it's not defined, we define it to the ARRAYSIZE_UNSAFE macro
@@ skipping 182 matching lines @@
     const char* expected;
   } resolve_cases[] = {
     {"http://www.google.com/", "foo.html", true, "http://www.google.com/foo.html"},
     {"http://www.google.com/", "http://images.google.com/foo.html", true, "http://images.google.com/foo.html"},
     {"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.
     {"data:blahblah", "http://google.com/", true, "http://google.com/"},
     {"data:blahblah", "http:google.com", true, "http://google.com/"},
-    {"data:/blahblah", "file.html", false, ""},
+    {"data:/blahblah", "file.html", true, "data:/file.html"},

[brettw, 2012/12/06 00:25:10]

This case seems fine to me. But it seems like we should have a different section
heading, since in this case "data" is being treated as quasi-standard.

[mkosiba (inactive), 2012/12/06 16:57:33]

Done.

       // 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"},
   };
 
   for (size_t i = 0; i < ARRAYSIZE(resolve_cases); i++) {
     // 8-bit code path.
     GURL input(resolve_cases[i].base);
     GURL output = input.Resolve(resolve_cases[i].relative);
     EXPECT_EQ(resolve_cases[i].expected_valid, output.is_valid()) << i;
@@ skipping 266 matching lines @@
 TEST(GURLTest, IsStandard) {
   GURL a("http:foo/bar");
   EXPECT_TRUE(a.IsStandard());
 
   GURL b("foo:bar/baz");
   EXPECT_FALSE(b.IsStandard());
 
   GURL c("foo://bar/baz");
   EXPECT_FALSE(c.IsStandard());
 }