Normalize and update the header value checks to RFC 7230 for Fetch

Source/core/fetch/FetchUtils.cpp

 // Copyright 2014 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 "config.h"
 #include "core/fetch/FetchUtils.h"
 
 #include "platform/network/HTTPHeaderMap.h"
 #include "platform/network/HTTPParsers.h"
 #include "wtf/HashSet.h"
 #include "wtf/Threading.h"
 #include "wtf/text/AtomicString.h"
 #include "wtf/text/WTFString.h"
 
 namespace blink {
 
+static bool isHTTPWhitespace(UChar chr)

[yhirano, 2015/09/09 04:41:50]

Please place this function in the unnamed namespace below instead of using
"static".

[shiva.jm, 2015/09/10 10:10:26]

Done.

+{
+    return chr <= ' ' && (chr == ' ' || chr == '\n' || chr == '\t' || chr == '\r');

[hiroshige, 2015/09/09 07:01:48]

"chr <= ' ' && " is not needed.

[shiva.jm, 2015/09/10 10:10:26]

Done.

+}
+
 namespace {
 
 class ForbiddenHeaderNames {
     WTF_MAKE_NONCOPYABLE(ForbiddenHeaderNames); WTF_MAKE_FAST_ALLOCATED(ForbiddenHeaderNames);
 public:
     bool has(const String& name) const
     {
         return m_fixedNames.contains(name)
             || name.startsWith(m_proxyHeaderPrefix, TextCaseInsensitive)
             || name.startsWith(m_secHeaderPrefix, TextCaseInsensitive);
@@ skipping 155 matching lines @@
     for (const auto& known : methods) {
         if (equalIgnoringCase(method, known)) {
             // Don't bother allocating a new string if it's already all
             // uppercase.
             return method == known ? method : known;
         }
     }
     return method;
 }
 
+String FetchUtils::normalizeHeaderValue(const String& value)
+{
+    // https://fetch.spec.whatwg.org/#concept-header-value-normalize
+    // Strip leading and trailing whitespace from header value.
+    // HTTP whitespace bytes are 0x09, 0x0A, 0x0D, and 0x20.
+
+    return value.stripWhiteSpace(isHTTPWhitespace);
+}
+
 } // namespace blink