Closed Bug 473702 Opened 16 years ago Closed 16 years ago

SVG SMIL: Implement SVGAnimationElement interface

Categories

(Core :: SVG, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: birtles, Assigned: birtles)

References

()

Details

Attachments

(1 file, 2 obsolete files)

In order to support more automated testing of SMIL we need to implement the SVGAnimationElement. Specifically, the following three methods are not implemented in our SMIL implementation: getStartTime() getCurrentTime() getSimpleDuration() Implementing these, particularly getStartTime(), would be valuable for testing parsing and other components of our implementation. Unfortunately, the specification of getStartTime() is not fully defined. I have posted a query to www-svg seeking clarification on this: http://lists.w3.org/Archives/Public/www-svg/2009Jan/0023.html I have also produced a test case which examines the behaviour of most of the methods in this interface here: http://brian.sol1.net/svg/tests/getStartTime.svg
Attached patch patch v1a (obsolete) (deleted) — Splinter Review
Implementation of remaining SVGAnimationElement methods including mochitests
Status: NEW → ASSIGNED
Attached patch patch v1b (obsolete) (deleted) — Splinter Review
Fixed some documentation issues
Attachment #357600 - Attachment is obsolete: true
Attachment #357605 - Flags: superreview?(roc)
Attachment #357605 - Flags: review?(roc)
Blocks: 474257
Attachment #357605 - Flags: superreview?(roc)
Attachment #357605 - Flags: superreview+
Attachment #357605 - Flags: review?(roc)
Attachment #357605 - Flags: review+
Comment on attachment 357605 [details] [diff] [review] patch v1b _TEST_FILES = test_smilRestart.xhtml \ + test_smilGetStartTime.xhtml \ + test_smilGetSimpleDuration.xhtml \ A wee bit overindented
Attached patch patch v1c (deleted) — Splinter Review
Fixed whitespace issue.
Attachment #357605 - Attachment is obsolete: true
Attachment #357613 - Flags: superreview+
Attachment #357613 - Flags: review+
Pushed 4e14d22f6961
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Whiteboard: [doc-waiting-1.9.3]
Given that we currently don't explicitly document SVG at all, I'm removing the doc needed keyword from this in favor of bug 602022, which covers writing a complete set of SVG documentation.
Keywords: dev-doc-needed
Whiteboard: [doc-waiting-1.9.3]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: