Closed
Bug 473702
Opened 16 years ago
Closed 16 years ago
SVG SMIL: Implement SVGAnimationElement interface
Categories
(Core :: SVG, defect)
Core
SVG
Tracking
()
RESOLVED
FIXED
People
(Reporter: birtles, Assigned: birtles)
References
()
Details
Attachments
(1 file, 2 obsolete files)
(deleted),
patch
|
birtles
:
review+
birtles
:
superreview+
|
Details | Diff | Splinter Review |
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
Assignee | ||
Comment 1•16 years ago
|
||
Implementation of remaining SVGAnimationElement methods including mochitests
Assignee | ||
Updated•16 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•16 years ago
|
||
Fixed some documentation issues
Attachment #357600 -
Attachment is obsolete: true
Attachment #357605 -
Flags: superreview?(roc)
Attachment #357605 -
Flags: review?(roc)
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
Assignee | ||
Comment 4•16 years ago
|
||
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
Flags: in-testsuite+
Updated•15 years ago
|
Keywords: dev-doc-needed
Updated•15 years ago
|
Whiteboard: [doc-waiting-1.9.3]
Comment 6•14 years ago
|
||
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.
Description
•