"ASSERTION: First-in-flow first-letter should have first-letter style enabled in nsLineLayout!"

RESOLVED FIXED

Status

()

Core
Layout
RESOLVED FIXED
10 years ago
10 years ago

People

(Reporter: Jesse Ruderman, Assigned: smontagu)

Tracking

({assertion, regression, testcase})

Trunk
x86
Mac OS X
assertion, regression, testcase
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

10 years ago
Created attachment 331810 [details]
reduced testcase

layout/generic/crashtests/393923-1.html triggers:

###!!! ASSERTION: First-in-flow first-letter should have first-letter style enabled in nsLineLayout!: 'll->GetFirstLetterStyleOK() || GetPrevInFlow()', file /Users/jruderman/central/mozilla/layout/generic/nsFirstLetterFrame.cpp, line 240

I am attaching a slightly simpler testcase, but it's not very different from the existing crashtest.

This assertion was added recently, in bug 419285.
(Assignee)

Comment 1

10 years ago
Created attachment 340117 [details] [diff] [review]
GetPrevContinuation instead of GetPrevInFlow

Bidi continuations don't have prev-in-flows...
Assignee: nobody → smontagu
Attachment #340117 - Flags: superreview?(roc)
Attachment #340117 - Flags: review?(roc)
Attachment #340117 - Flags: superreview?(roc)
Attachment #340117 - Flags: superreview+
Attachment #340117 - Flags: review?(roc)
Attachment #340117 - Flags: review+
(Assignee)

Comment 2

10 years ago
http://hg.mozilla.org/mozilla-central/rev/7b75a1f34434
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.