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

Side by Side Diff: chrome/renderer/resources/extensions/chrome_setting.js

Issue 12647017: Lazily require types when validating Extensions API calls (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: more fixes Created 7 years, 9 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
(Empty)
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 var chrome = requireNative('chrome').GetChrome();
6 var sendRequest = require('sendRequest').sendRequest;
7 var validate = require('schemaUtils').validate;
8
9 function extendSchema(schema) {
10 var extendedSchema = schema.slice();
11 extendedSchema.unshift({'type': 'string'});
12 return extendedSchema;
13 }
14
15 function ChromeSetting(prefKey, valueSchema) {
16 this.get = function(details, callback) {
17 var getSchema = this.functionSchemas.get.definition.parameters;
18 validate([details, callback], getSchema);
19 return sendRequest('types.ChromeSetting.get',
20 [prefKey, details, callback],
21 extendSchema(getSchema));
22 };
23 this.set = function(details, callback) {
24 var setSchema = this.functionSchemas.set.definition.parameters.slice();
25 setSchema[0].properties.value = valueSchema;
26 validate([details, callback], setSchema);
27 return sendRequest('types.ChromeSetting.set',
28 [prefKey, details, callback],
29 extendSchema(setSchema));
30 };
31 this.clear = function(details, callback) {
32 var clearSchema = this.functionSchemas.clear.definition.parameters;
33 validate([details, callback], clearSchema);
34 return sendRequest('types.ChromeSetting.clear',
35 [prefKey, details, callback],
36 extendSchema(clearSchema));
37 };
38 this.onChange = new chrome.Event('types.ChromeSetting.' + prefKey +
39 '.onChange');
40 };
41
42 exports.ChromeSetting = ChromeSetting;
OLDNEW
« no previous file with comments | « chrome/renderer/resources/extensions/binding.js ('k') | chrome/renderer/resources/extensions/content_setting.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698