Should not unregister BookmarkTreeItem from lookup table on handling moved event

chrome/browser/resources/bookmark_manager/js/bmm/bookmark_tree.js

 // Copyright (c) 2011 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('bmm', function() {
   /** @const */ var Tree = cr.ui.Tree;
   /** @const */ var TreeItem = cr.ui.TreeItem;
 
   var treeLookup = {};
@@ skipping 175 matching lines @@
         var parentItem = treeLookup[bookmarkNode.parentId];
         var newItem = new BookmarkTreeItem(bookmarkNode);
         addTreeItem(parentItem, newItem);
       }
     },
 
     handleMoved: function(id, moveInfo) {
       var treeItem = treeLookup[id];
       if (treeItem) {
         var oldParentItem = treeLookup[moveInfo.oldParentId];
-        oldParentItem.remove(treeItem);
+        TreeItem.prototype.remove.call(oldParentItem, treeItem);

[arv (Not doing code reviews), 2012/12/20 14:54:51]

This seems pretty hacky. Is there an alternative solution we could use?

         var newParentItem = treeLookup[moveInfo.parentId];
         // The tree only shows folders so the index is not the index we want. We
         // therefore get the children need to adjust the index.
         addTreeItem(newParentItem, treeItem);
       }
     },
 
     handleRemoved: function(id, removeInfo) {
       var parentItem = treeLookup[removeInfo.parentId];
       var itemToRemove = treeLookup[id];
@@ skipping 80 matching lines @@
     }
   };
 
   return {
     BookmarkTree: BookmarkTree,
     BookmarkTreeItem: BookmarkTreeItem,
     treeLookup: treeLookup,
     tree: tree
   };
 });