MD Settings: Allow editing a cookie site exception.

chrome/browser/resources/settings/site_settings/site_list.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.
 
 /**
  * Enumeration mapping all possible controlled-by values for exceptions to
  * icons.
  * @enum {string}
  */
 var iconControlledBy = {
@@ skipping 23 matching lines @@
       },
     },
 
     /**
      * The site serving as the model for the currently open action menu.
      * @private {?SiteException}
      */
     actionMenuSite_: Object,
 
     /**
+     * Whether the "edit exception" dialog should be shown.
+     * @private
+     */
+    showEditExceptionDialog_: Boolean,
+
+    /**
      * Array of sites to display in the widget.
      * @type {!Array<SiteException>}
      */
     sites: {
       type: Array,
       value: function() { return []; },
     },
 
     /**
      * Whether this list is for the All Sites category.
      */
     allSites: {
       type: Boolean,
       value: false,
     },
 
     /**
       * The type of category this widget is displaying data for. Normally
       * either 'allow' or 'block', representing which sites are allowed or
       * blocked respectively.
       */
     categorySubtype: {
       type: String,
       value: settings.INVALID_CATEGORY_SUBTYPE,
     },
 
     /**
      * Whether to show the Allow action in the action menu.
+     * @private
      */
     showAllowAction_: Boolean,
 
     /**
      * Whether to show the Block action in the action menu.
+     * @private
      */
     showBlockAction_: Boolean,
 
     /**
      * Whether to show the 'Clear on exit' action in the action
      * menu.
+     * @private
      */
     showSessionOnlyAction_: Boolean,
 
     /**
+     * Whether to show the 'edit' action in the action menu.
+     * @private
+     */
+    showEditAction_: Boolean,
+
+    /**
      * Keeps track of the incognito status of the current profile (whether one
      * exists).
+     * @private
      */
     incognitoProfileActive_: {
       type: Boolean,
       value: false,
     },
 
     /**
      * All possible actions in the action menu.
+     * @private
      */
     actions_: {
       readOnly: true,
       type: Object,
       values: {
         ALLOW: 'Allow',
         BLOCK: 'Block',
         RESET: 'Reset',
         SESSION_ONLY: 'SessionOnly',
       }
@@ skipping 248 matching lines @@
    * @private
    */
   setUpActionMenu_: function() {
     this.showAllowAction_ =
         this.categorySubtype != settings.PermissionValues.ALLOW;
     this.showBlockAction_ =
         this.categorySubtype != settings.PermissionValues.BLOCK;
     this.showSessionOnlyAction_ =
         this.categorySubtype != settings.PermissionValues.SESSION_ONLY &&
         this.category == settings.ContentSettingsTypes.COOKIES;
+    this.showEditAction_ =
+        this.category == settings.ContentSettingsTypes.COOKIES;
   },
 
   /**
    * @return {boolean} Whether to show the "Session Only" menu item for the
    *     currently active site.
    * @private
    */
   showSessionOnlyActionForSite_: function() {
     // It makes no sense to show "clear on exit" for exceptions that only apply
     // to incognito. It gives the impression that they might under some
@@ skipping 47 matching lines @@
     this.closeActionMenu_();
   },
 
   /** @private */
   onSessionOnlyTap_: function() {
     this.onActionMenuActivate_(settings.PermissionValues.SESSION_ONLY);
     this.closeActionMenu_();
   },
 
   /** @private */
+  onEditTap_: function() {
+    this.showEditExceptionDialog_ = true;
+  },
+
+  /** @private */
+  onEditExceptionDialogClosed_: function() {
+    this.showEditExceptionDialog_ = false;
+    // Close action menu after dialog has been closed, otherwise
+    // |actionMenuSite_| is reset while the dialog is still accessing it.
+    this.closeActionMenu_();
+  },
+
+  /** @private */
   onResetTap_: function() {
     this.onActionMenuActivate_(settings.PermissionValues.DEFAULT);
     this.closeActionMenu_();
   },
 
   /**
    * Returns the appropriate site description to display. This can, for example,
    * be blank, an 'embedded on <site>' or 'Current incognito session' (or a
    * mix of the last two).
    * @param {SiteException} item The site exception entry.
@@ skipping 24 matching lines @@
 
   /** @private */
   closeActionMenu_: function() {
     this.actionMenuSite_ = null;
     var actionMenu = /** @type {!CrActionMenuElement} */ (
         this.$$('dialog[is=cr-action-menu]'));
     if (actionMenu.open)
       actionMenu.close();
   },
 });