Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4761)

Unified Diff: chrome/browser/resources/ntp/drag_drop_controller.js

Issue 6543017: Track which apps the user has re-ordered on the NTP. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix presubmit check Created 9 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/resources/ntp/apps.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/ntp/drag_drop_controller.js
diff --git a/chrome/browser/resources/ntp/drag_drop_controller.js b/chrome/browser/resources/ntp/drag_drop_controller.js
index affa9de3ac0a02ee60e61aac6256ac6695235f21..92aaebd5069c2cd459234a3754ca485091f7e297 100644
--- a/chrome/browser/resources/ntp/drag_drop_controller.js
+++ b/chrome/browser/resources/ntp/drag_drop_controller.js
@@ -23,9 +23,10 @@
// tells the delegate that the dragged item is currently above
// the specified coordinates.
//
-// saveDrag() -->
+// saveDrag(draggedItem) -->
// tells the delegate that the drag is done. move the item to the
-// position last specified by setDragPlaceholder. (e.g., commit changes)
+// position last specified by setDragPlaceholder (e.g., commit changes).
+// draggedItem was the item being dragged.
//
// The distance, in px, that the mouse must move before initiating a drag.
@@ -162,7 +163,7 @@ DragAndDropController.prototype = {
return;
this.delegate_.dragItem = this.startItem_ = null;
- this.delegate_.saveDrag();
+ this.delegate_.saveDrag(dragItem);
dragItem.classList.remove('dragging');
setTimeout(function() {
« no previous file with comments | « chrome/browser/resources/ntp/apps.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698