Last Comment Bug 692203 - "ABORT: Rendering should be disabled" with markerHeight="0"
: "ABORT: Rendering should be disabled" with markerHeight="0"
: assertion, testcase
Product: Core
Classification: Components
Component: SVG (show other bugs)
: Trunk
: x86_64 Mac OS X
-- critical (vote)
: mozilla10
Assigned To: Robert Longson
: Jet Villegas (:jet)
Depends on:
Blocks: 344905 687830
  Show dependency treegraph
Reported: 2011-10-05 12:12 PDT by Jesse Ruderman
Modified: 2011-10-07 04:14 PDT (History)
3 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

testcase (asserts fatally when loaded) (151 bytes, image/svg+xml)
2011-10-05 12:12 PDT, Jesse Ruderman
no flags Details
stack trace (3.20 KB, text/plain)
2011-10-05 12:13 PDT, Jesse Ruderman
no flags Details
patch (1.92 KB, patch)
2011-10-05 13:09 PDT, Robert Longson
dholbert: review+
Details | Diff | Splinter Review

Description User image Jesse Ruderman 2011-10-05 12:12:26 PDT
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.
Comment 1 User image Jesse Ruderman 2011-10-05 12:13:28 PDT
Created attachment 564954 [details]
stack trace
Comment 2 User image Robert Longson 2011-10-05 13:09:24 PDT
Created attachment 564974 [details] [diff] [review]
Comment 3 User image Daniel Holbert [:dholbert] (vacation, returning 2/27) 2011-10-05 13:45:21 PDT
Comment on attachment 564974 [details] [diff] [review]

Looks good, r=me.

Could you add one additional crashtest, using 0 markerWidth (instead of markerHeight)?  We abort on that too, right now.
Comment 5 User image Ed Morley [:emorley] 2011-10-07 04:14:22 PDT

Note You need to log in before you can comment on or make changes to this bug.