[MD Bookmarks] Remove Multiple Selected Items.

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.
 
 var BookmarksStore = Polymer({
   is: 'bookmarks-store',
 
   properties: {
     /** @type {BookmarkTreeNode} */
     rootNode: {
@@ skipping 40 matching lines @@
   },
 
   /** @private {Object} */
   documentListeners_: null,
 
   /** @override */
   attached: function() {
     this.documentListeners_ = {
       'folder-open-changed': this.onFolderOpenChanged_.bind(this),
       'open-item': this.onItemOpened_.bind(this),
+      'remove-item': this.onItemRemoved_.bind(this),
       'search-term-changed': this.onSearchTermChanged_.bind(this),
       'select-item': this.onItemSelected_.bind(this),
       'selected-folder-changed': this.onSelectedFolderChanged_.bind(this),
     };
     for (var event in this.documentListeners_)
       document.addEventListener(event, this.documentListeners_[event]);
   },
 
   /** @override */
   detached: function() {
@@ skipping 196 matching lines @@
    * Selects a single item in the displayedList.
    * @param {BookmarkTreeNode} item
    * @private
    */
   selectItem_: function(item) {
     this.anchorIndex_ = this.getIndexInList_(item);
     this.set('displayedList.#' + this.anchorIndex_ + '.isSelectedItem', true);
     this.selectedIndexSet_.add(this.anchorIndex_);
   },
 
+  /**
+   * @param {BookmarkTreeNode} item
+   * @private
+   */
+  removeSingleItem_: function(item) {
+    if (item.url) {
+      chrome.bookmarks.remove(item.id, function() {
+        // TODO(jiaxi): Add toast later.
+      }.bind(this));
+    } else {
+      chrome.bookmarks.removeTree(item.id, function() {
+        // TODO(jiaxi): Add toast later.
+      }.bind(this));
+    }
+  },
+
   //////////////////////////////////////////////////////////////////////////////
   // bookmarks-store, bookmarks API event listeners:
 
   /**
    * 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,
@@ skipping 139 matching lines @@
         chrome.tabs.create({url: item.url});
     } else {
       for (let index of this.selectedIndexSet_) {
         if (!this.displayedList[index].url)
           continue;
 
         chrome.tabs.create({url: this.displayedList[index].url});
       }
     }
   },
+
+  /**
+   * @param {CustomEvent} e
+   * @private
+   */
+  onItemRemoved_: function(e) {
+    var item = /** BookmarkTreeNode */ (e.detail);
+    if (this.selectedIndexSet_.size == 1 || !item.isSelectedItem) {
+      this.removeSingleItem_(item);
+    } else {
+      var itemsToRemoved = [];
+      for (let index of this.selectedIndexSet_)
+        itemsToRemoved.push(this.displayedList[index]);
+
+      for (var i = 0; i < itemsToRemoved.length; i++)
+        this.removeSingleItem_(itemsToRemoved[i]);

[jiaxi, 2017/02/01 23:32:13]

This probably needs a test but I'm not sure how to write it. Should we just
multiple select items, add onRemoved listeners that adds a counter and just
check how many times the onRemoved has been called?

[tsergeant, 2017/02/02 02:12:15]

That sounds like a reasonable way to do it.

[jiaxi, 2017/02/02 05:37:19]

Done.

+    }
+  },
 });
 
 ////////////////////////////////////////////////////////////////////////////////
 // bookmarks-store, static methods:
 
 /**
  * Stores the path from the store to a node inside the node.
  * @param {BookmarkTreeNode} bookmarkNode
  * @param {number} startIndex
  */
@@ skipping 21 matching lines @@
     idToNodeMap[bookmarkNode.id] = bookmarkNode;
 
   if (bookmarkNode.url)
     return;
 
   bookmarkNode.isSelectedFolder = false;
   bookmarkNode.isOpen = true;
   for (var i = 0; i < bookmarkNode.children.length; i++)
     BookmarksStore.initNodes(bookmarkNode.children[i], idToNodeMap);
 };