Closed Bug 756988 Opened 10 years ago Closed 10 years ago

Mark more SVG feature strings as supported

Categories

(Core :: SVG, defect)

x86_64
Windows 7
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla15

People

(Reporter: longsonr, Assigned: longsonr)

References

Details

Attachments

(1 file)

Attached patch patchSplinter Review
We do all the animation dom interfaces now (http://www.w3.org/TR/SVG/animate.html#DOMInterfaces) apart from animateColor which is deprecated.

I think it's time to mark animation as done too. The only other thing we don't do is wallclock timing I think which we're not planning to do (see http://brian.sol1.net/svg/2010/10/13/its-about-time/)
Attachment #625582 - Attachment is patch: true
Attachment #625582 - Flags: review?(jwatt)
Attachment #625582 - Flags: feedback?(birtles)
Comment on attachment 625582 [details] [diff] [review]
patch

Seems good to me, assuming Brian agrees.
Attachment #625582 - Flags: review?(jwatt) → review+
Comment on attachment 625582 [details] [diff] [review]
patch

I'm a little hesitant since we don't support wallclock timing, but I think it's ok since we've argued for it to be deprecated (and it probably will be in SVG 2).

The description for 'Animation' is that UA supports "the ‘animate’, ‘set’, ‘animateMotion’, ‘animateTransform’, ‘animateColor’ and ‘mpath’ elements" and I'm comfortable saying we do that given that animateColor has already been deprecated.
Attachment #625582 - Flags: feedback?(birtles) → feedback+
https://hg.mozilla.org/integration/mozilla-inbound/rev/61e64b3dba61
Flags: in-testsuite+
Target Milestone: --- → mozilla15
https://hg.mozilla.org/mozilla-central/rev/61e64b3dba61
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Assignee: nobody → longsonr
Depends on: 435513
You need to log in before you can comment on or make changes to this bug.