Closed
Bug 398803
Opened 17 years ago
Closed 17 years ago
"ASSERTION: scrollbars should not have been created" with block-in-inline, overflow:inherit, and :first-line
Categories
(Core :: Layout, defect, P2)
Tracking
()
RESOLVED
FIXED
mozilla1.9beta2
People
(Reporter: jruderman, Assigned: bzbarsky)
References
Details
(Keywords: assertion, testcase)
Attachments
(3 files)
(deleted),
text/html
|
Details | |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
roc
:
review+
roc
:
superreview+
roc
:
approval1.9+
|
Details | Diff | Splinter Review |
Loading the testcase triggers:
###!!! ASSERTION: scrollbars should not have been created: 'result.mHorizontal != NS_STYLE_OVERFLOW_VISIBLE && result.mHorizontal != NS_STYLE_OVERFLOW_CLIP && result.mVertical != NS_STYLE_OVERFLOW_VISIBLE && result.mVertical != NS_STYLE_OVERFLOW_CLIP', file /Users/jruderman/trunk/mozilla/layout/generic/nsGfxScrollFrame.cpp, line 1433
Assignee | ||
Comment 1•17 years ago
|
||
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•17 years ago
|
||
Attachment #283802 -
Flags: superreview?(dbaron)
Attachment #283802 -
Flags: review?(dbaron)
Assignee | ||
Comment 3•17 years ago
|
||
This depends on the fix for bug 395623. Once that lands, I'll add this testcase to the first-line reftests too.
Assignee | ||
Comment 4•17 years ago
|
||
Comment on attachment 283802 [details] [diff] [review]
Same as diff -w
Trying roc for the review
Attachment #283802 -
Flags: superreview?(roc)
Attachment #283802 -
Flags: superreview?(dbaron)
Attachment #283802 -
Flags: review?(roc)
Attachment #283802 -
Flags: review?(dbaron)
Attachment #283802 -
Flags: superreview?(roc)
Attachment #283802 -
Flags: superreview+
Attachment #283802 -
Flags: review?(roc)
Attachment #283802 -
Flags: review+
Attachment #283802 -
Flags: approval1.9+
Comment 5•17 years ago
|
||
Just a reminder to please commit this by Monday if you want to get it in before beta. Otherwise, approval1.9+ will be revoked, and you will need to re-request it after M9 if you still want to land the patch. If you would like somebody else to commit this for you, please add the "checkin-needed" keyword.
Assignee | ||
Comment 6•17 years ago
|
||
Reed, see comment 3. This depends on a bug which was a blocker, and hence didn't have approval, and is now stuck in approval limbo. I'd love to get this in for beta, but that bug needs to go in first.
If someone else wants to commit this and deal with any resulting fallout (like it not actually fixing this bug on its own, for example, which it doesn't), go for it. I've only tested this patch and thought about it in the context of bug 395623 already having landed.
Comment 7•17 years ago
|
||
Comment on attachment 283802 [details] [diff] [review]
Same as diff -w
Resetting all approval1.9+ flags on bugs that have not been checked in by Oct 22 11:59 PM PDT. Please re-request approval if needed.
Attachment #283802 -
Flags: approval1.9+
Assignee | ||
Updated•17 years ago
|
Attachment #283802 -
Flags: approval1.9?
Attachment #283802 -
Flags: approval1.9? → approval1.9+
Assignee | ||
Comment 8•17 years ago
|
||
Checked in. layout/reftests/first-line/parent-style-1.html tests this.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Assignee | ||
Comment 9•17 years ago
|
||
Also layout/reftests/first-letter/parent-style-1.html
You need to log in
before you can comment on or make changes to this bug.
Description
•