167841 - Dragging message doesn't cause auto scroll in folder pane

Status: VERIFIED FIXED

(SeaMonkey :: MailNews: Message Display, defect)

Description

[scottputterman]

I'm using the 9/10 trunk build on Win XP.

I drag a message from the thread pane and hover over the bottom of the folder
pane.  Nothing happens.

Instead, the folder pane should start scrolling down.

Sometimes after I do this, the thread pane gets into a strange state where it
start autoscrolling for no reason.

Comment 1

[Cristiano Verondini]

Also on Win 2K, same build.

Comment 2

[laurel]

*** Bug 169100 has been marked as a duplicate of this bug. ***

Comment 3

[moshe weitzman]

apparently happenning for bookmarks also so this may be a problem outside of
Mail. the bookmarks misbehavior report comes from a post today in mail newsgroup.

Comment 4

[Jan Varga [:janv]]

I guess it has something to do with last nsITimer changes.
Especially something with idle timers on windows.

Comment 5

[Jan Varga [:janv]]

dougt, I think you meant:
ti->SetIdle(PR_FALSE) not PR_TRUE in nsTreeBodyFrame.cpp
since idle timers don't fire during a dnd session on windows

introduced in bug 157136

Comment 6

[Doug Turner (:dougt)]

ugh.  your right.  all the setIdle calls should be to PR_FALSE.  I will get a
patch together.

Comment 7

[Doug Turner (:dougt)]

Attachment: patch v.1 Fixes non idle timers.

Comment 8

[Doug Turner (:dougt)]

sorry for the trouble.  taking.

Comment 9

[Darin Fisher]

Comment on attachment 100027 [details] [diff] [review]
patch v.1 Fixes non idle timers.

sr=darin

Comment 10

[rpotts (gone)]

Comment on attachment 100027 [details] [diff] [review]
patch v.1 Fixes non idle timers.

sr=[e-mail]

Comment 11

[rpotts (gone)]

Comment on attachment 100027 [details] [diff] [review]
patch v.1 Fixes non idle timers.

sr=[e-mail]

Comment 12

[Doug Turner (:dougt)]

Checking in layout/html/base/src/nsPresShell.cpp;
/cvsroot/mozilla/layout/html/base/src/nsPresShell.cpp,v  <--  nsPresShell.cpp
new revision: 3.573; previous revision: 3.572
done
Checking in layout/xul/base/src/nsMenuFrame.cpp;
/cvsroot/mozilla/layout/xul/base/src/nsMenuFrame.cpp,v  <--  nsMenuFrame.cpp
new revision: 1.233; previous revision: 1.232
done
Checking in layout/xul/base/src/nsMenuPopupFrame.cpp;
/cvsroot/mozilla/layout/xul/base/src/nsMenuPopupFrame.cpp,v  <-- 
nsMenuPopupFrame.cpp
new revision: 1.180; previous revision: 1.179
done
Checking in layout/xul/base/src/tree/src/nsTreeBodyFrame.cpp;
/cvsroot/mozilla/layout/xul/base/src/tree/src/nsTreeBodyFrame.cpp,v  <-- 
nsTreeBodyFrame.cpp
new revision: 1.139; previous revision: 1.138
done

Jan, can you verify that this does fix the problem?

Comment 13

[Jan Varga [:janv]]

yes, it works ok now

Comment 14

[[e-mail] (gone - use [e-mail] instead)]

*** Bug 168794 has been marked as a duplicate of this bug. ***

Comment 15

[[e-mail] (gone - use [e-mail] instead)]

*** Bug 170331 has been marked as a duplicate of this bug. ***

Comment 16

[Elmar Ludwig]

*** Bug 171159 has been marked as a duplicate of this bug. ***

Comment 17

[[e-mail] (gone - use [e-mail] instead)]

Verified FIXED with:

Mac OS X 10.1.5 Mac OS 9.2.2 - 2002-09-27-08
Windows 2000 - 2002-09-27-08
RedHat Linux 8.0 - 2002-09-27-08