Closed Bug 670564 Opened 10 years ago Closed 10 years ago

"ASSERTION: illegal left edge" in nsDisplayText::Paint

Categories

(Core :: Layout, defect)

defect
Not set
minor

Tracking

()

RESOLVED FIXED
mozilla8

People

(Reporter: jruderman, Assigned: mats)

References

(Blocks 1 open bug)

Details

(Keywords: assertion, testcase)

Attachments

(2 files)

Attached file testcase
###!!! ASSERTION: illegal left edge: 'mLeftEdge >= 0', file layout/generic/nsTextFrameThebes.cpp
Leading up to the reported assertion (on Linux64):

WARNING: Overflowed nscoord_MAX in conversion to nscoord: 'aValue <= nscoord_MAX', file ../../dist/include/nsCoord.h, line 106
nsLineLayout: Text(0)@0x7ff67d230f10 metrics=1073741824,1140!
###!!! ASSERTION: Computed overflow area must contain frame bounds: 'aNewSize.width == 0 || aNewSize.height == 0 || aOverflowAreas.Overflow(otype).Contains(nsRect(nsPoint(0,0), aNewSize))', file layout/generic/nsFrame.cpp, line 6465
###!!! ASSERTION: Wrong bounds: 'bounds.IsEqualInterior(aChildren.GetBounds(aBuilder))', file layout/base/FrameLayerBuilder.cpp, line 1838
###!!! ASSERTION: illegal left edge: 'mLeftEdge >= 0', file layout/generic/nsTextFrameThebes.cpp, line 4179
OS: Mac OS X → All
The "illegal left edge" assertion was added in bug 312156.
The condition is quite harmless.
Assignee: nobody → matspal
Blocks: 312156
Severity: normal → minor
Hardware: x86_64 → All
Attached patch fix + crashtestSplinter Review
The distance to the marker edge is so large it causes integer overflow
on nscoord.  I think we can safely assume all of it is clipped (the
the width would have to be > nscoord_MAX to be visible).
Attachment #545146 - Flags: review?(roc)
Comment on attachment 545146 [details] [diff] [review]
fix + crashtest

Review of attachment 545146 [details] [diff] [review]:
-----------------------------------------------------------------
Attachment #545146 - Flags: review?(roc) → review+
Merged:
http://hg.mozilla.org/mozilla-central/rev/21d1ecfeabd8
http://hg.mozilla.org/mozilla-central/rev/94bf463e0040
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → mozilla8
You need to log in before you can comment on or make changes to this bug.