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

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

Issue 307833002: Move some extensions renderer resources to extensions_renderer_resources.grd. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 6 years, 7 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/messaging_utils.js
diff --git a/chrome/renderer/resources/extensions/messaging_utils.js b/chrome/renderer/resources/extensions/messaging_utils.js
deleted file mode 100644
index dd0fc28aa01c94260e50b82e7cd8214587430919..0000000000000000000000000000000000000000
--- a/chrome/renderer/resources/extensions/messaging_utils.js
+++ /dev/null
@@ -1,53 +0,0 @@
-// Copyright 2013 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.
-
-// Routines used to normalize arguments to messaging functions.
-
-function alignSendMessageArguments(args, hasOptionsArgument) {
- // Align missing (optional) function arguments with the arguments that
- // schema validation is expecting, e.g.
- // extension.sendRequest(req) -> extension.sendRequest(null, req)
- // extension.sendRequest(req, cb) -> extension.sendRequest(null, req, cb)
- if (!args || !args.length)
- return null;
- var lastArg = args.length - 1;
-
- // responseCallback (last argument) is optional.
- var responseCallback = null;
- if (typeof args[lastArg] == 'function')
- responseCallback = args[lastArg--];
-
- var options = null;
- if (hasOptionsArgument && lastArg >= 1) {
- // options (third argument) is optional. It can also be ambiguous which
- // argument it should match. If there are more than two arguments remaining,
- // options is definitely present:
- if (lastArg > 1) {
- options = args[lastArg--];
- } else {
- // Exactly two arguments remaining. If the first argument is a string,
- // it should bind to targetId, and the second argument should bind to
- // request, which is required. In other words, when two arguments remain,
- // only bind options when the first argument cannot bind to targetId.
- if (!(args[0] === null || typeof args[0] == 'string'))
- options = args[lastArg--];
- }
- }
-
- // request (second argument) is required.
- var request = args[lastArg--];
-
- // targetId (first argument, extensionId in the manifest) is optional.
- var targetId = null;
- if (lastArg >= 0)
- targetId = args[lastArg--];
-
- if (lastArg != -1)
- return null;
- if (hasOptionsArgument)
- return [targetId, request, options, responseCallback];
- return [targetId, request, responseCallback];
-}
-
-exports.alignSendMessageArguments = alignSendMessageArguments;
« no previous file with comments | « chrome/renderer/resources/extensions/messaging.js ('k') | chrome/renderer/resources/extensions/permissions_custom_bindings.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698