[MD settings] site details url below Site details subpage title

chrome/browser/resources/settings/site_settings/site_details.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
  * 'site-details' show the details (permissions and usage) for a given origin
  * under Site Settings.
  */
 Polymer({
@@ skipping 27 matching lines @@
 
   listeners: {
     'usage-deleted': 'onUsageDeleted',
   },
 
   ready: function() {
     this.ContentSettingsTypes = settings.ContentSettingsTypes;
   },
 
   /**
+   * @param {string} url Location of an image to get a set of icons for.
+   * @return {string} A set of icon URLs.
+   * @private
+   */
+  getIconSet_: function(url) {
+    return cr.icon.getFavicon(this.ensureUrlHasScheme(url));
+  },

[Finnur, 2016/09/14 11:00:21]

The behavior already has a computeSiteIcon function, which does essentially the
same.

[dschuyler, 2016/09/14 19:11:33]

Done.

+
+  /**
    * Handler for when the origin changes.
    */
   onSiteChanged_: function() {
     // Using originForDisplay avoids the [*.] prefix that some exceptions use.
     var url = new URL(this.ensureUrlHasScheme(this.site.originForDisplay));
     this.$.usageApi.fetchUsageTotal(url.hostname);
   },
 
   /**
    * Clears all data stored for the current origin.
@@ skipping 36 matching lines @@
 
   /**
    * Returns true if one or more permission is showing.
    */
   permissionShowing_: function() {
     return Array.prototype.some.call(
         this.root.querySelectorAll('site-details-permission'),
         function(element) { return element.offsetHeight > 0; });
   },
 });