[ServiceWorker] Make Request class better conformance with the spec.

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"
 #include "wtf/PassOwnPtr.h"
 
 namespace WebCore {
 
 PassRefPtr<FetchHeaderList> FetchHeaderList::create()
 {
     return adoptRef(new FetchHeaderList());
 }
 
+PassRefPtr<FetchHeaderList> FetchHeaderList::createCopy()
+{
+    RefPtr<FetchHeaderList> list(create());
+    for (size_t i = 0; i < m_headerList.size(); ++i) {

[jochen (gone - plz use gerrit), 2014/07/07 15:31:52]

nit. no { } for single line body, here and everywhere else

[horo, 2014/07/08 02:05:59]

Done.

+        list->append(m_headerList[i]->first, m_headerList[i]->second);
+    }
+    return list.release();
+}
+
 FetchHeaderList::FetchHeaderList()
 {
 }
 
 FetchHeaderList::~FetchHeaderList()
 {
 }
 
 void FetchHeaderList::append(const String& name, const String& value)
 {
@@ skipping 122 matching lines @@
 bool FetchHeaderList::isForbiddenHeaderName(const String& name)
 {
     // "A forbidden header name is a header names that is one of:
     //   `Accept-Charset`, `Accept-Encoding`, `Access-Control-Request-Headers`,
     //   `Access-Control-Request-Method`, `Connection`,
     //   `Content-Length, Cookie`, `Cookie2`, `Date`, `DNT`, `Expect`, `Host`,
     //   `Keep-Alive`, `Origin`, `Referer`, `TE`, `Trailer`,
     //   `Transfer-Encoding`, `Upgrade`, `User-Agent`, `Via`
     // or starts with `Proxy-` or `Sec-` (including when it is just `Proxy-` or
     // `Sec-`)."
-    return !XMLHttpRequest::isAllowedHTTPHeader(name);
+    return !XMLHttpRequest::isAllowedHTTPHeader(name) || equalIgnoringCase(name, "DNT");
 }
 
 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