Show local storage usage on the Website Settings options page.

chrome/browser/resources/options/website_settings.js

Index: chrome/browser/resources/options/website_settings.js
diff --git a/chrome/browser/resources/options/website_settings.js b/chrome/browser/resources/options/website_settings.js
index c43c6fcfe880aba2c630d0da5c78e0ba9eaef1ef..dead3d52e3bb781dc63019d7effeee06c39e6ae5 100644
--- a/chrome/browser/resources/options/website_settings.js
+++ b/chrome/browser/resources/options/website_settings.js
@@ -41,7 +41,10 @@ cr.define('options', function() {
       $('resourceType').onchange = function() {
         var target = event.target;
         assert(target.tagName == 'SELECT');
-        chrome.send('updateOrigins', [target.value]);
+        if (target.value == 'storage')
+          chrome.send('updateLocalStorage');
+        else
+          chrome.send('updateOrigins', [target.value]);
       };
 
       var searchBox = $('website-settings-search-box');
@@ -90,6 +93,23 @@ cr.define('options', function() {
     },
 
     /**
+     * Populates the origin list with all origins that are using local storage.
+     * @private
+     */
+    populateLocalStorage_: function(originDict) {
+      var origins = [];
+      for (var origin in originDict) {

[Bernhard Bauer, 2014/07/21 11:29:29]

Consider using Object.keys(originDict).map(...);

[Daniel Nishi, 2014/07/21 17:53:36]

Done.

+        origins.push({origin: origin,
+                      usage: originDict[origin]['usage'],

[Bernhard Bauer, 2014/07/21 11:29:29]

Use .usage?

[Daniel Nishi, 2014/07/21 17:53:36]

Done.

+                      usage_string: originDict[origin]['usage_string']});
+      }
+      origins.sort(function(first, second) {
+        return second.usage - first.usage;
+      });
+      this.originList_.dataModel = new ArrayDataModel(origins);
+    },
+
+    /**
      * Handle and delay search query changes.
      * @param {!Event} e The event object.
      * @private
@@ -107,6 +127,10 @@ cr.define('options', function() {
     WebsiteSettingsManager.getInstance().populateOrigins_(originDict);
   };
 
+  WebsiteSettingsManager.updateLocalStorage = function(originDict) {

[Bernhard Bauer, 2014/07/21 11:29:29]

Call this method also populateLocalStorage()?

[Daniel Nishi, 2014/07/21 17:53:36]

Done.

+    WebsiteSettingsManager.getInstance().populateLocalStorage_(originDict);
+  };
+
   // Export
   return {
     WebsiteSettingsManager: WebsiteSettingsManager