MD Settings: On Startup, extract dialog to its own file.

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
  * }}
  */
 var StartupPageInfo;
 
 /**
  * @fileoverview 'settings-startup-urls-page' is the settings page
  * containing the urls that will be opened when chrome is started.
- *
- * Example:
- *
- *    <neon-animated-pages>
- *      <settings-startup-urls-page prefs="{{prefs}}">
- *      </settings-startup-urls-page>
- *      ... other pages ...
- *    </neon-animated-pages>
  */
 Polymer({
   is: 'settings-startup-urls-page',
 
   behaviors: [WebUIListenerBehavior],
 
   properties: {
-    /**
-     * Preferences state.
-     */
-    prefs: {
-      type: Object,
-      notify: true,
-    },
-
     /** @type {settings.StartupUrlsPageBrowserProxy} */
     browserProxy_: Object,
 
-    /** @private {string} */
-    newUrl_: {
-      observer: 'newUrlChanged_',
-      type: String,
-      value: '',
-    },
-
-    isNewUrlValid_: {
-      type: Boolean,
-      value: false,
-    },
-
     /**
      * Pages to load upon browser startup.
      * @private {!Array<!StartupPageInfo>}
      */
     startupPages_: Array,
+
+    showStartupUrlDialog_: Boolean,
   },
 
-  created: function() {
+  /** @override */
+  attached: function() {
     this.browserProxy_ = settings.StartupUrlsPageBrowserProxyImpl.getInstance();
-  },
-
-  attached: function() {
-    this.addWebUIListener('update-startup-pages',
-                          this.updateStartupPages_.bind(this));
+    this.addWebUIListener('update-startup-pages', function(startupPages) {
+      this.startupPages_ = startupPages;
+    }.bind(this));
     this.browserProxy_.loadStartupPages();
   },
 
   /**
    * @param {string} url Location of an image to get a set of icons fors.
    * @return {string} A set of icon URLs.
    * @private
    */
   getIconSet_: function(url) {
     return getFaviconImageSet(url);
   },
 
-  /** @private */
-  updateStartupPages_: function(startupPages) {
-    this.startupPages_ = startupPages;
-  },
-
-  /** @private */
+  /**
+   * Opens the dialog and registers a listener for removing the dialog from the
+   * DOM once is closed. The listener is destroyed when the dialog is removed
+   * (because of 'restamp').
+   * @private
+   */
   onAddPageTap_: function() {
-    this.newUrl_ = '';
-    this.$.addUrlDialog.open();
+    this.showStartupUrlDialog_ = true;
+    this.async(function() {
+      var dialog = this.$$('settings-startup-url-dialog');
+      dialog.addEventListener('iron-overlay-closed', function() {
+        this.showStartupUrlDialog_ = false;
+      }.bind(this));
+    }.bind(this));
   },
 
   /** @private */
   onUseCurrentPagesTap_: function() {
     this.browserProxy_.useCurrentPages();
   },
 
-  /** @private */
-  onCancelTap_: function() {
-    this.$.addUrlDialog.close();
-  },
-
   /**
-   * @param {string} newUrl
-   * @private
-   */
-  newUrlChanged_: function(newUrl) {
-    if (this.validationResolver_)
-      this.validationResolver_.reject(false);
-
-    /** @type {?PromiseResolver<boolean>} */
-    this.validationResolver_ = this.browserProxy_.validateStartupPage(newUrl);
-
-    this.validationResolver_.promise.then(function(isValid) {
-      this.isNewUrlValid_ = isValid;
-      this.validationResolver_ = null;
-    }.bind(this), function() {
-      // Squelchs console errors.
-    });
-  },
-
-  /** @private */
-  onAddTap_: function() {
-    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);
   },
 });