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

Unified Diff: chrome/browser/resources/file_manager/js/drag_selector.js

Issue 17287004: Made the visibility of Buttom bar updated at the end of drag selection. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 6 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
Index: chrome/browser/resources/file_manager/js/drag_selector.js
diff --git a/chrome/browser/resources/file_manager/js/drag_selector.js b/chrome/browser/resources/file_manager/js/drag_selector.js
index 93bf1c5953e7bf63bd331b2bb39eea62ed015c6b..bd77e660c1531765c0959dedb9fc18e3c34fea75 100644
--- a/chrome/browser/resources/file_manager/js/drag_selector.js
+++ b/chrome/browser/resources/file_manager/js/drag_selector.js
@@ -238,7 +238,10 @@ DragSelector.prototype.onMouseUp_ = function(event) {
'mousemove', this.onMouseMoveBound_, true);
this.target_.ownerDocument.removeEventListener(
'mouseup', this.onMouseUpBound_, true);
+ this.target_.dispatchEvent(new Event('dragselectionend'));
yoshiki 2013/06/20 03:11:25 Is it possible to use cr.dispatchSimpleEvent(this.
hirono 2013/06/20 03:55:03 Done.
this.target_.cachedBounds = null;
- event.stopPropagation();
this.target_ = null;
+ // The target may select an item by reacting to the mouseup event.
+ // This surpress to the selecting behavior.
+ event.stopPropagation();
};

Powered by Google App Engine
This is Rietveld 408576698