[ServiceWorker] A header list contains lowercased header names

Source/modules/serviceworkers/FetchHeaderList.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 "FetchHeaderList.h"
 
 #include "core/fetch/CrossOriginAccessControl.h"
 #include "core/xml/XMLHttpRequest.h"
 #include "platform/network/HTTPParsers.h"
@@ skipping 10 matching lines @@
 {
 }
 
 FetchHeaderList::~FetchHeaderList()
 {
 }
 
 void FetchHeaderList::append(const String& name, const String& value)
 {
     // "To append a name/value (|name|/|value|) pair to a header list (|list|),
-    // append a new header whose name is |name| and value is |value|, to
-    // |list|."
-    m_headerList.append(adoptPtr(new Header(name, value)));
+    // append a new header whose name is |name|, byte lowercased, and value is
+    // |value|, to |list|."
+    m_headerList.append(adoptPtr(new Header(name.lower(), value)));
 }
 
 void FetchHeaderList::set(const String& name, const String& value)
 {
     // "To set a name/value (|name|/|value|) pair in a header list (|list|), run
     // these steps:
-    // 1. If there are any headers in |list| whose name is |name|, set the value
+    // 1. Byte lowercase |name|.
+    // 2. If there are any headers in |list| whose name is |name|, set the value
     //    of the first such header to |value| and remove the others.
-    // 2. Otherwise, append a new header whose name is |name| and value is
+    // 3. Otherwise, append a new header whose name is |name| and value is
     //    |value|, to |list|."
+    const String lowercasedName = name.lower();
     for (size_t i = 0; i < m_headerList.size(); ++i) {
-        if (m_headerList[i]->first == name) {
+        if (m_headerList[i]->first == lowercasedName) {
             m_headerList[i]->second = value;
             for (size_t j = i + 1; j < m_headerList.size(); ) {
-                if (m_headerList[j]->first == name)
+                if (m_headerList[j]->first == lowercasedName)
                     m_headerList.remove(j);
                 else
                     ++j;
             }
             return;
         }
     }
-    m_headerList.append(adoptPtr(new Header(name, value)));
+    m_headerList.append(adoptPtr(new Header(lowercasedName, value)));
 }
 
 size_t FetchHeaderList::size() const
 {
     return m_headerList.size();
 }
 
 void FetchHeaderList::remove(const String& name)
 {
+    // To delete a name (|name|) from a header list (|list|), remove all headers
+    // whose name is |name|, byte lowercased, from |list|.

[falken, 2014/07/02 08:12:12]

enclose this spec quote in quotes

[horo, 2014/07/02 08:42:46]

Done.

+    const String lowercasedName = name.lower();
     for (size_t i = 0; i < m_headerList.size(); ) {
-        if (m_headerList[i]->first == name)
+        if (m_headerList[i]->first == lowercasedName)
             m_headerList.remove(i);
         else
             ++i;
     }
 }
 
 bool FetchHeaderList::get(const String& name, String& result) const
 {
+    const String lowercasedName = name.lower();
     for (size_t i = 0; i < m_headerList.size(); ++i) {
-        if (m_headerList[i]->first == name) {
+        if (m_headerList[i]->first == lowercasedName) {
             result = m_headerList[i]->second;
             return true;
         }
     }
     return false;
 }
 
 void FetchHeaderList::getAll(const String& name, Vector<String>& result) const
 {
+    const String lowercasedName = name.lower();
     result.clear();
     for (size_t i = 0; i < m_headerList.size(); ++i) {
-        if (m_headerList[i]->first == name)
+        if (m_headerList[i]->first == lowercasedName)
             result.append(m_headerList[i]->second);
     }
 }
 
 bool FetchHeaderList::has(const String& name) const
 {
+    const String lowercasedName = name.lower();
     for (size_t i = 0; i < m_headerList.size(); ++i) {
-        if (m_headerList[i]->first == name)
+        if (m_headerList[i]->first == lowercasedName)
             return true;
     }
     return false;
 }
 
 void FetchHeaderList::clearList()
 {
     m_headerList.clear();
 }
 
 bool FetchHeaderList::isValidHeaderName(const String& name)
 {
-    // FIXME: According to the spec (http://fetch.spec.whatwg.org/) should we
-    // accept all characters in the range 0x00 to 0x7F?
+    // "A name is a case-insensitive byte sequence that matches the field-name
+    // token production."
     return isValidHTTPToken(name);
 }
 
 bool FetchHeaderList::isValidHeaderValue(const String& value)
 {
     // "A value is a byte sequence that matches the field-value token production
     // and contains no 0x0A or 0x0D bytes."
-    return (value.find('\r') == kNotFound) && (value.find('\n') == kNotFound);
+    return isValidHTTPHeaderValue(value);

[falken, 2014/07/02 08:12:12]

FYI isValidHTTPHeaderValue has a FIXME to follow the HTTP spec, though I don't
know exactly what's different. Anyway I presume this function is used in other
places for HTTP parsing so it's OK to reuse here, but you might want to
investigate.

 }
 
 bool FetchHeaderList::isSimpleHeader(const String& name, const String& value)
 {
     // "A simple header is a header whose name is either one of `Accept`,
     // `Accept-Language`, and `Content-Language`, or whose name is
     // `Content-Type` and value, once parsed, is one of
     // `application/x-www-form-urlencoded`, `multipart/form-data`, and
     // `text/plain`."
     if (equalIgnoringCase(name, "accept")
@@ skipping 25 matching lines @@
 }
 
 bool FetchHeaderList::isForbiddenResponseHeaderName(const String& name)
 {
     // "A forbidden response header name is a header name that is one of:
     // `Set-Cookie`, `Set-Cookie2`"
     return equalIgnoringCase(name, "set-cookie") || equalIgnoringCase(name, "set-cookie2");
 }
 
 } // namespace WebCore