Closed
Bug 1083670
Opened 10 years ago
Closed 10 years ago
Separate Web Animations API tests for CSS Animations and CSS Transitions
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla36
People
(Reporter: birtles, Assigned: birtles)
References
Details
Attachments
(1 file)
32.95 KB,
patch
|
dbaron
:
review+
|
Details | Diff | Splinter Review |
Currently all the tests that cover the Web Animations API for CSS Animations and CSS Transitions are lumped together in one directory: dom/animation/test/css-integration I think it makes more sense to split up the tests for CSS Animations and CSS Transitions. I imagine these will continue to be defined in separate specs and have separate test suites. Eventually I think we'll want to upstream these to web-platform-tests where I think they'll live in separate directories. Splitting them out will make that easier, encourages writing tests for both cases (it's currently easy to forget to test transitions), possibly allow more sharing of code, simplify the test files etc. Of course, we still need to test the combination of animations and transitions with regards to this API. I'm proposing those tests go in the CSS Animations folder since CSS Animations seems to build on CSS Transitions, at least at a spec level.
Assignee | ||
Comment 1•10 years ago
|
||
Attachment #8505997 -
Flags: review?(dbaron)
Assignee | ||
Updated•10 years ago
|
Attachment #8505997 -
Flags: review?(dbaron) → review+
Assignee | ||
Comment 2•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/2748f3c6b225
Comment 3•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/2748f3c6b225
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•