Closed
Bug 861118
Opened 12 years ago
Closed 8 years ago
"ASSERTION: unexpected aContextFrame" with "div:first-line { page-break-before: always; }"
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: jruderman, Assigned: heycam)
References
Details
(Keywords: assertion, regression, testcase)
Attachments
(3 files)
|
209 bytes,
text/html
|
Details | |
|
11.68 KB,
text/plain
|
Details | |
|
1.64 KB,
patch
|
Details | Diff | Splinter Review |
###!!! ASSERTION: unexpected aContextFrame: 'aContextFrame->StyleDisplay() == this', file nsStyleStructInlines.h, line 145
|
Reporter | |
Comment 1•12 years ago
|
||
|
||
Comment 3•12 years ago
|
||
Backing out bug 860370 + bug 848973 fixes it.
|
Assignee | |
Comment 4•12 years ago
|
||
Avoid the assertion by copying out the guts of the style struct accessors.
It's a bit unsatisfying doing this. I wonder how useful the assertions from bug 848973 actually are...
Assignee: nobody → cam
Status: NEW → ASSIGNED
Attachment #737088 -
Flags: review?(bzbarsky)
Flags: needinfo?(cam)
|
||
Comment 5•12 years ago
|
||
Can we change the asserts to assert the things we really care about (like equality of the style structs in terms of their effect on computed styles or something)?
|
|
||
Updated•12 years ago
|
Flags: needinfo?(cam)
|
|
||
Comment 6•12 years ago
|
||
Comment on attachment 737088 [details] [diff] [review]
patch
Pending an answer....
Attachment #737088 -
Flags: review?(bzbarsky)
|
|
Assignee | |
Comment 7•8 years ago
|
||
We no longer hit this assertion.
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Flags: needinfo?(cam)
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•