MD Settings: Manage search engines 3/3, hooking up UI.

chrome/browser/resources/settings/search_engines_page/search_engine_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-search-engine-dialog' is a component for adding
+ * or editing a search engine entry.
+ *
+ * @group Chrome Settings Elements
+ * @element settings-search-engine-dialog
+ */
+Polymer({
+  is: 'settings-search-engine-dialog',
+
+  properties: {
+    /**
+     * The search engine to be edited. If not populated a new search engine
+     * should be added.
+     * @type {?SearchEngine}
+     */
+    model: Object,
+
+    /** @private {string} */
+    searchEngine_: String,
+
+    /** @private {string} */
+    keyword_: String,
+
+    /** @private {string} */
+    queryUrl_: String,
+
+    /** @private {string} */
+    dialogTitle_: String,
+
+    /** @private {string} */
+    actionButtonText_: String,
+  },
+
+  /** @private {!settings.SearchEnginesBrowserProxy} */
+  browserProxy_: null,

[Dan Beam, 2016/02/18 23:27:41]

i was wondering if we [should] do this

i assume you're putting this outside of properties because it's never used in
bindings or computed/observers?

[dpapad, 2016/02/18 23:45:46]

Yes, that is the reason.

On top of that I actually tried making this a proper Polymer property and inject
it the same way I am injecting SearchEngine objects (see "model" property a few
lines above), instead of doing the getInstance() thing, but to my surprise I
found out that Polymer properties that are involved in bindings, can't be
arbitrary class instances (with prototypes), they can only be Array, Object, and
primitives it seems.

[Dan Beam, 2016/02/18 23:51:05]

I think type: Object would work

  /** @private {!settings.SearchEnginesBrowserProxy} */
  browserProxy_: {
    type: Object,
    value: null,
  },

+
+  /**
+   * The |modelIndex| to use when a new search engine is added. Must match with
+   * kNewSearchEngineIndex constant specified at
+   * chrome/browser/ui/webui/settings/search_engines_handler.cc
+   * @const {number}
+   */
+  DEFAULT_MODEL_INDEX: -1,
+
+  /** @override */
+  created: function() {
+    this.browserProxy_ = settings.SearchEnginesBrowserProxyImpl.getInstance();
+  },
+
+  /** @override */
+  ready: function() {
+    if (this.model) {
+      // TODO(dpapad): Localize strings.
+      this.dialogTitle_ = 'Edit search engine';
+      this.actionButtonText_ = 'Edit';
+
+      // If editing an existing search engine, pre-populate the input fields.
+      this.searchEngine_ = this.model.displayName;
+      this.keyword_ = this.model.keyword;
+      this.queryUrl_ = this.model.url;
+    } else {
+      // TODO(dpapad): Localize string.
+      this.dialogTitle_ = 'Add search engine';
+      this.actionButtonText_ = loadTimeData.getString('searchEnginesAdd');
+    }
+  },
+
+  /** @override */
+  attached: function() {
+    this.updateActionButtonState_();
+    this.browserProxy_.searchEngineEditStarted(
+        this.model ? this.model.modelIndex : this.DEFAULT_MODEL_INDEX);
+    this.$.dialog.open();
+  },
+
+  /** @private */
+  onCancelTap_: function() {
+    this.browserProxy_.searchEngineEditCancelled();
+    this.$.dialog.close();
+  },
+
+  /** @private */
+  onActionButtonTap_: function() {
+    this.browserProxy_.searchEngineEditCompleted(
+        this.searchEngine_, this.keyword_, this.queryUrl_);
+    this.$.dialog.close();
+  },
+
+  /**
+   * @param {!Event} event
+   * @private
+   */
+  validate_: function(event) {
+    var inputElement = Polymer.dom(event).localTarget;
+
+    this.browserProxy_.validateSearchEngineInput(
+        inputElement.id, inputElement.value).then(function(isValid) {
+      inputElement.invalid = !isValid;
+      this.updateActionButtonState_();
+    }.bind(this));
+  },
+
+  /** @private */
+  updateActionButtonState_: function() {
+    var allValid = [
+      this.$.searchEngine, this.$.keyword, this.$.queryUrl
+    ].every(function(inputElement) {
+      return !inputElement.invalid && inputElement.value.length != 0;
+    });
+    this.$.actionButton.disabled = !allValid;
+  },
+});