Flesh out the location-page class to make it more general.

chrome/browser/resources/settings/location_page/site_list.html

Index: chrome/browser/resources/settings/location_page/site_list.html
diff --git a/chrome/browser/resources/settings/location_page/site_list.html b/chrome/browser/resources/settings/location_page/site_list.html
new file mode 100644
index 0000000000000000000000000000000000000000..e8aec7999934715faf01abb1428473deeea9548f
--- /dev/null
+++ b/chrome/browser/resources/settings/location_page/site_list.html
@@ -0,0 +1,51 @@
+<link rel="import" href="chrome://resources/polymer/v1_0/polymer/polymer.html">
+<link rel="import" href="chrome://resources/polymer/v1_0/iron-flex-layout/classes/iron-flex-layout.html">
+<link rel="import" href="chrome://resources/polymer/v1_0/iron-icons/communication-icons.html">
+<link rel="import" href="chrome://resources/polymer/v1_0/paper-item/paper-item.html">
+<link rel="import" href="chrome://resources/polymer/v1_0/paper-menu/paper-menu.html">
+<link rel="import" href="chrome://resources/polymer/v1_0/paper-menu/paper-submenu.html">
+
+<dom-module id="cr-settings-site-list">

[Dan Beam, 2015/10/10 01:27:40]

remove cr-

[Finnur, 2015/10/15 15:46:33]

Done.

+  <link rel="import" type="css"
+      href="chrome://md-settings/settings_page/settings_page.css">
+  <link rel="import" type="css" href="site_list.css">
+  <template>
+    <paper-menu>
+      <paper-submenu>
+        <paper-item class="menu-trigger">
+          <div>[[computeSiteListHeader_(siteList_, toggleState_)]]</div>
+        </paper-item>
+        <paper-menu class="menu-content">
+          <template is="dom-repeat" items="{{siteList_}}">
+            <div class="horizontal layout center">
+              <div><iron-icon
+                  icon="[[computeSiteIcon_(item.url)]]"
+                  item-icon></iron-icon></div>

[Dan Beam, 2015/10/10 01:27:40]

wrapped weirdly.  does

<div>
  <iron-icon icon="[[computeSiteIcon_(item.url)]]" item-icon></iron-icon>
</div>

work?  also, why is the outer div needed?

[Finnur, 2015/10/15 15:46:33]

It isn't. Removed.

+              <div class="flex"><paper-item
+                   on-click="handleClick_">[[item.url]]</paper-item></div>
+              <div>
+                <paper-menu-button>
+                  <paper-icon-button icon="menu" class="dropdown-trigger"
+                      alt="menu"></paper-icon-button>

[Dan Beam, 2015/10/10 01:27:40]

alt="menu"?

[Finnur, 2015/10/15 15:46:33]

Huh. Removed.

+                  <paper-menu id="actionMenu" class="dropdown-content"
+                      on-iron-select="selectAction_">

[Dan Beam, 2015/10/10 01:27:40]

onActionMenuIronSelect_

[Finnur, 2015/10/15 15:46:33]

Done.

+                    <template is="dom-if" if="[[showAllowAction_]]">
+                      <paper-item
+                          value="Allow">[[allowActionLabel_]]</paper-item>
+                    </template>
+                    <template is="dom-if" if="[[showBlockAction_]]">
+                      <paper-item
+                          value="Block">[[blockActionLabel_]]</paper-item>
+                    </template>

[Dan Beam, 2015/10/10 01:27:40]

for these maybe consider hidden="[[!showBlockAction_]]" instead of <template
is="dom-if" if="[[showBlockAction_]]">

[Finnur, 2015/10/15 15:46:33]

Done. My thinking here was that this list could get really long, and dom-if
would make sure un-necessary menu items are not added to the DOM. But, if you
feel there's a benefit to doing it the other way, then I don't feel that
strongly about it.

+                    <paper-item value="Reset">[[resetActionLabel_]]</paper-item>
+                  </paper-menu>
+                </paper-menu-button>
+              </div>
+            </div>
+          </template>
+        </paper-menu>
+      </paper-submenu>
+    </paper-menu>
+  </template>
+  <script src="site_list.js"></script>
+</dom-module>