[MD settings] setup site settings menus

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

 <link rel="import" href="chrome://resources/cr_elements/icons.html">
 <link rel="import" href="chrome://resources/html/polymer.html">
 <link rel="import" href="chrome://resources/html/web_ui_listener_behavior.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/iron-icon/iron-icon.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-button/paper-button.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-icon-button/paper-icon-button.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">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-menu-button/paper-menu-button.html">
 <link rel="import" href="/i18n_setup.html">
@@ skipping 48 matching lines @@
                 <div class="selectable">[[item.originForDisplay]]</div>
                 <div class="selectable secondary">
                   [[item.embeddingOriginForDisplay]]
                 </div>
               </div>
             </div>
 
             <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>
 
         <template is="dom-if" if="[[!allSites]]">
           <div class="button-container">
             <paper-button on-tap="onAddSiteTap_"
                 class="primary-button no-upper">
               $i18n{addSiteLink}
             </paper-button>
           </div>
         </template>
       </div>
     </paper-submenu>
   </template>
   <script src="site_list.js"></script>
 </dom-module>