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

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

Issue 373423002: Split Dictionary's get and convert into DictionaryHelper. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Fixed LICENSE and windows build 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
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 RegistrationOptionList_h 5 #ifndef RegistrationOptionList_h
6 #define RegistrationOptionList_h 6 #define RegistrationOptionList_h
7 7
8 #include "bindings/core/v8/Dictionary.h" 8 #include "bindings/core/v8/Dictionary.h"
9 #include "bindings/core/v8/DictionaryHelper.h"
9 10
10 namespace WebCore { 11 namespace WebCore {
11 12
12 struct RegistrationOptionList { 13 struct RegistrationOptionList {
13 explicit RegistrationOptionList(const Dictionary& options) 14 explicit RegistrationOptionList(const Dictionary& options)
14 : scope("/*") 15 : scope("/*")
15 { 16 {
16 // FIXME: Should be ScalarValueString. http://crbug.com/379009 17 // FIXME: Should be ScalarValueString. http://crbug.com/379009
17 options.get("scope", scope); 18 DictionaryHelper::get(options, "scope", scope);
18 } 19 }
19 20
20 String scope; 21 String scope;
21 }; 22 };
22 23
23 } // namespace WebCore 24 } // namespace WebCore
24 25
25 #endif // RegistrationOptionList_h 26 #endif // RegistrationOptionList_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698