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)

x86
Linux
defect

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.
Attached file testcase 1: short hang. (0px / 100px) (deleted) —
This mild testcase hangs for ~30 sec on my machine
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)
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)
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
Yup -- backing out patch for bug 399407 fixes the hangs on all 3 testcases.
Assignee: nobody → fantasai.bugs
Flags: blocking1.9?
Priority: -- → P2
Flags: blocking1.9? → blocking1.9+
Version: unspecified → Trunk
Depends on: 404215
Priority: P2 → P3
Flags: blocking1.9+
Priority: P3 → P2
Flags: tracking1.9+
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. :)
Great. Thanks for the follow-up.
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+
Should be fixed with the checkin to bug 404215
Status: NEW → RESOLVED
Closed: 17 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Flags: wanted1.9.0.x+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: