Do not sanitize about:blank/#foo & about:blank?foo

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 a706f5a08a04bbfeca8a9cf936ad4bee664a079f..616e2ae6bfe7d38527623232ab56ac6348e53edc 100644
--- a/url/url_util_unittest.cc
+++ b/url/url_util_unittest.cc
@@ -417,4 +417,18 @@ TEST(URLUtilTest, TestDomainIs) {
   }
 }
 
+TEST(URLUtilTest, IsAboutBlank) {
+  const std::string kAboutBlankURLS[] = {"about:blank", "about:blank?foo",

[Charlie Reis, 2017/01/20 20:31:02]

Minor nit: Lowercase s at the end (kAboutBlankURLs).  Same below.

[clamy, 2017/01/23 12:26:27]

Done.

+                                         "about:blank/#foo",
+                                         "about:blank?foo#foo"};
+  for (const auto& url : kAboutBlankURLS)
+    EXPECT_TRUE(IsAboutBlank(GURL(url)));
+
+  const std::string kNotAboutBlankURLS[] = {
+      "http:blank",      "about:blan",          "about://blank",
+      "about:blank/foo", "about://:8000/blank", "about://foo:foo@/blank"};

[Charlie Reis, 2017/01/20 20:31:02]

A few more to add:
foo@about:blank
foo:bar@about:blank
about:blank:8000

[clamy, 2017/01/23 12:26:27]

Done.

+  for (const auto& url : kNotAboutBlankURLS)
+    EXPECT_FALSE(IsAboutBlank(GURL(url)));
+}
+
 }  // namespace url