Hang with display: -moz-deck, direction: rtl, fieldset

RESOLVED WORKSFORME

Status

()

--
critical
RESOLVED WORKSFORME
12 years ago
9 years ago

People

(Reporter: jruderman, Unassigned)

Tracking

(Blocks: 1 bug, {hang, rtl, testcase})

Trunk
x86
Mac OS X
hang, rtl, testcase
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

12 years ago
Created attachment 262466 [details]
testcase (hangs Firefox when loaded)

This testcase makes Firefox (Mac trunk debug) hang, spewing the following assertion the whole time:

###!!! ASSERTION: Bogus availSize.width; should be bigger: '!mContentFrame || GetContentMinWidth(aReflowState.rendContext) <= availSize.width', file /Users/jruderman/trunk/mozilla/layout/forms/nsFieldSetFrame.cpp, line 474
I have no idea what the deal is with the width, offhand, but the hang is due to height growth.  nsStackLayout keeps looping while the kid is growing, and each time it sets a computed height on the fieldset, the fieldset reflows at that height, overflows (because the content frame is height: 100% and the legend takes some space), nsFrame::BoxReflow does:

  aDesiredSize.height = aDesiredSize.mOverflowArea.YMost();

which increases aDesiredSize.height from what it used to be (which was equal to the computed height).

So we do this over and over again forever.
(Reporter)

Comment 2

11 years ago
Still hangs on trunk.
Is there a reason to not close it?
I haven't tested it on mac and I know that Jesse normally reacts quite quickly to comments (and maybe someone is interested in what fixed it).
Works for me on Mac trunk debug.
(Reporter)

Comment 7

11 years ago
Works for me, too.  No crash, no assertion failures.
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → WORKSFORME
Flags: in-testsuite?

Comment 8

11 years ago
Mass-assigning the new rtl keyword to RTL-related (see bug 349193).
Keywords: rtl
(Reporter)

Comment 9

9 years ago
Crashtest: http://hg.mozilla.org/mozilla-central/rev/f4622261cce8
Flags: in-testsuite? → in-testsuite+
You need to log in before you can comment on or make changes to this bug.