Site Settings Desktop: Polish the Site Data details dialog.

chrome/browser/resources/settings/site_settings/site_data_details_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
  * 'site-data-details-dialog' provides a dialog to show details of site data
  * stored by a given site.
  */
 Polymer({
@@ skipping 39 matching lines @@
    */
   open: function(site) {
     this.site_ = site;
     this.populateDialog_();
     this.listener_ = cr.addWebUIListener(
         'onTreeItemRemoved', this.onTreeItemRemoved_.bind(this));
     this.$.dialog.showModal();
   },
 
   /**
+   * Closes the dialog, if open.
+   */
+  close: function() {
+    var dialog = /** @type {!CrDialogElement} */(this.$.dialog);
+    if (dialog.open)
+      dialog.close();
+  },
+
+  /**
    * Populates the dialog with the data about the site.
    * @private
    */
   populateDialog_: function() {
     this.title_ = loadTimeData.getStringF('siteSettingsCookieDialog',
                                           this.site_.data_.title);
 
     this.entries_ = this.site_.getCookieList();
     if (this.entries_.length < 2) {
       // When there's only one item to show, hide the picker and change the
@@ skipping 51 matching lines @@
       site.addCookieData(root, node);
   },
 
   /**
    * Called when a single item has been removed.
    * @param {!CookieRemovePacket} args The details about what to remove.
    * @private
    */
   onTreeItemRemoved_: function(args) {
     this.entries_ = this.site_.getCookieList();
-    if (args.id == this.site_.data_.id || this.entries_.length == 0) {
-      this.$.dialog.close();
+    if (this.site_.children_.length == 0 || this.entries_.length == 0) {
+      this.close();
       return;
     }
 
     if (this.entries_.length <= this.lastSelectedIndex_)
       this.lastSelectedIndex_ = this.entries_.length - 1;
     var selectedId = this.entries_[this.lastSelectedIndex_].id;
     this.$.picker.selected = selectedId;
     this.populateItem_(selectedId, this.site_);
   },
 
@@ skipping 33 matching lines @@
         this.site_, this.site_.data_.id, this.$.picker.selected));
   },
 
   /**
    * A handler for when the user opts to remove all cookies.
    * @private
    */
   onRemoveAll_: function(event) {
     cr.removeWebUIListener(this.listener_);
     this.browserProxy.removeCookie(this.site_.data_.id);
-    this.$.dialog.close();
+    this.close();
   },
 
   /** @private */
   onCancelTap_: function() {
     this.$.dialog.cancel();
   },
 });