[MD settings] setup site settings menus

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

Index: chrome/browser/resources/settings/site_settings/site_list.html
diff --git a/chrome/browser/resources/settings/site_settings/site_list.html b/chrome/browser/resources/settings/site_settings/site_list.html
index 9a2e67e7ceb29fc29e346166c2e15508e02708ce..2a9b0585690245eb2344fe4d667b5e1bc2124dac 100644
--- a/chrome/browser/resources/settings/site_settings/site_list.html
+++ b/chrome/browser/resources/settings/site_settings/site_list.html
@@ -66,36 +66,33 @@
             <template is="dom-if" if="[[isPolicyControlled_(item.source)]]">
               <iron-icon icon="cr:domain"></iron-icon>
             </template>
-            <template is="dom-if" if="[[shouldShowMenu_(item.source)]]">
-              <paper-menu-button>
-                <paper-icon-button icon="cr:more-vert"
-                    class="dropdown-trigger">
-                </paper-icon-button>
-                <paper-menu id="actionMenu" class="dropdown-content"
-                    on-iron-activate="onActionMenuIronActivate_"
-                    attr-for-selected="menu-value">
-                  <paper-item
-                      hidden="[[!showAllowAction_]]"
-                      menu-value$="[[PermissionValues.ALLOW]]">
-                    $i18n{siteSettingsActionAllow}
-                  </paper-item>
-                  <paper-item
-                      hidden="[[!showBlockAction_]]"
-                      menu-value$="[[PermissionValues.BLOCK]]">
-                    $i18n{siteSettingsActionBlock}
-                  </paper-item>
-                  <paper-item
-                      hidden="[[!showSessionOnlyAction_]]"
-                      menu-value$="[[PermissionValues.SESSION_ONLY]]">
-                    $i18n{siteSettingsActionSessionOnly}
-                  </paper-item>
-                  <paper-item
-                      menu-value$="[[PermissionValues.DEFAULT]]">
-                    $i18n{siteSettingsActionReset}
-                  </paper-item>
-                </paper-menu>
-              </paper-menu-button>
-            </template>
+            <paper-menu-button hidden="[[isPolicyControlled_(item.source)]]">

[tommycli, 2016/08/10 17:31:23]

shouldShowMenu still exists in the js file.

Also this changes the meaning somewhat, since shouldShowMenu is true if
this.allSites is true. Does that still work?

[dschuyler, 2016/08/10 18:26:47]

Done.

+              <paper-icon-button icon="cr:more-vert" class="dropdown-trigger">
+              </paper-icon-button>
+              <paper-menu id="actionMenu" class="dropdown-content"
+                  on-iron-activate="onActionMenuIronActivate_"
+                  attr-for-selected="menu-value">
+                <paper-item
+                    hidden="[[!showAllowAction_]]"
+                    menu-value$="[[PermissionValues.ALLOW]]">
+                  $i18n{siteSettingsActionAllow}
+                </paper-item>
+                <paper-item
+                    hidden="[[!showBlockAction_]]"
+                    menu-value$="[[PermissionValues.BLOCK]]">
+                  $i18n{siteSettingsActionBlock}
+                </paper-item>
+                <paper-item
+                    hidden="[[!showSessionOnlyAction_]]"
+                    menu-value$="[[PermissionValues.SESSION_ONLY]]">
+                  $i18n{siteSettingsActionSessionOnly}
+                </paper-item>
+                <paper-item
+                    menu-value$="[[PermissionValues.DEFAULT]]">
+                  $i18n{siteSettingsActionReset}
+                </paper-item>
+              </paper-menu>
+            </paper-menu-button>
           </div>
         </template>