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

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

Issue 12647017: Lazily require types when validating Extensions API calls (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: 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 side-by-side diff with in-line comments
Download patch
Index: chrome/renderer/resources/extensions/content_settings_custom_bindings.js
diff --git a/chrome/renderer/resources/extensions/content_settings_custom_bindings.js b/chrome/renderer/resources/extensions/content_settings_custom_bindings.js
deleted file mode 100644
index 83470787cf45cfe115e8d17e40919777e35e25bf..0000000000000000000000000000000000000000
--- a/chrome/renderer/resources/extensions/content_settings_custom_bindings.js
+++ /dev/null
@@ -1,56 +0,0 @@
-// Copyright (c) 2012 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-// Custom binding for the contentSettings API.
-
-var binding = require('binding').Binding.create('contentSettings');
-
-var sendRequest = require('sendRequest').sendRequest;
-var validate = require('schemaUtils').validate;
-
-binding.registerCustomType('contentSettings.ContentSetting', function() {
- function extendSchema(schema) {
- var extendedSchema = schema.slice();
- extendedSchema.unshift({'type': 'string'});
- return extendedSchema;
- }
-
- function ContentSetting(contentType, settingSchema) {
- this.get = function(details, callback) {
- var getSchema = this.functionSchemas.get.definition.parameters;
- validate([details, callback], getSchema);
- return sendRequest('contentSettings.get',
- [contentType, details, callback],
- extendSchema(getSchema));
- };
- this.set = function(details, callback) {
- var setSchema = this.functionSchemas.set.definition.parameters.slice();
- setSchema[0].properties.setting = settingSchema;
- validate([details, callback], setSchema);
- return sendRequest('contentSettings.set',
- [contentType, details, callback],
- extendSchema(setSchema));
- };
- this.clear = function(details, callback) {
- var clearSchema = this.functionSchemas.clear.definition.parameters;
- validate([details, callback], clearSchema);
- return sendRequest('contentSettings.clear',
- [contentType, details, callback],
- extendSchema(clearSchema));
- };
- this.getResourceIdentifiers = function(callback) {
- var schema =
- this.functionSchemas.getResourceIdentifiers.definition.parameters;
- validate([callback], schema);
- return sendRequest(
- 'contentSettings.getResourceIdentifiers',
- [contentType, callback],
- extendSchema(schema));
- };
- }
-
- return ContentSetting;
-});
-
-exports.binding = binding.generate();

Powered by Google App Engine
This is Rietveld 408576698