Added ActivityLog tests and associated bugfixes/extra logging.

chrome/renderer/resources/extensions/app_custom_bindings.js

 // 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 app API.
 
 var GetAvailability = requireNative('v8_context').GetAvailability;
 if (!GetAvailability('app').is_available) {
   exports.chromeApp = {};
   exports.chromeHiddenApp = {};
   return;
 }
 
 var appNatives = requireNative('app');
 var chrome = requireNative('chrome').GetChrome();
+var process = requireNative('process');
+var extensionId = process.GetExtensionId();
+var logActivity = requireNative('activityLogger').LogActivity;
+
+function wrapForLogging(fun) {
+  var id = extensionId;
+  var slicer = Array.prototype.slice;
+  return (function() {
+    logActivity("API", id, "app." + fun.name, slicer.call(arguments));
+    return fun.apply(this, arguments);
+  });
+}
 
 // This becomes chrome.app
-var app = {
-  getIsInstalled: appNatives.GetIsInstalled,
-  install: appNatives.Install,
-  getDetails: appNatives.GetDetails,
-  getDetailsForFrame: appNatives.GetDetailsForFrame,
-  runningState: appNatives.GetRunningState
-};
+if (!extensionId) {

[Matt Perry, 2013/04/09 20:56:59]

When can the extensionId be NULL?

[felt, 2013/04/10 00:21:38]

If this is a website, extensionId is null and we don't want to do any logging.

+  var app = {

[Matt Perry, 2013/04/09 20:56:59]

I think you need to declare "var app;" at toplevel scope. otherwise this will
only exist withing the if-block (unless I'm wrong about JavaScript, which is
possible).

[felt, 2013/04/10 00:21:38]

It works fine actually. But maybe it's more clear with var app; first. (Done.)

+    getIsInstalled: appNatives.GetIsInstalled,
+    install: appNatives.Install,
+    getDetails: appNatives.GetDetails,
+    getDetailsForFrame: appNatives.GetDetailsForFrame,
+    runningState: appNatives.GetRunningState
+  };
+} else {
+  var app = {
+    getIsInstalled: wrapForLogging(appNatives.GetIsInstalled),
+    install: wrapForLogging(appNatives.Install),
+    getDetails: wrapForLogging(appNatives.GetDetails),
+    getDetailsForFrame: wrapForLogging(appNatives.GetDetailsForFrame),
+    runningState: wrapForLogging(appNatives.GetRunningState)
+  };
+}
 
 // Tricky; "getIsInstalled" is actually exposed as the getter "isInstalled",
 // but we don't have a way to express this in the schema JSON (nor is it
 // worth it for this one special case).
 //
 // So, define it manually, and let the getIsInstalled function act as its
 // documentation.
-app.__defineGetter__('isInstalled', appNatives.GetIsInstalled);
+if (!extensionId)
+  app.__defineGetter__('isInstalled', appNatives.GetIsInstalled);
+else
+  app.__defineGetter__('isInstalled',
+                       wrapForLogging(appNatives.GetIsInstalled));
 
 // Called by app_bindings.cc.
 // This becomes chromeHidden.app
 var chromeHiddenApp = {
   onInstallStateResponse: function(state, callbackId) {
     if (callbackId) {
       callbacks[callbackId](state);
       delete callbacks[callbackId];
     }
   }
 };
 
 // TODO(kalman): move this stuff to its own custom bindings.
 var callbacks = {};
 var nextCallbackId = 1;
 
 app.installState = function getInstallState(callback) {
   var callbackId = nextCallbackId++;
   callbacks[callbackId] = callback;
   appNatives.GetInstallState(callbackId);
 };
+if (extensionId)
+  app.installState = wrapForLogging(app.installState);
 
 // These must match the names in InstallAppbinding() in
 // chrome/renderer/extensions/dispatcher.cc.
 exports.chromeApp = app;
 exports.chromeHiddenApp = chromeHiddenApp;