[MD settings] site data icons and tool tips

This Cl restores the site data icons and tool tips on the siteData page.

Bug: None
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Ia29e09f8ed8bf42fb377461b19be86436556ea69
Reviewed-on: https://chromium-review.googlesource.com/669665
Reviewed-by: Patti <patricialor@chromium.org>
Commit-Queue: Dave Schuyler <dschuyler@chromium.org>
Cr-Commit-Position: refs/heads/master@{#502640}

chrome/browser/resources/settings/site_settings/site_data.html

@@ -39,7 +39,8 @@
         notify-dom-change>
       <div class="settings-box two-line" first$="[[!index]]" id="siteItem"
           on-tap="onSiteTap_" actionable>
-        <div class="favicon-image" style$="[[computeSiteIcon(item.site)]]">
+        <div class="favicon-image"
+            style$="background-image: [[favicon_(item.site)]]">
         </div>
         <div class="middle">
           [[item.site]]

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

@@ -22,6 +22,7 @@ Polymer({
 
   behaviors: [
     CookieTreeBehavior,
+    I18nBehavior,
   ],
 
   properties: {
@@ -50,6 +51,17 @@ Polymer({
     this.loadCookies();
   },
 
+  /**
+   * Returns the icon to use for a given site.
+   * @param {string} url The url of the site to fetch the icon for.
+   * @return {string} Value for background-image style.
+   * @private
+   */
+  favicon_: function(url) {
+    return cr.icon.getFavicon(url);
+  },
+
+
   /**
    * @param {!Map<string, string>} newConfig
    * @param {?Map<string, string>} oldConfig