MD Settings: validate startup URLs

chrome/browser/resources/settings/on_startup_page/startup_urls_page.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.
 
 /**
  * @typedef {{
  *   'title': string,
  *   'tooltip': string,
  *   'url': string
  * }}
@@ skipping 24 matching lines @@
     prefs: {
       type: Object,
       notify: true,
     },
 
     /** @type {settings.StartupUrlsPageBrowserProxy} */
     browserProxy_: Object,
 
     /** @private {string} */
     newUrl_: {
+      observer: 'newUrlChanged_',
       type: String,
       value: '',
     },
 
+    isNewUrlValid_: {

[dpapad, 2016/03/31 18:39:56]

Nit(optional): When the default value is false you can shorten more

isNewUrlValid_ : Boolean,

[Dan Beam, 2016/03/31 22:06:54]

i wouldn't encourage this, as one might assume the default is false, but it's
not crystal clear from looking at this site

+      type: Boolean,
+      value: false,
+    },
+
     /**
      * Pages to load upon browser startup.
      * @private {!Array<!StartupPageInfo>}
      */
     startupPages_: Array,
   },
 
   created: function() {
     this.browserProxy_ = settings.StartupUrlsPageBrowserProxyImpl.getInstance();
   },
@@ skipping 27 matching lines @@
   /** @private */
   onUseCurrentPagesTap_: function() {
     this.browserProxy_.useCurrentPages();
   },
 
   /** @private */
   onCancelTap_: function() {
     this.$.addUrlDialog.close();
   },
 
-  /**
-   * @return {boolean} Whether tapping the Add button should be allowed.
-   * @private
-   */
-  isAddEnabled_: function() {
-    return this.browserProxy_.canAddPage(this.newUrl_);
+  /** @private */
+  newUrlChanged_: function() {
+    this.browserProxy_.validateStartupPage(this.newUrl_).then(function(valid) {
+      this.isNewUrlValid_ = valid;

[dpapad, 2016/03/31 18:39:56]

Nit (optional): s/valid/isValid

[Dan Beam, 2016/03/31 22:06:54]

Done.

+    }.bind(this));
   },
 
   /** @private */
   onAddTap_: function() {
-    assert(this.isAddEnabled_());
+    assert(this.isNewUrlValid_);
     this.browserProxy_.addStartupPage(this.newUrl_);
     this.$.addUrlDialog.close();
   },
 
   /**
    * @param {!{model: !{index: number}}} e
    * @private
    */
   onRemoveUrlTap_: function(e) {
     this.browserProxy_.removeStartupPage(e.model.index);
   },
 });