Add chrome://quota-internals/ resources

chrome/browser/resources/quota_internals/quota-internals.js

+// Copyright (c) 2011 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.
+
+// require cr.js
+// require cr/event_target.js
+// require cr/util.js
+
+cr.define('cr.quota', function() {
+  'use strict';
+  return {
+    onAvailableSpaceUpdated: new cr.EventTarget(),
+    onGlobalDataUpdated: new cr.EventTarget(),
+    onHostDataUpdated: new cr.EventTarget(),
+    onOriginDataUpdated: new cr.EventTarget(),
+    onStatisticsUpdated: new cr.EventTarget(),
+
+    requestData: function() {

[Evan Stade, 2011/06/04 02:43:17]

can you define these function first and then just return them by name:

return {
  foo: foo,
  bar: bar,
  ...
}

[tzik, 2011/06/06 10:57:44]

Done.

+      chrome.send('requestData');
+    },
+
+    // Chrome Message Handler
+    messageHandler_: function(message, detail) {

[Evan Stade, 2011/06/04 02:43:17]

every function needs a comment. Every parameter needs a comment.

+      var target = null;
+      switch (message) {
+      case 'AvailableSpaceUpdated':
+        target = cr.quota.onAvailableSpaceUpdated;
+        break;
+      case 'GlobalDataUpdated':
+        target = cr.quota.onGlobalDataUpdated;
+        break;
+      case 'HostDataUpdated':
+        target = cr.quota.onHostDataUpdated;
+        break;
+      case 'OriginDataUpdated':
+        target = cr.quota.onOriginDataUpdated;
+        break;
+      case 'StatisticsUpdated':
+        target = cr.quota.onStatisticsUpdated;
+        break;
+      default:
+        console.log('Unknown Message');
+        break;
+      }
+      var event = cr.doc.createEvent('CustomEvent');
+      if (target) {
+        event.initCustomEvent('update', false, false, detail);
+        target.dispatchEvent(event);
+      }
+    }
+  };
+});