"ASSERTION: NS_BLOCK_HAS_FIRST_LETTER_STYLE state out of sync" with select::first-letter

RESOLVED FIXED in mozilla36

Status

()

defect
RESOLVED FIXED
12 years ago
5 years ago

People

(Reporter: martijn.martijn, Assigned: mats)

Tracking

({assertion, testcase})

Trunk
mozilla36
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments)

(Reporter)

Description

12 years ago
Posted file testcase
See testcase, I get this assertion in current debug build with it:
###!!! ASSERTION: NS_BLOCK_HAS_FIRST_LETTER_STYLE state out of sync: 'haveFirstL
etterStyle == ((mState & NS_BLOCK_HAS_FIRST_LETTER_STYLE) != 0)', file c:/mozill
a/mozilla/layout/generic/nsBlockFrame.cpp, line 6282

Updated

12 years ago
Summary: ASSERTION: NS_BLOCK_HAS_FIRST_LETTER_STYLE state out of sync with select::first-letter → "ASSERTION: NS_BLOCK_HAS_FIRST_LETTER_STYLE state out of sync" with select::first-letter

Comment 1

10 years ago
Still happens on trunk (Mac).
OS: Windows XP → All
Hardware: x86 → All
Also happens with button::first-letter; layout/reftests/bugs/496032-1 (or reference)
(Assignee)

Comment 3

5 years ago
Posted patch part 1Splinter Review
The assertion is a false positive.  We need to check for a -moz-button-content
pseudo in the if-statement to avoid it, since buttons are allowed to have
::first-letter.

https://tbpl.mozilla.org/?tree=Try&rev=2b0434b60791
https://tbpl.mozilla.org/?tree=Try&rev=7ee9c4244d03
Assignee: nobody → mats
Attachment #8512693 - Flags: review?(roc)
(Assignee)

Comment 4

5 years ago
Posted patch part 2Splinter Review
This part is to avoid asserting for <select> which we don't actually
support ::first-letter on.

https://tbpl.mozilla.org/?tree=Try&rev=b257de66ee87
Attachment #8525692 - Flags: review?(roc)
(Assignee)

Updated

5 years ago
Attachment #8512693 - Attachment description: fix → part 1
https://hg.mozilla.org/mozilla-central/rev/98b193cac44f
https://hg.mozilla.org/mozilla-central/rev/3c1d71ef1284
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
You need to log in before you can comment on or make changes to this bug.