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

Unified Diff: Source/platform/exported/WebServiceWorkerRequest.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/platform/exported/WebServiceWorkerRequest.cpp
diff --git a/Source/platform/exported/WebServiceWorkerRequest.cpp b/Source/platform/exported/WebServiceWorkerRequest.cpp
index 70fdc7d861ccd4d9d52b3dc570701a864b45e414..28e0fe49bae19ae86d66af09d9e53a8c30e7e826 100644
--- a/Source/platform/exported/WebServiceWorkerRequest.cpp
+++ b/Source/platform/exported/WebServiceWorkerRequest.cpp
@@ -10,13 +10,12 @@ namespace blink {
class WebServiceWorkerRequestPrivate : public RefCounted<WebServiceWorkerRequestPrivate> {
public:
WebServiceWorkerRequestPrivate()
- : isReload(false)
- {
- }
- WebURL url;
- WebString method;
- HashMap<String, String> headers;
- bool isReload;
+ : m_isReload(false) { }
+ WebURL m_url;
+ WebString m_method;
+ WebCore::HTTPHeaderMap m_headers;
+ WebCore::Referrer m_referrer;
+ bool m_isReload;
};
WebServiceWorkerRequest::WebServiceWorkerRequest()
@@ -36,42 +35,54 @@ void WebServiceWorkerRequest::assign(const WebServiceWorkerRequest& other)
void WebServiceWorkerRequest::setURL(const WebURL& url)
{
- m_private->url = url;
+ m_private->m_url = url;
}
WebURL WebServiceWorkerRequest::url() const
{
- return m_private->url;
+ return m_private->m_url;
}
void WebServiceWorkerRequest::setMethod(const WebString& method)
{
- m_private->method = method;
+ m_private->m_method = method;
}
WebString WebServiceWorkerRequest::method() const
{
- return m_private->method;
+ return m_private->m_method;
}
void WebServiceWorkerRequest::setHeader(const WebString& key, const WebString& value)
{
- m_private->headers.set(key, value);
+ if (equalIgnoringCase(key, "referer"))
+ return;
+ m_private->m_headers.add(key, value);
}
-const HashMap<String, String>& WebServiceWorkerRequest::headers() const
+const WebCore::HTTPHeaderMap& WebServiceWorkerRequest::headers() const
{
- return m_private->headers;
+ return m_private->m_headers;
+}
+
+void WebServiceWorkerRequest::setReferrer(const WebString& referrer, WebReferrerPolicy referrerPolicy)
+{
+ m_private->m_referrer = WebCore::Referrer(referrer, static_cast<WebCore::ReferrerPolicy>(referrerPolicy));
+}
+
+const WebCore::Referrer& WebServiceWorkerRequest::referrer() const
+{
+ return m_private->m_referrer;
}
void WebServiceWorkerRequest::setIsReload(bool isReload)
{
- m_private->isReload = isReload;
+ m_private->m_isReload = isReload;
}
bool WebServiceWorkerRequest::isReload() const
{
- return m_private->isReload;
+ return m_private->m_isReload;
}
} // namespace blink

Powered by Google App Engine
This is Rietveld 408576698