Closed Bug 82074 Opened 23 years ago Closed 23 years ago

Resizing file bookmark dialog results in hang then crash

Categories

(SeaMonkey :: Bookmarks & History, defect, P2)

Product:
SeaMonkey
Component:
Bookmarks & History
Platform:
x86
Windows 2000
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED
Milestone:
mozilla0.9.8

People

(Reporter: cplyon, Assigned: hewitt)

References

Details

(Keywords: crash, Whiteboard: worksforme?) 

Using build 2001052104 on Win2K

Steps to Reproduce:
1. Open the File Bookmarks dialog or the Manage Bookmarks window
2. Grabbing the lower-right corner (for example), slowly resize the dialog 
smaller (ie drag the corner up and to the right).
3. Resize and repeat if necessary.

Result:
Mozilla hangs then gobbles memory until it eventually crashes.

Reproducible:
Always (sometimes it takes a few tries).
Can reproduce in both themes.

Talkback ID:
TB30746663G and TB30746413G
from talkback:

nsWritingIterator<unsigned short>::normalize_forward  
[..\..\dist\include\nsStringIterator.h, line 404]
nsWritingIterator<unsigned short>::write [..\..\dist\include\nsStringIterator.h,
line 355]
nsWritingIterator<unsigned short>::write [..\..\dist\include\nsStringIterator.h,
line 355]
copy_string [..\..\dist\include\nsAlgorithm.h, line 78]
nsAString::do_AssignFromReadable
[d:\builds\seamonkey\mozilla\string\src\nsAString.cpp, line 254]
nsAString::AssignFromReadable
[d:\builds\seamonkey\mozilla\string\src\nsAString.cpp, line 658]
nsFont::operator= [d:\builds\seamonkey\mozilla\gfx\src\nsFont.cpp, line 78]
StyleFontBlob::CopyTo
[d:\builds\seamonkey\mozilla\content\base\src\nsStyleContext.cpp, line 187]
MapDeclarationFontInto
[d:\builds\seamonkey\mozilla\content\html\style\src\nsCSSStyleRule.cpp, line 1783]
CSSStyleRuleImpl::MapFontStyleInto
[d:\builds\seamonkey\mozilla\content\html\style\src\nsCSSStyleRule.cpp, line 1750]
MapStyleRuleFont
[d:\builds\seamonkey\mozilla\content\base\src\nsStyleContext.cpp, line 3148]
nsSupportsArray::EnumerateForwards
[d:\builds\seamonkey\mozilla\xpcom\ds\nsSupportsArray.cpp, line 360]
StyleContextImpl::RemapStyle
[d:\builds\seamonkey\mozilla\content\base\src\nsStyleContext.cpp, line 3361]
NS_NewStyleContext
[d:\builds\seamonkey\mozilla\content\base\src\nsStyleContext.cpp, line 4207]
StyleSetImpl::GetContext
[d:\builds\seamonkey\mozilla\content\base\src\nsStyleSet.cpp, line 857]
StyleSetImpl::ResolveStyleFor
[d:\builds\seamonkey\mozilla\content\base\src\nsStyleSet.cpp, line 941]
nsPresContext::ResolveStyleContextFor
[d:\builds\seamonkey\mozilla\layout\base\src\nsPresContext.cpp, line 696]
FrameManager::ReResolveStyleContext
[d:\builds\seamonkey\mozilla\layout\html\base\src\nsFrameManager.cpp, line 1626]
FrameManager::ReResolveStyleContext
[d:\builds\seamonkey\mozilla\layout\html\base\src\nsFrameManager.cpp, line 1776]
FrameManager::ReResolveStyleContext
[d:\builds\seamonkey\mozilla\layout\html\base\src\nsFrameManager.cpp, line 1776]
FrameManager::ReResolveStyleContext
[d:\builds\seamonkey\mozilla\layout\html\base\src\nsFrameManager.cpp, line 1776]
FrameManager::ReResolveStyleContext
[d:\builds\seamonkey\mozilla\layout\html\base\src\nsFrameManager.cpp, line 1776]
FrameManager::ComputeStyleChangeFor
[d:\builds\seamonkey\mozilla\layout\html\base\src\nsFrameManager.cpp, line 2020]
nsCSSFrameConstructor::AttributeChanged
[d:\builds\seamonkey\mozilla\layout\html\style\src\nsCSSFrameConstructor.cpp,
line 9970]
StyleSetImpl::AttributeChanged [\content\base\src\nsStyleSet.cpp, line 1290]
PresShell::AttributeChanged [\layout\html\base\src\nsPresShell.cpp, line 4816]
nsXULDocument::AttributeChanged [\content\xul\document\src\nsXULDocument.cpp,
line 1621]
nsXULElement::UnsetAttribute [\content\xul\content\src\nsXULElement.cpp, line 3312]
nsGfxScrollFrameInner::SetScrollbarVisibility
[\layout\html\base\src\nsGfxScrollFrame.cpp, line 1488]
nsGfxScrollFrameInner::AddRemoveScrollbar
[\layout\html\base\src\nsGfxScrollFrame.cpp, line 975]
nsGfxScrollFrameInner::AddHorizontalScrollbar
[\layout\html\base\src\nsGfxScrollFrame.cpp, line 944]
nsGfxScrollFrameInner::Layout [\layout\html\base\src\nsGfxScrollFrame.cpp, line
1201]
nsGfxScrollFrame::DoLayout [\layout\html\base\src\nsGfxScrollFrame.cpp, line 1047]
nsBox::Layout [\layout\xul\base\src\nsBox.cpp, line 985]
nsSprocketLayout::Layout [\layout\xul\base\src\nsSprocketLayout.cpp, line 417]
nsContainerBox::DoLayout [\layout\xul\base\src\nsContainerBox.cpp, line 553]
nsBoxFrame::DoLayout [\layout\xul\base\src\nsBoxFrame.cpp, line 979]
nsBox::Layout [\layout\xul\base\src\nsBox.cpp, line 985]
nsStackLayout::Layout [\layout\xul\base\src\nsStackLayout.cpp, line 256]
nsContainerBox::DoLayout [\layout\xul\base\src\nsContainerBox.cpp, line 553]
nsBoxFrame::DoLayout [\layout\xul\base\src\nsBoxFrame.cpp, line 979]
nsXULTreeFrame::DoLayout [\layout\xul\base\src\nsXULTreeFrame.cpp, line 114]
nsBox::Layout [\layout\xul\base\src\nsBox.cpp, line 985]
nsSprocketLayout::Layout [\layout\xul\base\src\nsSprocketLayout.cpp, line 417]
nsContainerBox::DoLayout [\layout\xul\base\src\nsContainerBox.cpp, line 553]
nsBoxFrame::DoLayout [\layout\xul\base\src\nsBoxFrame.cpp, line 979]
nsBox::Layout [\layout\xul\base\src\nsBox.cpp, line 985]
nsStackLayout::Layout [\layout\xul\base\src\nsStackLayout.cpp, line 256]
nsContainerBox::DoLayout [\layout\xul\base\src\nsContainerBox.cpp, line 553]
nsBoxFrame::DoLayout [\layout\xul\base\src\nsBoxFrame.cpp, line 979]
nsBox::Layout [\layout\xul\base\src\nsBox.cpp, line 985]
nsBoxFrame::Reflow [\layout\xul\base\src\nsBoxFrame.cpp, line 781]
nsRootBoxFrame::Reflow [\layout\xul\base\src\nsRootBoxFrame.cpp, line 209]
nsContainerFrame::ReflowChild [\layout\html\base\src\nsContainerFrame.cpp, line 745]
ViewportFrame::Reflow [\layout\html\base\src\nsViewportFrame.cpp, line 538]
nsHTMLReflowCommand::Dispatch [\layout\html\base\src\nsHTMLReflowCommand.cpp,
line 145]
PresShell::ProcessReflowCommand [\layout\html\base\src\nsPresShell.cpp, line 5712]
PresShell::ProcessReflowCommands [\layout\html\base\src\nsPresShell.cpp, line 5767]
PresShell::FlushPendingNotifications [\layout\html\base\src\nsPresShell.cpp,
line 4743]
PresShell::HandlePostedReflowCallbacks [\layout\html\base\src\nsPresShell.cpp,
line 4659]
PresShell::ProcessReflowCommands [\layout\html\base\src\nsPresShell.cpp, line 5841]
PresShell::FlushPendingNotifications [\layout\html\base\src\nsPresShell.cpp,
line 4743]
PresShell::HandlePostedReflowCallbacks [\layout\html\base\src\nsPresShell.cpp,
line 4659]
PresShell::ProcessReflowCommands [\layout\html\base\src\nsPresShell.cpp, line 5841]

