15479 - [Tree] Crash when mousing over scrollbar in folder pane

Status: VERIFIED WORKSFORME

(SeaMonkey :: MailNews: Message Display, defect, P1)

Description

[scottputterman]

Open up an account.  Make sure there is a scrollbar.  Mouse over it.  Crash.
I have a fix and it's due to not constraining the folder tree correctly.

Comment 1

[lchiang]

Perhaps the same as http://bugzilla.mozilla.org/show_bug.cgi?id=15474?

Comment 2

[scottputterman]

It had a different stack on my machine which is why i created this one.  I'm
hoping they are the same though so that the other one magically goes away.

Comment 3

[scottputterman]

checked in fix for this crash.

Comment 4

[Karen Huang]

Even this bug is original opened from WinNT.
But it failed by retesting on today's 10-07-09-M11 build/Win98 platform.

If I scroll down the scroll bar at first time that I get in to the mail tool.
It's working fine...

But after I do the following procedures. it crashed again.
1) Compose new message.(Do not type any receivers from address pane area yet)
2) Just save this new msg as draft.
3) Scrolled down to the local draft folder and tried to open the msg
4) Actual result: Crashed

Expected Result: Shouldn't crash at this point.

Comment 5

[scottputterman]

Your description makes this seem like a different bug.  Are you crashing while
scrolling? Or are you crashing while selecting the draft message?  Do you have a
talkback stack by any chance?  I'm unable to reproduce this following your
steps.

Comment 6

[lchiang]

clearing resolution.  Karen - can you work with Scott on this?

Comment 7

[lchiang]

Oh, karen is not on the cc: - adding her.

Comment 8

[Karen Huang]

Yes, it crashing while scrolling.
I did have a talkback but I didn't submitted and got the talkback number.

It's not such straight forward if just based on that procedure that I gave you.
You need to play around between the messages, folders and scroll bar.

It seemed sometime happened, sometime not.....

Comment 9

[lchiang]

Karen - the original bug as reported occurred all the time and was easy to
reproduce.  Perhaps, you may want to open a new bug on the steps you found since
your steps seem to pertain to the drafts folder.

Comment 10

[Karen Huang]

Scott, I am just trying to reopen the existing bug instead of open new bug.
But if you say so.... I agree with you.... I will log another new bug if it
happen again.

Comment 11

[scottputterman]

I don't care if we make this bug represent the new bug if it's really a random
crash.  If you could get it to crash and get a talkback report, I'd appreciate
that.  And if you could make this happen without drafts and message compose,
that would be even better.

Comment 12

[Karen Huang]

O.K. I will try and let you know once I get the crash again...

Comment 13

[scottputterman]

I just realized that my inability to crash may not mean anything.  I've been
working on other scrolling bugs unrelated to the original crash I fixed.
There's a good possibility that my changes, which seem to be working, may have
fixed other scrolling bugs including the crash you are seeing which is why I
may not be able to reproduce it.  It would still be useful to narrow this down
to a simple case, in case I'm wrong. But perhaps when I eventually check my new
code in, the crash will go away for you.

Comment 14

[Karen Huang]

Used the 10-08-12-M11 build for Mac, Win98 & WinNT platforms:

I got crashs for Mac, Win98 & WinNT today.
I already sent the talkback and got the following Incident#:
You can search by using my e-mail:huang@netscape or IncidentID
They are:
14181414 & 14181514 -> for Mac: Scroll down after expand the news group cause
crash

14181017,14181560 -> for Win98:
1)Scrolling the scroll bar after subscribe the newsgoup cause crash
2)Crash after expand the folders & scroll up/down the scroll bar

14181592 -> for WinNT: Crash after expand the folders & scroll up/down the
scroll bar

I copy/paste the WinNT: Talkback Incident ID: 14181592 as following:

Call Stack:    (Signature = MSVCRT.dll + 0x1799 (0x78001799) 30f5db49)
MSVCRT.dll + 0x1799 (0x78001799)
MSVCRT.dll + 0x1465 (0x78001465)
   nsTitledButtonFrame::Destroy
