"ASSERTION: If display lists are enabled, only painting of non-display SVG should take this code path"

RESOLVED FIXED in mozilla20

Status

()

Core
SVG
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Jesse Ruderman, Assigned: jwatt)

Tracking

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

Trunk
mozilla20
x86_64
Mac OS X
assertion, testcase
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments)

(Reporter)

Description

5 years ago
Created attachment 658642 [details]
testcase

###!!! ASSERTION: If display lists are enabled, only painting of non-display SVG should take this code path: '!NS_SVGDisplayListPaintingEnabled() || (aFrame->GetStateBits() & NS_STATE_SVG_NONDISPLAY_CHILD)', file layout/svg/base/src/nsSVGUtils.cpp, line 1140

This assertion is part of code added in bug 614732.
(Reporter)

Comment 1

5 years ago
Created attachment 658646 [details]
stack trace
(Reporter)

Comment 2

5 years ago
Still happens after the assert was weakened in
http://hg.mozilla.org/mozilla-central/rev/d1b07cfa6e47

###!!! ASSERTION: If display lists are enabled, only painting of non-display SVG should take this code path: '!NS_SVGDisplayListPaintingEnabled() || (aFrame->GetStateBits() & NS_STATE_SVG_NONDISPLAY_CHILD) || aFrame->PresContext()->IsGlyph()', file /Users/jruderman/trees/mozilla-central/layout/svg/base/src/nsSVGUtils.cpp, line 1142
(Assignee)

Updated

5 years ago
Assignee: nobody → jwatt
Created attachment 694428 [details] [diff] [review]
patch
Attachment #694428 - Flags: review?(longsonr)
Comment on attachment 694428 [details] [diff] [review]
patch


Please check in the testcase as a crashtest too.
Attachment #694428 - Flags: review?(longsonr) → review+
https://hg.mozilla.org/mozilla-central/rev/b323378d9d27
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla20
You need to log in before you can comment on or make changes to this bug.