Closed
Bug 408737
Opened 17 years ago
Closed 17 years ago
Hang with -moz-column and tall div inside short div
Categories
(Core :: Layout, defect, P2)
Tracking
()
RESOLVED
FIXED
People
(Reporter: dholbert, Assigned: fantasai.bugs)
References
Details
(4 keywords)
Attachments
(3 files)
I'm getting some serious hangs with simple setups like:
[-moz-column div]
[short div]
[tall div]
The length of the hang seems to be related to the ratio of tallHeight to shortHeight.
Reporter | ||
Comment 1•17 years ago
|
||
This mild testcase hangs for ~30 sec on my machine
Reporter | ||
Comment 2•17 years ago
|
||
This testcase is more serious -- I'm not sure how long it hangs. (It's been going for several minutes, with no sign of stopping)
Reporter | ||
Comment 3•17 years ago
|
||
This testcase demonstrates that the hang doesn't depend on "short div" having exactly 0px height.
If we increase both short and tall heights, the hang remains. This testcase hangs for more than a few minutes. (I stopped it after a few minutes)
Comment 4•17 years ago
|
||
Regression window Testcase 1 is http://bonsai.mozilla.org/cvsquery.cgi?module=PhoenixTinderbox&date=explicit&mindate=1194893940&maxdate=1194894899
Reporter | ||
Comment 5•17 years ago
|
||
Based on that, it looks like this is from bug 399407, since that bug deals with nsColumnSetFrame.cpp (-moz-column) and none of the other checkins seem related.
Blocks: 399407
Reporter | ||
Comment 6•17 years ago
|
||
Yup -- backing out patch for bug 399407 fixes the hangs on all 3 testcases.
Assignee: nobody → fantasai.bugs
Flags: blocking1.9?
Priority: -- → P2
Updated•17 years ago
|
Flags: blocking1.9? → blocking1.9+
Updated•17 years ago
|
Updated•17 years ago
|
Version: unspecified → Trunk
Updated•17 years ago
|
Priority: P2 → P3
Flags: blocking1.9+
Priority: P3 → P2
Updated•17 years ago
|
Flags: tracking1.9+
Comment 7•17 years ago
|
||
Fantasai, making progress on this? If you aren't looking at this, maybe dholbert can take a look here?
It's fixed by the patches in bug 404215. Just need to write one that's good enough to check in. :)
Comment 9•17 years ago
|
||
Great. Thanks for the follow-up.
Comment 10•17 years ago
|
||
We wouldn't hold the release for this if it were the last bug on the list (we're at that point now). Moving to wanted1.9.0.x+.
Flags: wanted1.9.0.x+
Flags: blocking1.9-
Flags: blocking1.9+
Assignee | ||
Comment 11•17 years ago
|
||
Should be fixed with the checkin to bug 404215
Status: NEW → RESOLVED
Closed: 17 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Updated•16 years ago
|
Flags: wanted1.9.0.x+
You need to log in
before you can comment on or make changes to this bug.
Description
•