[d:\builds\seamonkey\mozilla\layout\xul\base\src\nsTitledButtonFrame.cpp, line
243]
   nsFrameList::DestroyFrames
[d:\builds\seamonkey\mozilla\layout\base\src\nsFrameList.cpp, line 29]
   nsContainerFrame::Destroy
[d:\builds\seamonkey\mozilla\layout\html\base\src\nsContainerFrame.cpp, line 88]
   nsLineBox::DeleteLineList
[d:\builds\seamonkey\mozilla\layout\html\base\src\nsLineBox.cpp, line 166]
   nsBlockFrame::Destroy
[d:\builds\seamonkey\mozilla\layout\html\base\src\nsBlockFrame.cpp, line 980]
   nsFrameList::DestroyFrames
[d:\builds\seamonkey\mozilla\layout\base\src\nsFrameList.cpp, line 29]
   nsContainerFrame::Destroy
[d:\builds\seamonkey\mozilla\layout\html\base\src\nsContainerFrame.cpp, line 88]
   nsTreeCellFrame::Destroy
[d:\builds\seamonkey\mozilla\layout\xul\base\src\nsTreeCellFrame.cpp, line 477]
   nsFrameList::DestroyFrames
[d:\builds\seamonkey\mozilla\layout\base\src\nsFrameList.cpp, line 29]
   nsContainerFrame::Destroy
[d:\builds\seamonkey\mozilla\layout\html\base\src\nsContainerFrame.cpp, line 88]
   nsFrameList::DestroyFrames
[d:\builds\seamonkey\mozilla\layout\base\src\nsFrameList.cpp, line 29]
   nsContainerFrame::Destroy
[d:\builds\seamonkey\mozilla\layout\html\base\src\nsContainerFrame.cpp, line 88]
   nsTreeRowGroupFrame::Destroy
[d:\builds\seamonkey\mozilla\layout\xul\base\src\nsTreeRowGroupFrame.cpp, line
89]
   nsFrameList::DestroyFrame
[d:\builds\seamonkey\mozilla\layout\base\src\nsFrameList.cpp, line 115]
   nsTreeRowGroupFrame::OnContentRemoved
[d:\builds\seamonkey\mozilla\layout\xul\base\src\nsTreeRowGroupFrame.cpp, line
1168]
   nsCSSFrameConstructor::ContentRemoved
[d:\builds\seamonkey\mozilla\layout\html\style\src\nsCSSFrameConstructor.cpp,
line 6213]
   StyleSetImpl::ContentRemoved
[d:\builds\seamonkey\mozilla\layout\base\src\nsStyleSet.cpp, line 960]
   PresShell::ContentRemoved
[d:\builds\seamonkey\mozilla\layout\html\base\src\nsPresShell.cpp, line 1801]
   XULDocumentImpl::ContentRemoved
[d:\builds\seamonkey\mozilla\rdf\content\src\nsXULDocument.cpp, line 1882]
   RDFElementImpl::RemoveChildAt
[d:\builds\seamonkey\mozilla\rdf\content\src\nsRDFElement.cpp, line 2175]
   RDFGenericBuilderImpl::RemoveGeneratedContent
[d:\builds\seamonkey\mozilla\rdf\content\src\nsRDFGenericBuilder.cpp, line 2695]
   RDFGenericBuilderImpl::CloseContainer
[d:\builds\seamonkey\mozilla\rdf\content\src\nsRDFGenericBuilder.cpp, line 834]
   XULDocumentImpl::CloseWidgetItem
[d:\builds\seamonkey\mozilla\rdf\content\src\nsXULDocument.cpp, line 4184]
   XULDocumentImpl::AttributeChanged
[d:\builds\seamonkey\mozilla\rdf\content\src\nsXULDocument.cpp, line 1755]
   RDFElementImpl::UnsetAttribute
[d:\builds\seamonkey\mozilla\rdf\content\src\nsRDFElement.cpp, line 2690]
   RDFElementImpl::RemoveAttribute
[d:\builds\seamonkey\mozilla\rdf\content\src\nsRDFElement.cpp, line 1311]
   nsTreeTwistyListener::MouseDown
