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

Side by Side 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: 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "config.h" 5 #include "config.h"
6 #include "public/platform/WebServiceWorkerRequest.h" 6 #include "public/platform/WebServiceWorkerRequest.h"
7 7
8 namespace blink { 8 namespace blink {
9 9
10 class WebServiceWorkerRequestPrivate : public RefCounted<WebServiceWorkerRequest Private> { 10 class WebServiceWorkerRequestPrivate : public RefCounted<WebServiceWorkerRequest Private> {
11 public: 11 public:
12 WebServiceWorkerRequestPrivate() 12 WebServiceWorkerRequestPrivate()
13 : isReload(false) 13 : isReload(false) { }
14 {
15 }
16 WebURL url; 14 WebURL url;
jochen (gone - plz use gerrit) 2014/07/07 07:15:33 same here
horo 2014/07/07 09:31:38 Done.
17 WebString method; 15 WebString method;
18 HashMap<String, String> headers; 16 Vector<OwnPtr<WebServiceWorkerRequest::Header> > headerList;
17 WebString referrer;
19 bool isReload; 18 bool isReload;
20 }; 19 };
21 20
22 WebServiceWorkerRequest::WebServiceWorkerRequest() 21 WebServiceWorkerRequest::WebServiceWorkerRequest()
23 : m_private(adoptRef(new WebServiceWorkerRequestPrivate)) 22 : m_private(adoptRef(new WebServiceWorkerRequestPrivate))
24 { 23 {
25 } 24 }
26 25
27 void WebServiceWorkerRequest::reset() 26 void WebServiceWorkerRequest::reset()
28 { 27 {
(...skipping 20 matching lines...) Expand all
49 m_private->method = method; 48 m_private->method = method;
50 } 49 }
51 50
52 WebString WebServiceWorkerRequest::method() const 51 WebString WebServiceWorkerRequest::method() const
53 { 52 {
54 return m_private->method; 53 return m_private->method;
55 } 54 }
56 55
57 void WebServiceWorkerRequest::setHeader(const WebString& key, const WebString& v alue) 56 void WebServiceWorkerRequest::setHeader(const WebString& key, const WebString& v alue)
58 { 57 {
59 m_private->headers.set(key, value); 58 m_private->headerList.append(adoptPtr(new Header(key, value)));
jochen (gone - plz use gerrit) 2014/07/07 07:15:33 it should not be possible to directly set the refe
horo 2014/07/07 09:31:38 Done.
60 } 59 }
61 60
62 const HashMap<String, String>& WebServiceWorkerRequest::headers() const 61 const Vector<OwnPtr<WebServiceWorkerRequest::Header> >& WebServiceWorkerRequest: :headerList() const
63 { 62 {
64 return m_private->headers; 63 return m_private->headerList;
64 }
65
66 void WebServiceWorkerRequest::setReferrer(const WebString& referrer)
jochen (gone - plz use gerrit) 2014/07/07 07:15:33 referrers must always come with a referrer policy
horo 2014/07/07 09:31:38 Done.
67 {
68 m_private->referrer = referrer;
69 }
70
71 WebString WebServiceWorkerRequest::referrer() const
72 {
73 return m_private->referrer;
65 } 74 }
66 75
67 void WebServiceWorkerRequest::setIsReload(bool isReload) 76 void WebServiceWorkerRequest::setIsReload(bool isReload)
68 { 77 {
69 m_private->isReload = isReload; 78 m_private->isReload = isReload;
70 } 79 }
71 80
72 bool WebServiceWorkerRequest::isReload() const 81 bool WebServiceWorkerRequest::isReload() const
73 { 82 {
74 return m_private->isReload; 83 return m_private->isReload;
75 } 84 }
76 85
77 } // namespace blink 86 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698