Firefox overlay is seen opened even if Import Overlay is Cancelled.

chrome/browser/resources/settings/people_page/import_data_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-import-data-dialog' is a component for importing
  * bookmarks and other data from other sources.
  */
 Polymer({
   is: 'settings-import-data-dialog',
@@ skipping 111 matching lines @@
   /** @private */
   onActionButtonTap_: function() {
     if (this.isImportFromFileSelected_())
       this.browserProxy_.importFromBookmarksFile();
     else
       this.browserProxy_.importData(this.$.browserSelect.selectedIndex);
   },
 
   /** @private */
   closeDialog_: function() {
+    this.browserProxy_.closeDialog();

[Ilya Sherman, 2017/03/27 15:21:52]

This line is confusing: One dialog is already being closed, and here you're
sending a message to close a different dialog... but that's not very obvious
from context.  Please add a comment here to explain.  (Having a more specific
method name than "closeDialog()" will also help.)

[nikhil.sahni, 2017/03/28 11:22:00]

Done.

     this.$.dialog.close();
   },
 
   /**
    * @return {boolean} Whether the import button should be disabled.
    * @private
    */
   shouldDisableImport_: function() {
     return this.hasImportStatus_(settings.ImportDataStatus.IN_PROGRESS) ||
         this.noImportDataTypeSelected_;
   },
 });