[FIXr]"ASSERTION: next-in-flow has different content" with XBL, block-in-inline

RESOLVED FIXED in mozilla1.9alpha8

Status

()

Core
Layout
RESOLVED FIXED
12 years ago
11 years ago

People

(Reporter: Jesse Ruderman, Assigned: bz)

Tracking

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

Trunk
mozilla1.9alpha8
assertion, testcase
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

12 years ago
This testcase triggers

###!!! ASSERTION: next-in-flow has different content: '!kidFrame || parentFrame->GetContent() == aParentContent', file /Users/admin/trunk/mozilla/layout/base/nsCSSFrameConstructor.cpp, line 11352
(Reporter)

Comment 1

12 years ago
Created attachment 235345 [details]
testcase
(Reporter)

Comment 2

11 years ago
Still happens on trunk.
Created attachment 274361 [details] [diff] [review]
Remove the bogus assert

The assert is just bogus: there's no reason aParentContent (which is the content tree parent of aContent) should be the content of the parent of aContent's frame.  Especially so if XBL insertion points are involved, as here.
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
Attachment #274361 - Flags: superreview?(roc)
Attachment #274361 - Flags: review?(roc)
OS: Mac OS X → All
Hardware: Macintosh → All
Summary: "ASSERTION: next-in-flow has different content" with XBL, block-in-inline → [FIX]"ASSERTION: next-in-flow has different content" with XBL, block-in-inline
Attachment #274361 - Flags: superreview?(roc)
Attachment #274361 - Flags: superreview+
Attachment #274361 - Flags: review?(roc)
Attachment #274361 - Flags: review+
Summary: [FIX]"ASSERTION: next-in-flow has different content" with XBL, block-in-inline → [FIXr]"ASSERTION: next-in-flow has different content" with XBL, block-in-inline
Comment on attachment 274361 [details] [diff] [review]
Remove the bogus assert

Removes a bogus assert for yet more testing peace.
Attachment #274361 - Flags: approval1.9?
Comment on attachment 274361 [details] [diff] [review]
Remove the bogus assert

a19=dbaron
Attachment #274361 - Flags: approval1.9? → approval1.9+
Fixed.
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9 M8
(Reporter)

Comment 7

11 years ago
Crashtest checked in.
Flags: in-testsuite? → in-testsuite+
You need to log in before you can comment on or make changes to this bug.