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

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

Issue 13726026: Added ActivityLog tests and associated bugfixes/extra logging. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Added message passing test and logging Created 7 years, 8 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/binding.js
diff --git a/chrome/renderer/resources/extensions/binding.js b/chrome/renderer/resources/extensions/binding.js
index 80c81242e7d2ca63da64bb1a40bcb68829ee973e..d2a2dc8d22871caac3c9a02010681ad1ea6f62c0 100644
--- a/chrome/renderer/resources/extensions/binding.js
+++ b/chrome/renderer/resources/extensions/binding.js
@@ -53,14 +53,15 @@ APIFunctions.prototype.setHook_ =
APIFunctions.prototype.setHandleRequest =
function(apiName, customizedFunction) {
var prefix = this.namespace;
+ var slicer = Array.prototype.slice;
Matt Perry 2013/04/09 20:56:59 why is this necessary?
felt 2013/04/10 22:25:04 Removed for now.
return this.setHook_(apiName, 'handleRequest',
function() {
var ret = customizedFunction.apply(this, arguments);
// Logs API calls to the Activity Log if it doesn't go through an
// ExtensionFunction.
if (!sendRequestHandler.getCalledSendRequest())
- logActivity(extensionId, prefix + "." + apiName,
- Array.prototype.slice.call(arguments));
+ logActivity("API", extensionId, prefix + "." + apiName,
+ slicer.call(arguments));
return ret;
});
};

Powered by Google App Engine
This is Rietveld 408576698