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

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

Issue 373423002: Split Dictionary's get and convert into DictionaryHelper. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
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 | Annotate | Revision Log
« no previous file with comments | « Source/modules/serviceworkers/RequestInit.h ('k') | Source/modules/webmidi/MIDIOptions.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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/Headers.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 DictionaryHelper::get(options, "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 DictionaryHelper::get(options, "statusText", statusText);
27 options.get("headers", headers); 27 DictionaryHelper::get(options, "headers", headers);
28 if (!headers) { 28 if (!headers) {
29 options.get("headers", headersDictionary); 29 DictionaryHelper::get(options, "headers", headersDictionary);
30 } 30 }
31 } 31 }
32 32
33 unsigned short status; 33 unsigned short status;
34 String statusText; 34 String statusText;
35 RefPtr<Headers> headers; 35 RefPtr<Headers> headers;
36 Dictionary headersDictionary; 36 Dictionary headersDictionary;
37 }; 37 };
38 38
39 } 39 }
40 40
41 #endif // ResponseInit_h 41 #endif // ResponseInit_h
OLDNEW
« no previous file with comments | « Source/modules/serviceworkers/RequestInit.h ('k') | Source/modules/webmidi/MIDIOptions.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698