Fix some case-insensitive cases for StartsWith

net/base/net_util.cc

Index: net/base/net_util.cc
diff --git a/net/base/net_util.cc b/net/base/net_util.cc
index c59b3cd7ff4e91e1468ae5ca13a3c8110cc07882..3f36382aab2fcaeac65a0a2f2c4c4b688827f0f4 100644
--- a/net/base/net_util.cc
+++ b/net/base/net_util.cc
@@ -279,7 +279,8 @@ bool IsCanonicalizedHostCompliant(const std::string& host) {
 
 base::string16 StripWWW(const base::string16& text) {
   const base::string16 www(base::ASCIIToUTF16("www."));
-  return base::StartsWith(text, www, true) ? text.substr(www.length()) : text;
+  return base::StartsWith(text, www, base::CompareCase::INSENSITIVE_ASCII)

[sky, 2015/07/01 03:18:28]

It seems right that this code is insensitive, yet previously it was sensitive.
Is this an ok change? Meaning I'm not familiar with what callers of this expect.

[brettw, 2015/07/01 16:54:26]

This was a mistake, I didn't mean to change it. I fixed it.

If it's dealing with normal host names, they'll already be lower-cased so the
existing code doesn't jump out as necessarily wrong to me.

+      ? text.substr(www.length()) : text;
 }
 
 base::string16 StripWWWFromHost(const GURL& url) {