[MD Bookmarks] Edit Bookmarks.

chrome/browser/resources/md_bookmarks/store.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.
 
 Polymer({
   is: 'bookmarks-store',
 
   properties: {
     /** @type {BookmarkTreeNode} */
     rootNode: {
@@ skipping 37 matching lines @@
   },
 
   /**
    * Initializes the store with data from the bookmarks API.
    * Called by app on attached.
    */
   initializeStore: function() {
     chrome.bookmarks.getTree(function(results) {
       this.setupStore_(results[0]);
     }.bind(this));
+    // Attach bookmarks API listeners.
     chrome.bookmarks.onRemoved.addListener(this.onBookmarkRemoved_.bind(this));
+    chrome.bookmarks.onChanged.addListener(this.onBookmarkChanged_.bind(this));
   },
 
+////////////////////////////////////////////////////////////////////////////////
+// bookmarks-store, private:

[tsergeant, 2017/01/09 00:21:05]

👍 I like this lots

+
   /**
    * @param {BookmarkTreeNode} rootNode
    * @private
    */
   setupStore_: function(rootNode) {
     this.rootNode = rootNode;
     this.idToNodeMap_ = {};
     this.rootNode.path = 'rootNode';
     this.generatePaths_(rootNode, 0);
     this.initNodes_(this.rootNode);
     this.fire('selected-folder-changed', this.rootNode.children[0].id);
   },
 
   /**
-   * Selects the folder specified by the event and deselects the previously
-   * selected folder.
-   * @param {CustomEvent} e
+   * Initializes the nodes in the bookmarks tree as follows:
+   * - Populates |idToNodeMap_| with a mapping of all node ids to their
+   *   corresponding BookmarkTreeNode.
+   * - Sets all the nodes to not selected and open by default.
+   * @param {BookmarkTreeNode} bookmarkNode
    * @private
    */
-  onSelectedFolderChanged_: function(e) {
-    // Deselect the old folder if defined.
-    if (this.selectedId)
-      this.set(this.idToNodeMap_[this.selectedId].path + '.isSelected', false);
+  initNodes_: function(bookmarkNode) {
+    this.idToNodeMap_[bookmarkNode.id] = bookmarkNode;
+    if (bookmarkNode.url)
+      return;
 
-    var selectedId = /** @type {string} */ (e.detail);
-    var newFolder = this.idToNodeMap_[selectedId];
-    this.set(newFolder.path + '.isSelected', true);
-    this.selectedId = selectedId;
+    bookmarkNode.isSelected = false;
+    bookmarkNode.isOpen = true;
+    for (var i = 0; i < bookmarkNode.children.length; i++) {
+      this.initNodes_(bookmarkNode.children[i]);
+    }
   },
 
   /**
-   * Handles events that open and close folders.
-   * @param {CustomEvent} e
-   * @private
-   */
-  onFolderOpenChanged_: function(e) {
-    var folder = this.idToNodeMap_[e.detail.id];
-    this.set(folder.path + '.isOpen', e.detail.open);
-    if (!folder.isOpen && this.isAncestorOfSelected_(folder))
-      this.fire('selected-folder-changed', folder.id);
-  },
-
-  /**
    * @param {BookmarkTreeNode} folder
    * @private
    * @return {boolean}
    */
   isAncestorOfSelected_: function(folder) {
     return this.selectedNode.path.startsWith(folder.path);
   },
 
   /** @private */
   updateSelectedNode_: function() {
     var selectedNode = this.idToNodeMap_[this.selectedId];
     this.linkPaths('selectedNode', selectedNode.path);
     this._setSelectedNode(selectedNode);
   },
 
   /**
+   * Stores the path from the store to a node inside the node.
+   * @param {BookmarkTreeNode} bookmarkNode
+   * @param {number} startIndex
+   * @private
+   */
+  generatePaths_: function(bookmarkNode, startIndex) {
+    if (!bookmarkNode.children)
+      return;
+
+    for (var i = startIndex; i < bookmarkNode.children.length; i++) {
+      bookmarkNode.children[i].path = bookmarkNode.path + '.children.#' + i;
+      this.generatePaths_(bookmarkNode.children[i], 0);
+    }
+  },
+
+  /**
+   * Remove all descendants of a given node from the map.
+   * @param {string} id
+   * @private
+   */
+  removeDescendantsFromMap_: function(id) {
+    var node = this.idToNodeMap_[id];
+    if (!node)
+      return;
+
+    if (node.children) {
+      for (var i = 0; i < node.children.length; i++)
+        this.removeDescendantsFromMap_(node.children[i].id);
+    }
+    delete this.idToNodeMap_[id];
+  },
+
+////////////////////////////////////////////////////////////////////////////////
+// bookmarks-store, bookmarks API event listeners:
+

[tsergeant, 2017/01/09 00:21:04]

Just one blank line here

[jiaxi, 2017/01/09 02:17:46]

Done.

+
+  /**
    * Callback for when a bookmark node is removed.
    * If a folder is selected or is an ancestor of a selected folder, the parent
    * of the removed folder will be selected.
    * @param {string} id The id of the removed bookmark node.
    * @param {!{index: number,
    *           parentId: string,
    *           node: BookmarkTreeNode}} removeInfo
    */
   onBookmarkRemoved_: function(id, removeInfo) {
     if (this.isAncestorOfSelected_(this.idToNodeMap_[id]))
       this.fire('selected-folder-changed', removeInfo.parentId);
 
     var parentNode = this.idToNodeMap_[removeInfo.parentId];
     this.splice(parentNode.path + '.children', removeInfo.index, 1);
     this.removeDescendantsFromMap_(id);
     this.generatePaths_(parentNode, removeInfo.index);
   },
 
   /**
-   * Initializes the nodes in the bookmarks tree as follows:
-   * - Populates |idToNodeMap_| with a mapping of all node ids to their
-   *   corresponding BookmarkTreeNode.
-   * - Sets all the nodes to not selected and open by default.
-   * @param {BookmarkTreeNode} bookmarkNode
+   * Called when the title of a bookmark changes.
+   * @param {string} id The id of changed bookmark node.
+   * @param {!Object} changeInfo
+   */
+  onBookmarkChanged_: function(id, changeInfo) {
+    if (changeInfo.title)
+      this.set(this.idToNodeMap_[id].path + '.title', changeInfo.title);
+    if (changeInfo.url)
+      this.set(this.idToNodeMap_[id].path + '.url', changeInfo.url);
+  },
+
+////////////////////////////////////////////////////////////////////////////////
+// bookmarks-store, bookmarks app event listeners:
+
+  /**
+   * Selects the folder specified by the event and deselects the previously
+   * selected folder.
+   * @param {CustomEvent} e
    * @private
    */
-  initNodes_: function(bookmarkNode) {
-    this.idToNodeMap_[bookmarkNode.id] = bookmarkNode;
-    if (bookmarkNode.url)
-      return;
+  onSelectedFolderChanged_: function(e) {
+    // Deselect the old folder if defined.
+    if (this.selectedId)
+      this.set(this.idToNodeMap_[this.selectedId].path + '.isSelected', false);
 
-    bookmarkNode.isSelected = false;
-    bookmarkNode.isOpen = true;
-    for (var i = 0; i < bookmarkNode.children.length; i++) {
-      this.initNodes_(bookmarkNode.children[i]);
-    }
+    var selectedId = /** @type {string} */ (e.detail);
+    var newFolder = this.idToNodeMap_[selectedId];
+    this.set(newFolder.path + '.isSelected', true);
+    this.selectedId = selectedId;
   },
 
   /**
-   * Stores the path from the store to a node inside the node.
-   * @param {BookmarkTreeNode} bookmarkNode
-   * @param {number} startIndex
+   * Handles events that open and close folders.
+   * @param {CustomEvent} e
    * @private
    */
-  generatePaths_: function(bookmarkNode, startIndex) {
-    if (!bookmarkNode.children)
-      return;
-
-    for (var i = startIndex; i < bookmarkNode.children.length; i++) {
-      bookmarkNode.children[i].path = bookmarkNode.path + '.children.#' + i;
-      this.generatePaths_(bookmarkNode.children[i], 0);
-    }
+  onFolderOpenChanged_: function(e) {
+    var folder = this.idToNodeMap_[e.detail.id];
+    this.set(folder.path + '.isOpen', e.detail.open);
+    if (!folder.isOpen && this.isAncestorOfSelected_(folder))
+      this.fire('selected-folder-changed', folder.id);
   },
-
-  /**
-   * Remove all descendants of a given node from the map.
-   * @param {string} id
-   * @private
-   */
-  removeDescendantsFromMap_: function(id) {
-    var node = this.idToNodeMap_[id];
-    if (!node)
-      return;
-
-    if (node.children) {
-      for (var i = 0; i < node.children.length; i++)
-        this.removeDescendantsFromMap_(node.children[i].id);
-    }
-    delete this.idToNodeMap_[id];
-  }
 });