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

url/url_util_unittest.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <stddef.h>
 
 #include "base/macros.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "url/third_party/mozilla/url_parse.h"
 #include "url/url_canon.h"
@@ skipping 399 matching lines @@
     SCOPED_TRACE(testing::Message() << "(host, domain): ("
                                     << test_case.canonicalized_host << ", "
                                     << test_case.lower_ascii_domain << ")");
 
     EXPECT_EQ(
         test_case.expected_domain_is,
         DomainIs(test_case.canonicalized_host, test_case.lower_ascii_domain));
   }
 }
 
+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