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

Side by Side Diff: Source/modules/serviceworkers/ResponseInit.h

Issue 373613004: [ServiceWorker] Make Response 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 #ifndef ResponseInit_h 5 #ifndef ResponseInit_h
6 #define ResponseInit_h 6 #define ResponseInit_h
7 7
8 #include "bindings/core/v8/Dictionary.h" 8 #include "bindings/core/v8/Dictionary.h"
9 #include "modules/serviceworkers/HeaderMap.h" 9 #include "modules/serviceworkers/Headers.h"
10 #include "wtf/RefPtr.h" 10 #include "wtf/RefPtr.h"
11 11
12 namespace WebCore { 12 namespace WebCore {
13 13
14 struct ResponseInit { 14 struct ResponseInit {
15 ResponseInit() 15 ResponseInit()
16 : status(200) 16 : status(200)
17 , statusText("OK") 17 , statusText("OK")
18 { 18 {
19 } 19 }
20 explicit ResponseInit(const Dictionary& options) 20 explicit ResponseInit(const Dictionary& options)
21 : status(200) 21 : status(200)
22 , statusText("OK") 22 , statusText("OK")
23 { 23 {
24 options.get("status", status); 24 options.get("status", status);
25 // FIXME: Spec uses ByteString for statusText. http://crbug.com/347426 25 // FIXME: Spec uses ByteString for statusText. http://crbug.com/347426
26 options.get("statusText", statusText); 26 options.get("statusText", statusText);
27 options.get("headers", headers); 27 options.get("headers", headers);
28 if (!headers) {
29 options.get("headers", headersDictionary);
30 }
28 } 31 }
29 32
30 unsigned short status; 33 unsigned short status;
31 String statusText; 34 String statusText;
32 RefPtr<HeaderMap> headers; 35 RefPtr<Headers> headers;
36 Dictionary headersDictionary;
falken 2014/07/07 08:10:40 We haven't been following it but I guess members s
horo 2014/07/07 10:19:44 Done.
33 }; 37 };
34 38
35 } 39 }
36 40
37 #endif // ResponseInit_h 41 #endif // ResponseInit_h
OLDNEW
« Source/modules/serviceworkers/Response.cpp ('K') | « Source/modules/serviceworkers/Response.idl ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698