and

ntdll.dll + 0x4a07a (0x77fca07a)
MSVCRT.DLL + 0x1089 (0x78001089)
MSVCRT.DLL + 0x1026 (0x78001026)
nsMemory::Alloc [\xpcom\base\nsMemoryImpl.cpp, line 542]
nsStr::Alloc [\string\obsolete\nsStr.cpp, line 683]
nsStr::Realloc [\string\obsolete\nsStr.cpp, line 709]
nsStr::EnsureCapacity [\string\obsolete\nsStr.cpp, line 103]
nsStr::GrowCapacity [\string\obsolete\nsStr.cpp, line 133]
nsString::SetCapacity [\string\obsolete\nsString2.cpp, line 215]
nsString::SetLength [\string\obsolete\nsString2.cpp, line 198]
nsAString::do_AssignFromReadable [\string\src\nsAString.cpp, line 253]
nsAString::AssignFromReadable [\string\src\nsAString.cpp, line 658]
StyleListBlob::SetFrom [\content\base\src\nsStyleContext.cpp, line 902]
StyleContextImpl::RemapStyle [\content\base\src\nsStyleContext.cpp, line 3515]
StyleContextImpl::RemapStyle [\content\base\src\nsStyleContext.cpp, line 3515]
NS_NewStyleContext [\content\base\src\nsStyleContext.cpp, line 4207]
StyleSetImpl::GetContext [\content\base\src\nsStyleSet.cpp, line 857]
StyleSetImpl::ResolveStyleFor [\content\base\src\nsStyleSet.cpp, line 941]
nsPresContext::ResolveStyleContextFor [\layout\base\src\nsPresContext.cpp, line 696]
FrameManager::ReResolveStyleContext [\layout\html\base\src\nsFrameManager.cpp,
line 1626]
FrameManager::ReResolveStyleContext [\layout\html\base\src\nsFrameManager.cpp,
line 1776]
FrameManager::ReResolveStyleContext [\layout\html\base\src\nsFrameManager.cpp,
line 1776]
FrameManager::ReResolveStyleContext [\layout\html\base\src\nsFrameManager.cpp,
line 1776]
FrameManager::ReResolveStyleContext [\layout\html\base\src\nsFrameManager.cpp,
line 1776]
FrameManager::ComputeStyleChangeFor [\layout\html\base\src\nsFrameManager.cpp,
line 2020]
nsCSSFrameConstructor::AttributeChanged
[\layout\html\style\src\nsCSSFrameConstructor.cpp, line 9970]
StyleSetImpl::AttributeChanged [\content\base\src\nsStyleSet.cpp, line 1290]
PresShell::AttributeChanged [\layout\html\base\src\nsPresShell.cpp, line 4816]
nsXULDocument::AttributeChanged [\content\xul\document\src\nsXULDocument.cpp,
line 1621]
nsXULElement::SetAttribute [\content\xul\content\src\nsXULElement.cpp, line 3057]
nsXULElement::SetAttribute [\content\xul\content\src\nsXULElement.cpp, line 3077]
nsGfxScrollFrameInner::SetScrollbarVisibility
[\layout\html\base\src\nsGfxScrollFrame.cpp, line 1485]
nsGfxScrollFrameInner::AddRemoveScrollbar
[\layout\html\base\src\nsGfxScrollFrame.cpp, line 981]
nsGfxScrollFrameInner::RemoveHorizontalScrollbar
[\layout\html\base\src\nsGfxScrollFrame.cpp, line 956]
nsGfxScrollFrameInner::Layout [\layout\html\base\src\nsGfxScrollFrame.cpp, line
1227]
nsGfxScrollFrame::DoLayout [\layout\html\base\src\nsGfxScrollFrame.cpp, line 1047]
nsBox::Layout [\layout\xul\base\src\nsBox.cpp, line 985]
nsSprocketLayout::Layout [\layout\xul\base\src\nsSprocketLayout.cpp, line 417]
nsContainerBox::DoLayout [\layout\xul\base\src\nsContainerBox.cpp, line 553]
nsBoxFrame::DoLayout [\layout\xul\base\src\nsBoxFrame.cpp, line 979]
nsBox::Layout [\layout\xul\base\src\nsBox.cpp, line 985]
nsStackLayout::Layout [\layout\xul\base\src\nsStackLayout.cpp, line 256]
nsContainerBox::DoLayout [\layout\xul\base\src\nsContainerBox.cpp, line 553]
nsBoxFrame::DoLayout [\layout\xul\base\src\nsBoxFrame.cpp, line 979]
nsXULTreeFrame::DoLayout [\layout\xul\base\src\nsXULTreeFrame.cpp, line 114]
nsBox::Layout [\layout\xul\base\src\nsBox.cpp, line 985]
nsSprocketLayout::Layout [\layout\xul\base\src\nsSprocketLayout.cpp, line 417]
nsContainerBox::DoLayout [\layout\xul\base\src\nsContainerBox.cpp, line 553]
nsBoxFrame::DoLayout [\layout\xul\base\src\nsBoxFrame.cpp, line 979]
nsBox::Layout [\layout\xul\base\src\nsBox.cpp, line 985]
nsSprocketLayout::Layout [\layout\xul\base\src\nsSprocketLayout.cpp, line 417]
nsContainerBox::DoLayout [\layout\xul\base\src\nsContainerBox.cpp, line 553]
nsBoxFrame::DoLayout [\layout\xul\base\src\nsBoxFrame.cpp, line 979]
nsBox::Layout [\layout\xul\base\src\nsBox.cpp, line 985]
nsSprocketLayout::Layout [\layout\xul\base\src\nsSprocketLayout.cpp, line 417]
nsContainerBox::DoLayout [\layout\xul\base\src\nsContainerBox.cpp, line 553]
nsBoxFrame::DoLayout [\layout\xul\base\src\nsBoxFrame.cpp, line 979]
nsBox::Layout [\layout\xul\base\src\nsBox.cpp, line 985]
nsSprocketLayout::Layout [\layout\xul\base\src\nsSprocketLayout.cpp, line 417]
nsContainerBox::DoLayout [\layout\xul\base\src\nsContainerBox.cpp, line 553]
nsBoxFrame::DoLayout [\layout\xul\base\src\nsBoxFrame.cpp, line 979]
nsBox::Layout [\layout\xul\base\src\nsBox.cpp, line 985]
nsStackLayout::Layout [\layout\xul\base\src\nsStackLayout.cpp, line 256]
nsContainerBox::DoLayout [\layout\xul\base\src\nsContainerBox.cpp, line 553]
Assignee: asa → ben
Status: UNCONFIRMED → NEW
Component: Browser-General → Bookmarks
Ever confirmed: true
QA Contact: doronr → claudius
Severity: normal → critical
Keywords: crash
ben - could you move this right to the top of your list as it is a crash.
Keywords: nsbeta1+, nsBranch
Priority: -- → P1
Target Milestone: --- → mozilla0.9.3
The hang is actually in layout, further down the stack it looks like there's
lots of activity:

