[MD settings] move cookie tree management to cookie tree behavior

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 109 matching lines @@
 
   /**
    * Add the cookie data to the content section of this dialog.
    * @param {string} id The id of the cookie node to display.
    * @param {!settings.CookieTreeNode} site The current site.
    * @private
    */
   populateItem_: function(id, site) {
     var node = site.fetchNodeById(id, true);
     if (node)
-      this.cookieNodes_ = site.getCookieData(node);
+      this.cookieNodes_ = getCookieData(node.data_);

[Dan Beam, 2016/10/27 01:31:55]

why is this accessing a private member?

[dschuyler, 2016/10/27 20:47:35]

I'd like to refactor the way the node data is fetched
and organized, but for this CL I added an accessor.

Done.

   },
 
   /**
    * 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 (this.site_.children_.length == 0 || this.entries_.length == 0) {
@@ skipping 53 matching lines @@
     cr.removeWebUIListener(this.listener_);
     this.browserProxy.removeCookie(this.site_.data_.id);
     this.close();
   },
 
   /** @private */
   onCancelTap_: function() {
     this.$.dialog.cancel();
   },
 });