Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1382)

Unified Diff: Source/modules/serviceworkers/FetchRequestData.cpp

Issue 329853012: [ServiceWorker] Make Request class better conformance with the spec. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: incorporated jochen's comment Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/modules/serviceworkers/FetchRequestData.cpp
diff --git a/Source/modules/serviceworkers/FetchRequestData.cpp b/Source/modules/serviceworkers/FetchRequestData.cpp
new file mode 100644
index 0000000000000000000000000000000000000000..0d4a97da2a24bb9026def4b2b524468c97348bee
--- /dev/null
+++ b/Source/modules/serviceworkers/FetchRequestData.cpp
@@ -0,0 +1,75 @@
+// 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/dom/ExecutionContext.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(ExecutionContext* context)
+{
+ RefPtr<FetchRequestData> request(adoptRef(new FetchRequestData()));
+ request->m_referrer.setClient(WebCore::Referrer(context->url().strippedForUseAsReferrer(), ReferrerPolicyDefault));
jochen (gone - plz use gerrit) 2014/07/07 15:31:52 if the executioncontext is a document, use Documen
horo 2014/07/08 02:05:59 Done.
+ return request.release();
+}
+
+PassRefPtr<FetchRequestData> FetchRequestData::create(const blink::WebServiceWorkerRequest& webRequest)
+{
+ RefPtr<FetchRequestData> request(adoptRef(new FetchRequestData()));
+ request->m_url = webRequest.url();
+ request->m_method = webRequest.method();
+ for (HTTPHeaderMap::const_iterator it = webRequest.headers().begin(); it != webRequest.headers().end(); ++it) {
+ request->m_headerList->append(it->key, it->value);
+ }
+ request->m_referrer.setURL(webRequest.referrer());
+ return request.release();
+}
+
+PassRefPtr<FetchRequestData> FetchRequestData::createRestrictedCopy(ExecutionContext* context, 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 = m_headerList->createCopy();
+ // FIXME: Support body.
+ request->m_origin = origin;
+ request->m_referrer.setClient(WebCore::Referrer(context->url().strippedForUseAsReferrer(), ReferrerPolicyDefault));
jochen (gone - plz use gerrit) 2014/07/07 15:31:52 same here. we could also make referrerPolicy a met
horo 2014/07/08 02:05:59 Done.
+ 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_unsafeRequestFlag(false)
+ , m_context(NullContext)
+ , m_sameOriginDataURLFlag(false)
+ , m_mode(NoCORSMode)
+ , m_credentials(OmitCredentials)
+ , m_responseTainting(BasicTainting)
+{
+}
+
+} // namespace WebCore

Powered by Google App Engine
This is Rietveld 408576698