570555 - CSS animation on font-size should not require units

Status: RESOLVED FIXED

(Core :: SVG, defect)

Description

[Robert Longson [:longsonr]]

Attachment: patch

See font-size row in http://www.w3.org/Graphics/SVG/Test/20061213/htmlObjectHarness/full-animate-elem-46-t.html and http://lists.w3.org/Archives/Public/www-svg/2010Jun/0088.html

Comment 1

[Daniel Holbert [:dholbert]]

I posted a response to that thread:
http://lists.w3.org/Archives/Public/www-svg/2010Jun/0102.html

At this point, based on my reading of the relevant specs, I don't think we should make this change.  I think our current behavior[1] is the most sane of the various options.

[1] To be clear, our current behavior is: accept unitless <length> values for attributeType="XML" animations, but reject unitless <length> values for attributeType="CSS" animations. (and for attributeType="auto", follow the rules for CSS vs XML depending on which bucket "auto" falls into for the attributeName in question)

Comment 2

[Daniel Holbert [:dholbert]]

(though I'm willing to be convinced that I'm wrong)

Comment 3

[Daniel Holbert [:dholbert]]

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

And I've been convinced, per the second half of http://lists.w3.org/Archives/Public/www-svg/2010Jun/0114.html

r=dholbert

Comment 4

[Robert Longson [:longsonr]]

pushed http://hg.mozilla.org/mozilla-central/rev/e63754724e81