PresShell::ProcessReflowCommands(int 0) line 5895
PresShell::FlushPendingNotifications(PresShell * const 0x044635f0) line 4797
PresShell::HandlePostedReflowCallbacks() line 4713
PresShell::ProcessReflowCommands(int 0) line 5895
PresShell::FlushPendingNotifications(PresShell * const 0x044635f0) line 4797
PresShell::HandlePostedReflowCallbacks() line 4713
PresShell::ProcessReflowCommands(int 0) line 5895
PresShell::FlushPendingNotifications(PresShell * const 0x044635f0) line 4797
PresShell::HandlePostedReflowCallbacks() line 4713
PresShell::ProcessReflowCommands(int 0) line 5895
PresShell::FlushPendingNotifications(PresShell * const 0x044635f0) line 4797
PresShell::HandlePostedReflowCallbacks() line 4713
PresShell::ProcessReflowCommands(int 0) line 5895
PresShell::FlushPendingNotifications(PresShell * const 0x044635f0) line 4797
PresShell::HandlePostedReflowCallbacks() line 4713
PresShell::ProcessReflowCommands(int 0) line 5895
PresShell::FlushPendingNotifications(PresShell * const 0x044635f0) line 4797
Assignee: ben → hyatt
Target Milestone: mozilla0.9.3 → mozilla0.9.4
Status: NEW → ASSIGNED
Target Milestone: mozilla0.9.4 → Future
Blocks: 99227
Peter - Looks like this one hasn't been paid attention to in months. Should we
nsbranch- it? Its is a crasher?!?!?!?

