[MD setting] tool tips on controlledBy site exceptions

chrome/browser/resources/settings/site_settings/site_list.js

Index: chrome/browser/resources/settings/site_settings/site_list.js
diff --git a/chrome/browser/resources/settings/site_settings/site_list.js b/chrome/browser/resources/settings/site_settings/site_list.js
index a33ae74929030acd467d0487dc1e196107ba37af..4d9c836f0ee3d01418b0403f7d06ea645c2e33bd 100644
--- a/chrome/browser/resources/settings/site_settings/site_list.js
+++ b/chrome/browser/resources/settings/site_settings/site_list.js
@@ -3,18 +3,6 @@
 // found in the LICENSE file.
 
 /**
- * Enumeration mapping all possible controlled-by values for exceptions to
- * icons.
- * @enum {string}
- */
-var iconControlledBy = {
-  'extension': 'cr:extension',
-  'HostedApp': 'cr:extension',
-  'platform_app': 'cr:extension',
-  'policy' : 'cr20:domain',
-};
-
-/**
  * @fileoverview
  * 'site-list' shows a list of Allowed and Blocked sites for a given
  * category.
@@ -180,19 +168,6 @@ Polymer({
   },
 
   /**
-   * Returns which icon, if any, should represent the fact that this exception
-   * is controlled.
-   * @param {!SiteException} item The item from the list we're computing the
-   *    icon for.
-   * @return {string} The icon to show (or blank, if none).
-   */
-  computeIconControlledBy_: function(item) {
-    if (this.allSites)
-      return '';
-    return iconControlledBy[item.source] || '';
-  },
-
-  /**
    * Whether there are any site exceptions added for this content setting.
    * @return {boolean}
    * @private
@@ -202,24 +177,25 @@ Polymer({
   },
 
   /**
-   * @param {string} source Where the setting came from.
+   * @param {string} enforcement Whether the exception is controlled.

[tommycli, 2017/03/27 16:02:07]

nit: you may be able to use the actual chrome.settingsPrivate.Enforcement type
here. 

Also the comment "Whether the..." kind of implies a boolean type variable...
while it's actually more like an enum.

[dschuyler, 2017/03/28 01:05:53]

Done.

    * @param {boolean} readOnlyList Whether the site exception list is read-only.
    * @return {boolean}
    * @private
    */
-  isResetButtonHidden_: function(source, readOnlyList) {
-    return this.isExceptionControlled_(source) || this.allSites ||
-        !readOnlyList;
+  isResetButtonHidden_: function(enforcement, readOnlyList) {
+    return enforcement == chrome.settingsPrivate.Enforcement.ENFORCED ||
+        this.allSites || !readOnlyList;
   },
 
   /**
-   * @param {string} source Where the setting came from.
+   * @param {string} enforcement Whether the exception is controlled.
    * @param {boolean} readOnlyList Whether the site exception list is read-only.
    * @return {boolean}
    * @private
    */
-  isActionMenuHidden_: function(source, readOnlyList) {
-    return this.isExceptionControlled_(source) || this.allSites || readOnlyList;
+  isActionMenuHidden_: function(enforcement, readOnlyList) {
+    return enforcement == chrome.settingsPrivate.Enforcement.ENFORCED ||
+        this.allSites || readOnlyList;
   },
 
   /**
@@ -263,14 +239,24 @@ Polymer({
 
   /**
    * Process the exception list returned from the native layer.
-   * @param {!Array<!Array<SiteException>>} data List of sites (exceptions) to
-   *     process.
+   * @param {!Array<!Array<RawSiteException>>} data List of sites (exceptions)
+   *     to process.
    * @private
    */
   processExceptions_: function(data) {
-    var sites = [];
-    for (var i = 0; i < data.length; ++i)
-      sites = this.appendSiteList_(sites, data[i]);
+    var sites = /** @type {!Array<RawSiteException>} */ ([]);
+    for (var i = 0; i < data.length; ++i) {
+      var exceptionList = data[i];
+      for (var k = 0; k < exceptionList.length; ++k) {
+        if (!this.allSites &&
+            (exceptionList[k].setting == settings.PermissionValues.DEFAULT ||
+             exceptionList[k].setting != this.categorySubtype)) {
+          continue;
+        }
+
+        sites.push(exceptionList[k]);
+      }
+    }
     this.sites = this.toSiteArray_(sites);
   },
 
@@ -301,34 +287,11 @@ Polymer({
   },
 
   /**
-   * Appends to |list| the sites for a given category and subtype.
-   * @param {!Array<SiteException>} sites The site list to add to.
-   * @param {!Array<SiteException>} exceptionList List of sites (exceptions) to
-   *     add.
-   * @return {!Array<SiteException>} The list of sites.
-   * @private
-   */
-  appendSiteList_: function(sites, exceptionList) {
-    for (var i = 0; i < exceptionList.length; ++i) {
-      if (!this.allSites) {
-        if (exceptionList[i].setting == settings.PermissionValues.DEFAULT)
-          continue;
-
-        if (exceptionList[i].setting != this.categorySubtype)
-          continue;
-      }
-
-      sites.push(exceptionList[i]);
-    }
-    return sites;
-  },
-
-  /**
    * Converts a list of exceptions received from the C++ handler to
    * full SiteException objects. If this site-list is used as an all sites
    * view, the list is sorted by site name, then protocol and port and de-duped
    * (by origin).
-   * @param {!Array<SiteException>} sites A list of sites to convert.
+   * @param {!Array<RawSiteException>} sites A list of sites to convert.
    * @return {!Array<SiteException>} A list of full SiteExceptions. Sorted and
    *    deduped if allSites is set.
    * @private