MD Settings: Add import data dialog.

chrome/browser/resources/settings/people_page/import_data_dialog.js

Index: chrome/browser/resources/settings/people_page/import_data_dialog.js
diff --git a/chrome/browser/resources/settings/people_page/import_data_dialog.js b/chrome/browser/resources/settings/people_page/import_data_dialog.js
new file mode 100644
index 0000000000000000000000000000000000000000..a90481beeddacde6d742ced2fd59621331f619e5
--- /dev/null
+++ b/chrome/browser/resources/settings/people_page/import_data_dialog.js
@@ -0,0 +1,74 @@
+// 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-import-data-dialog' is a component for importing
+ * bookmarks and other data from other sources.
+ */
+Polymer({
+  is: 'settings-import-data-dialog',
+
+  behaviors: [I18nBehavior, WebUIListenerBehavior],
+
+  properties: {
+    /** @private {!Array<!settings.BrowserProfile>} */
+    browserProfiles_: Array,
+
+    /** @private {!settings.BrowserProfile} */
+    selected_: Object,

[Dan Beam, 2016/11/14 22:06:51]

can we change this from a set of booleans to just prefs?

[dpapad, 2016/11/14 22:19:30]

My plan was to investigate the prefs involved in follow up CL, to address the
checkboxes syncing across tabs. I am not familiar with the C++ side of things
yet. Goal of this CL is mainly to add the dummy dialog, not hooked up to
anything, and not functional yet.

Do you mean, in this CL? Or is it OK to investigate your suggestion for follow
up? FWIW, in this CL I relied on using the proxy as it was landed at
https://chromium.googlesource.com/chromium/src/+/4a4963ec66964b69df3c677384f6....

[dpapad, 2016/11/14 23:18:03]

Done, I added this in this CL. Checkboxes are now syncing across tabs.

Regarding the contents of selected_, the booleans are not same as the prefs.
These booleans depict whether the pref is relevant for the selected browser (for
example the "Import from HTML file" case has all booleans set to false except
one (not prefs).

+  },
+
+  /** @private {?settings.ImportDataBrowserProxy} */
+  browserProxy_: null,
+
+  /** @override */
+  attached: function() {
+    this.browserProxy_ = settings.ImportDataBrowserProxyImpl.getInstance();
+    this.browserProxy_.initializeImportDialog().then(function(data) {
+      this.browserProfiles_ = data;
+      this.selected_ = this.browserProfiles_[0];
+    }.bind(this));
+
+    this.addWebUIListener('import-data-status-changed', function(e) {
+      // TODO(dpapad): Handle events to show spinner or success message.
+    });
+
+    this.$.dialog.showModal();
+  },
+
+  /** @private */
+  isImportFromFileSelected_: function() {
+    // The last entry in |browserProfiles_| always refers to dummy profile for
+    // importing from a bookmarks file.
+    return this.selected_.index == this.browserProfiles_.length - 1;
+  },
+
+  /**
+   * @return {string}
+   * @private
+   */
+  getActionButtonText_: function() {
+    return this.i18n(this.isImportFromFileSelected_() ?
+        'importChooseFile' : 'importCommit');
+  },
+
+  /** @private */
+  onChange_: function() {
+    this.selected_ = this.browserProfiles_[this.$.browserSelect.selectedIndex];
+  },
+
+  /** @private */
+  onActionButtonTap_: function() {
+    if (this.isImportFromFileSelected_()) {
+      this.browserProxy_.importFromBookmarksFile();
+    } else {
+      // TODO(dpapad): Implement this.
+    }
+  },
+
+  /** @private */
+  onCancelTap_: function() {
+    this.$.dialog.close();
+  },
+});