jpatel - What's this look like on the crash reports?
We need to find out if this is still occurring with recent builds and get a new
stack trace and/or Talkback incident ids for this crash.  Claudius, see if you
can reproduce this and get me some Talkback info.  

As far as I know, I haven't seen this crash show up in recent topcrash reports,
but it wouldn't hurt to get an update on this one.
I am unable to reproduce this bug on Win and Mac bracnh builds from 20010910.
Unless someone differs this'll get verified WFM.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → WORKSFORME
No longer blocks: 99227
Reproduced it on a clean install (brand new profile) on build 2001092003 on 
Win2k.  It seems to be easier to reproduce if you expand the folders and resize 
slowly.

Hang, then Mozilla died without bringing up Talkback.
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
seems to be a pretty rare crash. I think Dave should look at it after he he
finishes with known showstoppers, and plus it iff there is a very safe fix. 
Otherwise, I think we can live with it a bit longer. 0.9.5/p2, cc evaughan for
possible box involvement.
Priority: P1 → P2
Target Milestone: Future → mozilla0.9.5
Status: REOPENED → ASSIGNED
Target Milestone: mozilla0.9.5 → mozilla0.9.7
If this is a rare crash, can someone mark it as nsbranch-?  That way we'll
remove it for consideration for eMojo
nsbranch-
Keywords: nsbranch → nsbranch-
Blocks: 107067
I can't reproduce this on a clean install (brand new profile) on build 
2001102408 on Win2k. cplyon? worksforme?
Whiteboard: worksforme?
Keywords: nsbranch-
Yes, I'm still seeing the crash.  Make sure you have several nested folders 
(like the default) all expanded, and resize slowly from the bottom right up to 
the top left.

Talkback ID TB37316943H
I can reproduce this in the File Bookmarks dialog, using the NS 102603 build,
The key seems to be resizing very slowly.  What may be significant is the larger
number of truncations of bookmark titles that occur then, relative to resizing
fast or even at what I consider normal speed.
...er, folder names, not bookmark titles, obviously.
--> hewitt.  This bug will just go away once bookmarks use outliner, so really
this is a non-bug.
Assignee: hyatt → hewitt
Status: ASSIGNED → NEW
Target Milestone: mozilla0.9.7 → mozilla0.9.8
Status: NEW → ASSIGNED
blake just landed outliner bookmarks so this should be fixed
Status: ASSIGNED → RESOLVED
Closed: 23 years ago → 23 years ago
Resolution: --- → FIXED
v Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.0.0) Gecko/20020530
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.