MD Settings: Allow editing a cookie site exception.

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

+// Copyright 2016 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.
+
+/**
+ * @fileoverview 'settings-edit-exception-dialog' is a component for editing a
+ * site exception entry.
+ */
+Polymer({
+  is: 'settings-edit-exception-dialog',
+
+  properties: {
+    /**
+     * @type {!SiteException}
+     */
+    model: Object,
+
+    /** @private */
+    origin_: String,
+  },
+
+  /** @private {!settings.SiteSettingsPrefsBrowserProxy} */
+  browserProxy_: null,
+
+  /** @override */
+  attached: function() {
+    this.browserProxy_ =
+        settings.SiteSettingsPrefsBrowserProxyImpl.getInstance();
+    this.origin_ = this.model.origin;
+
+    this.$.dialog.showModal();
+  },
+
+  /** @private */
+  onCancelTap_: function() {
+    this.$.dialog.close();
+  },
+
+  /** @private */
+  onActionButtonTap_: function() {
+    if (this.model.origin != this.origin_) {
+      // Only COOKIES category can be edited currently.
+      var category = settings.ContentSettingsTypes.COOKIES;

[dschuyler, 2017/02/23 01:31:49]

I see that the bug is specific to Cookies, but this edit feature was present in
other sections (other than cookies) in the old options. Can we make this match
that? Or at a minimum, add a TODO here and a crbug saying that this feature
needs to be expanded to the other content settings.

[dpapad, 2017/02/23 19:39:47]

Done, https://bugs.chromium.org/p/chromium/issues/detail?id=695578.

+
+      this.browserProxy_.resetCategoryPermissionForOrigin(
+          this.model.origin,
+          this.model.embeddingOrigin,
+          category,
+          this.model.incognito);
+
+      this.browserProxy_.setCategoryPermissionForOrigin(
+          this.origin_,
+          this.origin_,
+          category,
+          this.model.setting,
+          this.model.incognito);
+    }
+
+    this.$.dialog.close();
+  },
+
+  /**
+   * @param {!KeyboardEvent} e
+   * @private
+   */
+  onKeypress_: function(e) {
+    if (e.key == 'Enter' && !this.$.actionButton.disabled)
+      this.onActionButtonTap_();
+  },
+});