"ABORT: Rendering should be disabled" with markerHeight="0"

RESOLVED FIXED in mozilla10

Status

()

Core
SVG
--
critical
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Jesse Ruderman, Assigned: Robert Longson)

Tracking

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

Trunk
mozilla10
x86_64
Mac OS X
assertion, testcase
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments)

(Reporter)

Description

6 years ago
Created attachment 564952 [details]
testcase (asserts fatally when loaded)

###!!! ABORT: Rendering should be disabled: 'viewbox.width > 0.0f && viewbox.height > 0.0f', file content/svg/content/src/nsSVGMarkerElement.cpp, line 383

Guessing this is a regression from bug 687830, which added the assertion.
(Reporter)

Comment 1

6 years ago
Created attachment 564954 [details]
stack trace
(Assignee)

Comment 2

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

Looks good, r=me.

Could you add one additional crashtest, using 0 markerWidth (instead of markerHeight)?  We abort on that too, right now.
Attachment #564974 - Flags: review?(dholbert) → review+
(Assignee)

Comment 4

6 years ago
pushed https://hg.mozilla.org/integration/mozilla-inbound/rev/e390059dce99

Comment 5

6 years ago
https://hg.mozilla.org/mozilla-central/rev/e390059dce99
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla10
You need to log in before you can comment on or make changes to this bug.