Convert SiteSettingsCategory to use the HostContentSettingsMap instead of

chrome/browser/resources/settings/site_settings_page/site_settings_page.js

 // Copyright 2015 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.
 
 /**
  * @fileoverview
  * 'settings-site-settings-page' is the settings page containing privacy and
  * security site settings.
  *
  * Example:
@@ skipping 33 matching lines @@
      * The category selected by the user.
      */
     categorySelected: {
       type: String,
       notify: true,
     },
   },
 
   ready: function() {
     CrSettingsPrefs.initialized.then(function() {
-      this.addCategory(-1);  // Every category (All Sites).
-      this.addCategory(settings.ContentSettingsTypes.COOKIES);
-      this.addCategory(settings.ContentSettingsTypes.GEOLOCATION);
-      this.addCategory(settings.ContentSettingsTypes.CAMERA);
-      this.addCategory(settings.ContentSettingsTypes.MIC);
-      this.addCategory(settings.ContentSettingsTypes.JAVASCRIPT);
-      this.addCategory(settings.ContentSettingsTypes.POPUPS);
-      this.addCategory(settings.ContentSettingsTypes.FULLSCREEN);
-      this.addCategory(settings.ContentSettingsTypes.NOTIFICATIONS);
-      this.addCategory(settings.ContentSettingsTypes.IMAGES);
+      this.addCategory_(-1);  // Every category (All Sites).

[tommycli, 2016/02/18 22:15:19]

Ahh... the -1 sentinel value seems confusing. What if you just made a new
addAllCategories method?

+      this.addCategory_(settings.ContentSettingsTypes.COOKIES);
+      this.addCategory_(settings.ContentSettingsTypes.GEOLOCATION);
+      this.addCategory_(settings.ContentSettingsTypes.CAMERA);
+      this.addCategory_(settings.ContentSettingsTypes.MIC);
+      this.addCategory_(settings.ContentSettingsTypes.JAVASCRIPT);
+      this.addCategory_(settings.ContentSettingsTypes.POPUPS);
+      this.addCategory_(settings.ContentSettingsTypes.FULLSCREEN);
+      this.addCategory_(settings.ContentSettingsTypes.NOTIFICATIONS);
+      this.addCategory_(settings.ContentSettingsTypes.IMAGES);
     }.bind(this));
   },
 
   /**
    * Adds a single category to the page.
    * @param {number} category The category to add.
+   * @private
    */
-  addCategory: function(category) {
-    var icon, title, categoryDescription;
+  addCategory_: function(category) {
     if (category === -1) {
-      icon = 'list';
-      title = loadTimeData.getString('siteSettingsCategoryAllSites');
-      categoryDescription = '';
-    } else {
-      icon = this.computeIconForContentCategory(category);
-      title = this.computeTitleForContentCategory(category);
-      categoryDescription = this.computeCategoryDesc(
-          category, this.isCategoryAllowed(category), false);
+      var description = loadTimeData.getString('siteSettingsCategoryAllSites');
+      this.addCategoryImpl_(category, 'list', description, '');
+      return;
     }
 
+    var icon = this.computeIconForContentCategory(category);
+    var title = this.computeTitleForContentCategory(category);
+    settings.SiteSettingsPrefsProxy.getInstance().getDefaultValueForContentType(
+        category).then(function(enabled) {
+          var description = this.computeCategoryDesc(category, enabled, false);
+          this.addCategoryImpl_(category, icon, title, description);
+        }.bind(this));
+  },
+
+  /**
+   * Constructs the actual HTML elements for the category.
+   * @param {number} category The category to add.
+   * @param {string} icon The icon to show with it.
+   * @param {string} title The title to show for the category.
+   * @param {string} defaultValue The default value (permission) for the
+   *     category.
+   * @private
+   */
+  addCategoryImpl_: function(category, icon, title, defaultValue) {
     var root = this.$.list;
     var paperIcon = document.createElement('paper-icon-item');
     paperIcon.addEventListener('tap', this.onTapCategory.bind(this, category));
 
     var ironIcon = document.createElement('iron-icon');
     ironIcon.setAttribute('icon', icon);
     ironIcon.setAttribute('item-icon', '');
 
     var description = document.createElement('div');
     description.setAttribute('class', 'flex');
     description.appendChild(document.createTextNode(title));
     var setting = document.createElement('div');
     setting.setAttribute('class', 'option-value');
 
-    setting.appendChild(document.createTextNode(categoryDescription));
+    setting.appendChild(document.createTextNode(defaultValue));
 
     paperIcon.appendChild(ironIcon);
     paperIcon.appendChild(description);
     paperIcon.appendChild(setting);
     root.appendChild(paperIcon);
   },
 
   /**
    * Handles selection of a single category and navigates to the details for
    * that category.
@@ skipping 11 matching lines @@
       this.categorySelected = this.computeCategoryTextId(category);
       this.currentRoute = {
         page: this.currentRoute.page,
         section: 'privacy',
         subpage: ['site-settings', 'site-settings-category-' +
             this.categorySelected],
       };
     }
   },
 });