chrome/renderer/resources/extensions/types_custom_bindings.js - Issue 10535030: Allow updateArgumentsPostValidate to support callbacks and added / removed arguments.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: chrome/renderer/resources/extensions/types_custom_bindings.js

Issue 10535030: Allow updateArgumentsPostValidate to support callbacks and added / removed arguments. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src

Patch Set: Unit tests Created 8 years, 6 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 side-by-side diff with in-line comments

Download patch

« chrome/renderer/resources/extensions/storage_custom_bindings.js ('K') | « chrome/renderer/resources/extensions/storage_custom_bindings.js ('k') | chrome/renderer/resources/extensions/validate_utils.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: chrome/renderer/resources/extensions/types_custom_bindings.js

diff --git a/chrome/renderer/resources/extensions/types_custom_bindings.js b/chrome/renderer/resources/extensions/types_custom_bindings.js

index daf029a7907e186b3012f10bed4e946e09b32bef..0858991293c32ea5e5795e628eea74aaf46e5917 100644

--- a/chrome/renderer/resources/extensions/types_custom_bindings.js

+++ b/chrome/renderer/resources/extensions/types_custom_bindings.js

@@ -6,6 +6,7 @@

var chromeHidden = requireNative('chrome_hidden').GetChromeHidden();

var sendRequest = require('sendRequest').sendRequest;

+var validate = require('validateUtils').validate;

chromeHidden.registerCustomType('types.ChromeSetting', function() {

@@ -18,7 +19,7 @@ chromeHidden.registerCustomType('types.ChromeSetting', function() {

function ChromeSetting(prefKey, valueSchema) {

this.get = function(details, callback) {

var getSchema = this.parameters.get;

- chromeHidden.validate([details, callback], getSchema);

+ validate([details, callback], getSchema);

return sendRequest('types.ChromeSetting.get',

[prefKey, details, callback],

extendSchema(getSchema));

@@ -26,14 +27,14 @@ chromeHidden.registerCustomType('types.ChromeSetting', function() {

this.set = function(details, callback) {

var setSchema = this.parameters.set.slice();

setSchema[0].properties.value = valueSchema;

- chromeHidden.validate([details, callback], setSchema);

+ validate([details, callback], setSchema);

return sendRequest('types.ChromeSetting.set',

[prefKey, details, callback],

extendSchema(setSchema));

};

this.clear = function(details, callback) {

var clearSchema = this.parameters.clear;

- chromeHidden.validate([details, callback], clearSchema);

+ validate([details, callback], clearSchema);

return sendRequest('types.ChromeSetting.clear',

[prefKey, details, callback],

extendSchema(clearSchema));