DoResolveRelative must not parse mailto schemes as path URLs.

url/url_util_unittest.cc

Index: url/url_util_unittest.cc
diff --git a/url/url_util_unittest.cc b/url/url_util_unittest.cc
index aba44e469157733236927072cdf2888443e5e8e7..da8fdc5d64eb24a69f5345fb6dae8aa81a8d4ad4 100644
--- a/url/url_util_unittest.cc
+++ b/url/url_util_unittest.cc
@@ -295,4 +295,25 @@ TEST(URLUtilTest, TestResolveRelativeWithNonStandardBase) {
   }
 }
 
+TEST(URLUtilTest, TestNoRefComponent) {
+  // The following address has base_parsed_authority.host, but

[brettw, 2014/12/01 20:15:09]

base_parsed_authority has no meaning in this context. Can you write this comment
in such a way that you don't need to understand all the code to know what's
being tested?

+  // it is still a mailto: scheme, and cannot be parsed as a path.
+  const char* base = "mailto://to/";
+  const char* rel = "any#body";
+
+  Parsed base_parsed;
+  ParsePathURL(base, strlen(base), false, &base_parsed);
+
+  std::string resolved;
+  StdStringCanonOutput output(&resolved);
+  Parsed resolved_parsed;
+
+  bool valid = ResolveRelative(base, strlen(base),
+                               base_parsed, rel,
+                               strlen(rel), NULL, &output,
+                               &resolved_parsed);
+  EXPECT_TRUE(valid);
+  EXPECT_FALSE(resolved_parsed.ref.is_valid());
+}
+
 }  // namespace url