| Index: chrome/renderer/resources/extensions/schema_generated_bindings.js
|
| diff --git a/chrome/renderer/resources/extensions/schema_generated_bindings.js b/chrome/renderer/resources/extensions/schema_generated_bindings.js
|
| index 66c193acac0d7f831b787cc5c8bcf9daa10f519a..d28c7d5e7c82f5ea07f29774571bc0ce21a2fc84 100644
|
| --- a/chrome/renderer/resources/extensions/schema_generated_bindings.js
|
| +++ b/chrome/renderer/resources/extensions/schema_generated_bindings.js
|
| @@ -5,11 +5,15 @@
|
| // This script contains privileged chrome extension related javascript APIs.
|
| // It is loaded by pages whose URL has the chrome-extension protocol.
|
|
|
| + // TODO(battre): cleanup the usage of packages everywhere, as described here
|
| + // http://codereview.chromium.org/10392008/diff/38/chrome/renderer/resources/extensions/schema_generated_bindings.js
|
| +
|
| require('json_schema');
|
| require('event_bindings');
|
| var GetExtensionAPIDefinition =
|
| requireNative('apiDefinitions').GetExtensionAPIDefinition;
|
| var sendRequest = require('sendRequest').sendRequest;
|
| + var utils = require('utils');
|
|
|
| var chromeHidden = requireNative('chrome_hidden').GetChromeHidden();
|
|
|
| @@ -22,13 +26,6 @@
|
| var internalAPIs = {};
|
| chromeHidden.internalAPIs = internalAPIs;
|
|
|
| - function forEach(dict, f) {
|
| - for (key in dict) {
|
| - if (dict.hasOwnProperty(key))
|
| - f(key, dict[key]);
|
| - }
|
| - }
|
| -
|
| // Validate arguments.
|
| var schemaValidator = new chromeHidden.JSONSchemaValidator();
|
| chromeHidden.validate = function(args, parameterSchemas) {
|
| @@ -510,7 +507,7 @@
|
| if (!properties)
|
| return;
|
|
|
| - forEach(properties, function(propertyName, propertyDef) {
|
| + utils.forEach(properties, function(propertyName, propertyDef) {
|
| if (propertyName in m)
|
| return; // TODO(kalman): be strict like functions/events somehow.
|
| if (!isSchemaNodeSupported(propertyDef, platform, manifestVersion))
|
|
|