MD Bookmarks: Remove deleted nodes from state tree

chrome/browser/resources/md_bookmarks/actions.js

 // Copyright 2017 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 Module for functions which produce action objects. These are
  * listed in one place to document available actions and their parameters.
  */
 
 cr.define('bookmarks.actions', function() {
@@ skipping 26 matching lines @@
       index: index,
       oldParentId: oldParentId,
       oldIndex: oldIndex,
     };
   }
 
   /**
    * @param {string} id
    * @param {string} parentId
    * @param {number} index
+   * @param {NodeList} nodes
    * @return {!Action}
    */
-  function removeBookmark(id, parentId, index) {
+  function removeBookmark(id, parentId, index, nodes) {

[tsergeant, 2017/03/23 05:36:07]

My intention is still that actions should be able to access the store state
without being passed it in.

This has some difficulties with Closure (yayyyyy) and testing that I haven't
fully figured out yet. I'll get to it next week, but wanted to send this CL to
you in the meantime anyway.

+    var descendants = bookmarks.util.getDescendants(nodes, id);
     return {
       name: 'remove-bookmark',
       id: id,
+      descendants: descendants,
       parentId: parentId,
       index: index,
     };
   }
 
   /**
    * @param {NodeList} nodeMap
    * @return {!Action}
    */
   function refreshNodes(nodeMap) {
@@ skipping 104 matching lines @@
     editBookmark: editBookmark,
     moveBookmark: moveBookmark,
     refreshNodes: refreshNodes,
     removeBookmark: removeBookmark,
     selectFolder: selectFolder,
     selectItem: selectItem,
     setSearchResults: setSearchResults,
     setSearchTerm: setSearchTerm,
   };
 });