[d:\builds\seamonkey\mozilla\layout\xul\base\src\nsTreeTwistyListener.cpp, line
131]
   nsEventListenerManager::HandleEvent
[d:\builds\seamonkey\mozilla\layout\events\src\nsEventListenerManager.cpp, line
574]
   RDFElementImpl::HandleDOMEvent
[d:\builds\seamonkey\mozilla\rdf\content\src\nsRDFElement.cpp, line 2906]
   RDFElementImpl::HandleDOMEvent
[d:\builds\seamonkey\mozilla\rdf\content\src\nsRDFElement.cpp, line 2893]
   RDFElementImpl::HandleDOMEvent
[d:\builds\seamonkey\mozilla\rdf\content\src\nsRDFElement.cpp, line 2893]
   RDFElementImpl::HandleDOMEvent
[d:\builds\seamonkey\mozilla\rdf\content\src\nsRDFElement.cpp, line 2893]
   RDFElementImpl::HandleDOMEvent
[d:\builds\seamonkey\mozilla\rdf\content\src\nsRDFElement.cpp, line 2893]
   RDFElementImpl::HandleDOMEvent
[d:\builds\seamonkey\mozilla\rdf\content\src\nsRDFElement.cpp, line 2893]
   RDFElementImpl::HandleDOMEvent
[d:\builds\seamonkey\mozilla\rdf\content\src\nsRDFElement.cpp, line 2893]
   PresShell::HandleEvent
[d:\builds\seamonkey\mozilla\layout\html\base\src\nsPresShell.cpp, line 2102]
   nsView::HandleEvent
[d:\builds\seamonkey\mozilla\view\src\nsView.cpp, line 834]
   nsViewManager::DispatchEvent
[d:\builds\seamonkey\mozilla\view\src\nsViewManager.cpp, line 1673]
   HandleEvent
[d:\builds\seamonkey\mozilla\view\src\nsView.cpp, line 63]
   nsWindow::DispatchEvent
[d:\builds\seamonkey\mozilla\widget\src\windows\nsWindow.cpp, line 346]
   nsWindow::DispatchWindowEvent
[d:\builds\seamonkey\mozilla\widget\src\windows\nsWindow.cpp, line 363]
   nsWindow::DispatchMouseEvent
[d:\builds\seamonkey\mozilla\widget\src\windows\nsWindow.cpp, line 3315]
   ChildWindow::DispatchMouseEvent
[d:\builds\seamonkey\mozilla\widget\src\windows\nsWindow.cpp, line 3531
   nsWindow::ProcessMessage
[d:\builds\seamonkey\mozilla\widget\src\windows\nsWindow.cpp, line 2530]
   nsWindow::WindowProc
[d:\builds\seamonkey\mozilla\widget\src\windows\nsWindow.cpp, line 521]
   USER32.dll + 0x1820 (0x77e71820)
   0x01040008

Comment 15

[scottputterman]

Karen, thanks for the talkback report.  This looks similar to a bug I'm working
on that hasn't been filed yet, so this can serve as that bug.  I'm getting
random crashes when opening and closing account.  Sometimes I can do it by
opening and scrolling around and sometimes with just opening and closing.

Comment 16

[Karen Huang]

See bug#15474, still crashed on MAC for 10-22-15-M11 commercial build.

Comment 17

[Karen Huang]

Crash on Linux6.0, too.(10-22-08-M11 build)
Can I change the Platform/OS to "All"?

Comment 18

[Alec Flett]

taking

Comment 19

[Alec Flett]

I can't reproduce this on linux with today's build.
Can you try again? Otherwise, I'll mark worksforme

Comment 20

[Alec Flett]

mass-moving bugs I can't fix for M11 to M12

Comment 21

[Alec Flett]

I'm really not seeing this any more, and the tree has gained alot of
stability... the only known crash at this point is #16929... it's possible that
that's the crash you were seeing.

...marking WORKSFORME

Comment 22

[fenella]

Linux Redhat 6.0 (1999-11-22-08 M12)
Win_nt 4.0 (1999-11-22-09 M12)
Mac (1999-11-22-08 M12)
I try different scenario as described here. No crash mousing over scrollbar any
more. Great job.