String::contains should use StringView.

third_party/WebKit/Source/platform/network/HTTPParsers.cpp

Index: third_party/WebKit/Source/platform/network/HTTPParsers.cpp
diff --git a/third_party/WebKit/Source/platform/network/HTTPParsers.cpp b/third_party/WebKit/Source/platform/network/HTTPParsers.cpp
index 593f095228b8991210d9d69e33263f33a1de6f08..8dbe206e855b49390f16350b4bc42f360783ef3c 100644
--- a/third_party/WebKit/Source/platform/network/HTTPParsers.cpp
+++ b/third_party/WebKit/Source/platform/network/HTTPParsers.cpp
@@ -186,7 +186,10 @@ bool isValidHTTPHeaderValue(const String& name)
     // FIXME: This should really match name against
     // field-value in section 4.2 of RFC 2616.
 
-    return name.containsOnlyLatin1() && !name.contains('\r') && !name.contains('\n') && !name.contains(static_cast<UChar>('\0'));

[esprehn, 2016/08/09 04:39:04]

There's no reason to upcast to UChar here, we just end up downcasting again
here:

https://cs.chromium.org/chromium/src/third_party/WebKit/Source/wtf/text/Strin...

+    return name.containsOnlyLatin1()
+        && !name.contains('\r')
+        && !name.contains('\n')
+        && !name.contains('\0');
 }
 
 // See RFC 7230, Section 3.2.