Closed Bug 389603 Opened 17 years ago Closed 17 years ago

Crash loading URL, IsImpactedByFloat assertion and spinning 1000 times

Categories

(Core :: Layout, defect)

x86
All
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: roc, Unassigned)

References

()

Details

(Keywords: assertion, hang, regression)

Load the given URL and make the text size bigger a few times.

I'm working on this. So far I've discovered that the crash (for me) is caused by textrun construction building a textrun with missing characters. In particular there's some text ". Box Office Mojo. Retrieved on 2007-04-09." for which we build a textrun which is actually ". Box Office Mojo. Retrieved on -04-09." --- i.e. the "2007" (which has its own DOM element and frame) is skipped. This causes problems because gfxTextRun::MeasureText returns zero for that substring and the text frame ends up thinking none of its text fit, and gets confused because non-whitespace text at the start of a line not impacted by a float should always have some text forced to fit.
OS: Mac OS X → All
Flags: blocking1.9? → blocking1.9+
After the check-in of bug 385270, it seems to hang instead of crashing.
This works for me on Mac. Can you still reproduce? Can you get a stack trace for the hang?
Well, I can't repro the hang anymore with 2007082805.
FWIW, this works for me too with a 2 days old build on Windows.
I think this is fixed now.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.