MD Bookmarks: Scroll and select items that are added to the main list

chrome/browser/resources/md_bookmarks/dnd_manager.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.
 
 
 /** @typedef {?{elements: !Array<BookmarkNode>, sameProfile: boolean}} */
 var NormalizedDragData;
 
 cr.define('bookmarks', function() {
   /**
@@ skipping 446 matching lines @@
      * @private
      */
     onMouseUp_: function() {
       if (!this.internalDragElement_)
         return;
 
       if (this.dropDestination_) {
         // Complete the drag by moving all dragged items to the drop
         // destination.
         var dropInfo = this.calculateDropInfo_(this.dropDestination_);
-        this.dragInfo_.dragData.elements.forEach((item) => {
-          chrome.bookmarks.move(item.id, {
-            parentId: dropInfo.parentId,
-            index: dropInfo.index == -1 ? undefined : dropInfo.index
-          });
+        var shouldHighlight = this.shouldHighlight_(this.dropDestination_);
+        var dragElements = this.dragInfo_.dragData.elements;
+        var completedMoves = 0;
+
+        if (shouldHighlight)
+          bookmarks.ApiListener.trackUpdatedItems();
+
+        dragElements.forEach((item) => {
+          chrome.bookmarks.move(
+              item.id, {
+                parentId: dropInfo.parentId,
+                index: dropInfo.index == -1 ? undefined : dropInfo.index
+              },
+              function() {
+                completedMoves += 1;
+                if (shouldHighlight && completedMoves == dragElements.length)
+                  bookmarks.ApiListener.highlightUpdatedItems();

[calamity, 2017/07/26 05:36:06]

optional: Consider using an array of PromiseResolvers and then Promise.all()ing
them.

[tsergeant, 2017/07/26 06:34:07]

Done, it works out a little bit nicer.

+              });
         });
       }
 
       this.clearDragData_();
     },
 
     ////////////////////////////////////////////////////////////////////////////
     // DragEvent handlers:
 
     /**
@@ skipping 32 matching lines @@
 
     /**
      * @private
      * @param {!Event} e
      */
     onDrop_: function(e) {
       if (this.dropDestination_) {
         e.preventDefault();
 
         var dropInfo = this.calculateDropInfo_(this.dropDestination_);
-        if (dropInfo.index != -1)
-          chrome.bookmarkManagerPrivate.drop(dropInfo.parentId, dropInfo.index);
-        else
-          chrome.bookmarkManagerPrivate.drop(dropInfo.parentId);
+        var index = dropInfo.index != -1 ? dropInfo.index : undefined;
+        var shouldHighlight = this.shouldHighlight_(this.dropDestination_);
+
+        if (shouldHighlight)
+          bookmarks.ApiListener.trackUpdatedItems();
+
+        chrome.bookmarkManagerPrivate.drop(
+            dropInfo.parentId, index,
+            shouldHighlight ? bookmarks.ApiListener.highlightUpdatedItems :
+                              undefined);
       }
 
       this.clearDragData_();
     },
 
     /**
      * @private
      * @param {Event} e
      */
     onDragEnter_: function(e) {
@@ skipping 317 matching lines @@
             state.nodes[state.selectedFolder].children.length == 0;
       }
 
       // We can only drop on a folder.
       if (getBookmarkNode(overElement).url)
         return false;
 
       return !this.dragInfo_.isDraggingChildBookmark(overElement.itemId);
     },
 
+    /**
+     * @param {DropDestination} dropDestination
+     * @private
+     */
+    shouldHighlight_: function(dropDestination) {
+      return isBookmarkItem(dropDestination.element) ||
+          isBookmarkList(dropDestination.element);
+    },
+
     /** @param {bookmarks.TimerProxy} timerProxy */
     setTimerProxyForTesting: function(timerProxy) {
       this.timerProxy_ = timerProxy;
       this.dropIndicator_.timerProxy = timerProxy;
     },
 
     /** @return {BookmarksDndChipElement} */
     get dndChip() {
       if (!this.chip_) {
         this.chip_ =
             /** @type {BookmarksDndChipElement} */ (
                 document.createElement('bookmarks-dnd-chip'));
         document.body.appendChild(this.chip_);
       }
 
       return this.chip_;
     },
   };
 
   return {
     DNDManager: DNDManager,
     DragInfo: DragInfo,
     DropIndicator: DropIndicator,
   };
 });