Refactoring CookiesTreeModel to support multiple data sources.

chrome/browser/resources/options2/cookies_list.js

 // Copyright (c) 2012 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.
 
 cr.define('options', function() {
   /** @const */ var DeletableItemList = options.DeletableItemList;
   /** @const */ var DeletableItem = options.DeletableItem;
   /** @const */ var ArrayDataModel = cr.ui.ArrayDataModel;
   /** @const */ var ListSingleSelectionModel = cr.ui.ListSingleSelectionModel;
 
@@ skipping 587 matching lines @@
     },
 
     /**
      * The cookie tree path id.
      * @type {string}
      */
     get pathId() {
       var parent = this.parent;
       if (parent && parent instanceof CookieTreeNode)
         return parent.pathId + ',' + this.data.id;
+      // TODO(nasko): until the code is modified to account for the "app" level
+      // in the tree model, use this hack of adding the root node id.
+      if ($('cookies-list').rootId)
+        return $('cookies-list').rootId + ',' + this.data.id;
+
       return this.data.id;
     },
   };
 
   /**
    * Creates a new cookies list.
    * @param {Object=} opt_propertyBag Optional properties.
    * @constructor
    * @extends {DeletableItemList}
    */
@@ skipping 201 matching lines @@
 
     /**
      * Removes tree nodes by parent id.
      * This is used by cookies_view.js.
      * @param {string} parentId Id of the parent node.
      * @param {number} start The index at which to start removing the nodes.
      * @param {number} count Number of nodes to remove.
      */
     removeByParentId: function(parentId, start, count) {
       var parent = parentId ? parentLookup[parentId] : this;
-      if (!parent)
-        return;
+      if (!parent) {
+        // TODO(nasko): Remove this once the "app" level in the tree model is
+        // accounted for.
+        if (parentId) {
+          parent = this;
+        } else {
+          return;
+        }
+      }
 
       parent.startBatchUpdates();
       while (count-- > 0)
         parent.remove(start);
       parent.endBatchUpdates();
 
       cr.dispatchSimpleEvent(this, 'change');
     },
 
     /**
      * Loads the immediate children of given parent node.
      * This is used by cookies_view.js.
      * @param {string} parentId Id of the parent node.
      * @param {Array} children The immediate children of parent node.
      */
     loadChildren: function(parentId, children) {
       if (parentId)
         delete lookupRequests[parentId];
       var parent = parentId ? parentLookup[parentId] : this;
-      if (!parent)
-        return;
+      // TODO(nasko): Remove this once the "app" level in the tree model is
+      // accounted for.
+      if (!parent) {
+        if (parentId) {
+          parent = this;
+        } else {
+          return;
+        }
+      }
 
       parent.startBatchUpdates();
       parent.clear();
       this.addByParent_(parent, 0, children);
       parent.endBatchUpdates();
     },
   };
 
   return {
     CookiesList: CookiesList
   };
 });