Implementation of the Extension Activity Log UI behind a flag.

chrome/browser/resources/extensions/extension_activity.js

Index: chrome/browser/resources/extensions/extension_activity.js
diff --git a/chrome/browser/resources/extensions/extension_activity.js b/chrome/browser/resources/extensions/extension_activity.js
new file mode 100644
index 0000000000000000000000000000000000000000..ee5b525ad3ca61dfbf1f880b12c45649f69fa604
--- /dev/null
+++ b/chrome/browser/resources/extensions/extension_activity.js
@@ -0,0 +1,54 @@
+cr.define('extension_activity', function() {

[Dan Beam, 2012/01/28 04:24:41]

copyright

[Eric Dingle, 2012/01/30 18:27:11]

Done.

+  'use_strict';

[Dan Beam, 2012/01/28 04:24:41]

'use strict'; // Otherwise it won't work, ;)

[Eric Dingle, 2012/01/30 18:27:11]

Done.

+
+  function initialize() {
+    i18nTemplate.process(document, templateData);
+
+    var params = parseQueryParams(location);
+    if (params.extensionId)
+      chrome.send('requestExtensionData', [params.extensionId]);
+  }
+
+  function handleExtensionData(result) {
+    var extension = result.extension;
+
+    var item = document.querySelector('.extension-list-item');

[Dan Beam, 2012/01/28 04:24:41]

nit: I don't really see a reason for all these var's, but I'm assuming it's to
get around 80 char wrap (but there no particularly bad effect, I don't think).

[Eric Dingle, 2012/01/30 18:27:11]

Done.

+    item.style.backgroundImage = 'url(' + extension.icon + ')';
+
+    var title = item.querySelector('.extension-title');
+    title.textContent = extension.name;
+
+    var version = item.querySelector('.extension-version');
+    version.textContent = extension.version;
+
+    var description = item.querySelector('.extension-description');
+    description.textContent = extension.description;
+  }
+
+  function handleExtensionActivity(result) {
+    var template = $('template-collection');
+
+    var item = template.querySelector('.extension-activity-item').cloneNode(

[Dan Beam, 2012/01/28 04:24:41]

nit: this might be clearer if after = was on the next line

[Eric Dingle, 2012/01/30 18:27:11]

Done.

+        true);
+
+    var time = item.querySelector('.extension-activity-time');
+    time.textContent = new Date().toLocaleTimeString();
+
+    var label = item.querySelector('.extension-activity-label');
+    label.textContent = template.querySelector(
+        '.extension-activity-label-' + result.activity).textContent;
+
+    var message = item.querySelector('.extension-activity-message');
+    message.textContent = result.message;
+
+    $('extension-activity-list').appendChild(item);
+  }
+
+  return {
+    initialize: initialize,
+    handleExtensionData: handleExtensionData,
+    handleExtensionActivity: handleExtensionActivity
+  };
+});
+
+document.addEventListener('DOMContentLoaded', extension_activity.initialize);