934156 - Make the SVG elements' BuildPath methods return nullptr when the element is disabled

Status: RESOLVED FIXED

(Core :: SVG, defect)

Description

[Jonathan Watt [:jwatt]]

I should have made the SVG elements' BuildPath methods return nullptr when the element is disabled in bug 932761. It's more efficient, but more to the point it turns out some of the Moz2D backends can end up returning stroke bounds with NaN values in the case that no path is emitted. :/

This also fixes a bug with the width/height checking in SVGImageElement::BuildPath where the logic was the wrong way around.

Comment 1

[Jonathan Watt [:jwatt]]

Attachment: patch

Comment 2

[Daniel Holbert [:dholbert]]

Comment on attachment 826342 [details] [diff] [review]
patch

(In reply to Jonathan Watt [:jwatt] from comment #0)
> it turns out some of the Moz2D backends can end up returning stroke
> bounds with NaN values in the case that no path is emitted. :/

Are there automated testcases somewhere (maybe some random already-in-the-tree tests) that fail due to this when using those backends?

r=me

Comment 3

[Daniel Holbert [:dholbert]]

Also: Please update the BuildPath() header-comment here...
http://mxr.mozilla.org/mozilla-central/source/content/svg/content/src/nsSVGPathGeometryElement.h?rev=e05ec9f34435&mark=63-65#63
...to mention that the method may return null, in cases where the element has no valid path. (or something like that)

Comment 4

[Jonathan Watt [:jwatt]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/f2f5e3c75e15

(In reply to Daniel Holbert [:dholbert] from comment #2)
> Are there automated testcases somewhere (maybe some random
> already-in-the-tree tests) that fail due to this when using those backends?

Yes, quite a few.

(In reply to Daniel Holbert [:dholbert] from comment #3)
> Also: Please update the BuildPath() header-comment

Done.

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/f2f5e3c75e15