Bug 336241 canDrop isn't called when just the drag and drop action changes. r=Neil,sr=roc

This commit is contained in:
bugzilla%standard8.demon.co.uk 2006-05-12 16:30:53 +00:00
parent a4c160e338
commit 3d9664eaf8
2 changed files with 20 additions and 3 deletions

View File

@ -30,6 +30,7 @@
* Pierre Chanial <pierrechanial@netscape.net>
* Rene Pronk <r.pronk@its.tudelft.nl>
* Nate Nielsen <nielsen@memberwebs.com>
* Mark Banner <mark@standard8.demon.co.uk>
*
* Alternatively, the contents of this file may be used under the terms of
* either of the GNU General Public License Version 2 or later (the "GPL"),
@ -2167,9 +2168,13 @@ nsTreeBodyFrame::HandleEvent(nsPresContext* aPresContext,
// Cache the drag session.
nsCOMPtr<nsIDragService> dragService =
do_GetService("@mozilla.org/widget/dragservice;1");
nsCOMPtr<nsIDragSession> dragSession;
dragService->GetCurrentSession(getter_AddRefs(mSlots->mDragSession));
NS_ASSERTION(mSlots->mDragSession, "can't get drag session");
if (mSlots->mDragSession)
mSlots->mDragSession->GetDragAction(&mSlots->mDragAction);
else
mSlots->mDragAction = 0;
}
else if (aEvent->message == NS_DRAGDROP_OVER) {
// The mouse is hovering over this tree. If we determine things are
@ -2189,6 +2194,11 @@ nsTreeBodyFrame::HandleEvent(nsPresContext* aPresContext,
PRInt16 lastDropOrient = mSlots->mDropOrient;
PRInt16 lastScrollLines = mSlots->mScrollLines;
// Find out the current drag action
PRUint32 lastDragAction = mSlots->mDragAction;
if (mSlots->mDragSession)
mSlots->mDragSession->GetDragAction(&mSlots->mDragAction);
// Compute the row mouse is over and the above/below/on state.
// Below we'll use this to see if anything changed.
// Also check if we want to auto-scroll.
@ -2223,7 +2233,10 @@ nsTreeBodyFrame::HandleEvent(nsPresContext* aPresContext,
// If changed from last time, invalidate primary cell at the old location and if allowed,
// invalidate primary cell at the new location. If nothing changed, just bail.
if (mSlots->mDropRow != lastDropRow || mSlots->mDropOrient != lastDropOrient) {
if (mSlots->mDropRow != lastDropRow ||
mSlots->mDropOrient != lastDropOrient ||
mSlots->mDragAction != lastDragAction) {
// Invalidate row at the old location.
if (mSlots->mDropAllowed) {
mSlots->mDropAllowed = PR_FALSE;

View File

@ -25,6 +25,7 @@
* Brian Ryner <bryner@brianryner.com>
* Jan Varga <varga@ku.sk>
* Nate Nielsen <nielsen@memberwebs.com>
* Mark Banner <mark@standard8.demon.co.uk>
*
* Alternatively, the contents of this file may be used under the terms of
* either of the GNU General Public License Version 2 or later (the "GPL"),
@ -446,6 +447,9 @@ protected: // Data Members
// Number of lines to be scrolled.
PRInt16 mScrollLines;
// The drag action that was received for this slot
PRUint32 mDragAction;
nsCOMPtr<nsIDragSession> mDragSession;
// Timer for opening/closing spring loaded folders or scrolling the tree.