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

RESOLVED FIXED in mozilla8

Status

()

Core
Layout
--
minor
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Jesse Ruderman, Assigned: mats)

Tracking

(Blocks: 1 bug, {assertion, testcase})

Trunk
mozilla8
assertion, testcase
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

6 years ago
Created attachment 545093 [details]
testcase

###!!! ASSERTION: illegal left edge: 'mLeftEdge >= 0', file layout/generic/nsTextFrameThebes.cpp
(Assignee)

Comment 1

6 years ago
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
(Assignee)

Comment 2

6 years ago
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
(Assignee)

Comment 3

6 years ago
Created attachment 545146 [details] [diff] [review]
fix + crashtest

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+
(Assignee)

Comment 5

6 years ago
http://hg.mozilla.org/integration/mozilla-inbound/rev/21d1ecfeabd8
http://hg.mozilla.org/integration/mozilla-inbound/rev/94bf463e0040
Flags: in-testsuite+
Whiteboard: [inbound]
Merged:
http://hg.mozilla.org/mozilla-central/rev/21d1ecfeabd8
http://hg.mozilla.org/mozilla-central/rev/94bf463e0040
Status: NEW → RESOLVED
Last Resolved: 6 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.