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

url/url_util.h

Index: url/url_util.h
diff --git a/url/url_util.h b/url/url_util.h
index a4b74b13e5db1fe6241b4a70ce747d81fd52e67f..e424c503757edbf2002f081bb2cc8499ffaebcd1 100644
--- a/url/url_util.h
+++ b/url/url_util.h
@@ -15,6 +15,8 @@
 #include "url/url_constants.h"
 #include "url/url_export.h"
 
+class GURL;
+
 namespace url {
 
 // Init ------------------------------------------------------------------------
@@ -153,6 +155,10 @@ URL_EXPORT bool GetStandardSchemeType(const char* spec,
                                       const Component& scheme,
                                       SchemeType* type);
 
+// Check whether the url is of the form about:blank, about:blank?foo or
+// about:blank/#foo.
+URL_EXPORT bool IsAboutBlank(const GURL& url);

[brettw, 2017/01/26 21:30:43]

url_util is not right right place for this function. This is really the
infrastructure that underlies both GURL and KURL, rather than functions that
operate on URLs. This is probably misleading given the name, but you can see
that there are no other functions dealing with GURL here.

Can you remove this function?

On https://codereview.chromium.org/2584513003/ I recommended adding a
"EqualsIgnoringRef function on GURL. Then you can change the constant to just
"about:blank" and the two callers can call EqualsIgnoringRef(kAboutBlankUrl");

I don't think this will be slower than the current implementation. Are there
cases where this implementation wouldn't work?

+
 // Hosts  ----------------------------------------------------------------------
 
 // Returns true if the |canonicalized_host| matches or is in the same domain as