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

Source/modules/serviceworkers/FetchRequestData.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 "FetchRequestData.h"
+
+#include "core/fetch/ResourceLoaderOptions.h"
+#include "core/loader/ThreadableLoader.h"
+#include "modules/serviceworkers/FetchHeaderList.h"
+#include "platform/network/ResourceRequest.h"
+#include "public/platform/WebServiceWorkerRequest.h"
+
+namespace WebCore {
+
+PassRefPtr<FetchRequestData> FetchRequestData::create()
+{
+    return adoptRef(new FetchRequestData());
+}
+
+PassRefPtr<FetchRequestData> FetchRequestData::create(const blink::WebServiceWorkerRequest& webRequest)
+{
+    RefPtr<FetchRequestData> request = adoptRef(new FetchRequestData());

[yhirano, 2014/07/02 01:53:06]

Let's use |create|.

[horo, 2014/07/02 03:33:04]

Done.

+    request->m_url = webRequest.url();
+    request->m_method = webRequest.method();
+    for (size_t i = 0; i < webRequest.headerList().size(); ++i) {
+        request->m_headerList->append(webRequest.headerList()[i]->first, webRequest.headerList()[i]->second);
+    }
+    request->m_referrer.setURL(webRequest.referrer());
+    return request.release();
+}
+
+PassRefPtr<FetchRequestData> FetchRequestData::createRestrictedCopy(bool isServiceWorker, PassRefPtr<SecurityOrigin> origin) const
+{
+    // "To make a restricted copy of a request |request|, run these steps:"
+    // "1. Let |r| be a new request whose url is |request|'s url, method is
+    // |request|'s method, header list is a copy of |request|'s header list,
+    // body is a tee of |request|'s body, client is entry settings object's
+    // global object, origin is entry settings object's origin, referrer is
+    // |client|, context is |connect|, mode is |request|'s mode, and credentials
+    //  mode is |request|'s credentials mode."
+    RefPtr<FetchRequestData> request = adoptRef(new FetchRequestData());
+    request->m_url = m_url;
+    request->m_method = m_method;
+    request->m_headerList = request->m_headerList->createCopy();
+    // FIXME: Support body.
+    request->m_clientIsServiceWorker = isServiceWorker;

[yhirano, 2014/07/02 01:53:06]

Can you tell me what this flag is?

[horo, 2014/07/02 03:33:04]

Humm..
I think we don't need this.
Deleted.

+    request->m_origin = origin;
+    request->m_referrer.setClient();
+    request->m_context = ConnectContext;
+    request->m_mode = m_mode;
+    request->m_credentials = m_credentials;
+    // "2. Return |r|."
+    return request.release();
+}
+
+FetchRequestData::~FetchRequestData()
+{
+}
+
+FetchRequestData::FetchRequestData()
+    : m_method("GET")
+    , m_headerList(FetchHeaderList::create())
+    , m_clientIsServiceWorker(false)
+    , m_context(NullContext)
+    , m_sameOriginDataURLFlag(false)
+    , m_mode(NoCORSMode)
+    , m_credentials(OmitCredentials)
+    , m_responseTainting(BasicTainting)
+{
+}
+
+} // namespace WebCore