Rename AnimationTimeline to DocumentTimeline

RESOLVED FIXED in Firefox 40



3 years ago
3 years ago


(Reporter: birtles, Assigned: birtles)



Dependency tree / graph

Firefox Tracking Flags

(firefox40 fixed)



(3 attachments)



3 years ago
The Web Animations spec has been updated such that AnimationTimeline is now an abstract interface for representing potentially many different types of timelines. The concrete timeline used by default for CSS Animations etc., i.e. the document timeline, is called DocumentTimeline.

We don't plan to implement other types of timelines yet but we should rename the existing names to match the spec for now.


3 years ago
Depends on: 1152619

Comment 1

3 years ago
Created attachment 8590624 [details] [diff] [review]
part 1 - Remove AnimationTimeline IDL tests from dom/animation/tests

The tests in dom/animation/tests/ use an old version of idlharness.js that
doesn't support inherited interfaces. As discussed in bug 1152619 we're not
looking at updating these old tests (under dom/imptests) at the moment which
means we won't be able to update the IDL tests in dom/animation/tests/ to
continue passing once we introduce DocumentTimeline as a subinterface of

As a result, this patch simply the removes the IDL tests for this interface from
dom/animation/tests. However, we have a test for this interface in
web-platform-tests where I've set up a pull request to apply the required
renaming so we should eventually get test coverage for this renaming.

In the long run, all the tests in dom/animation/tests should end up in
web-platform-tests. The main reason they aren't there yet is that most of them
test the mapping between the Web Animations API and CSS and there's currently no
spec defining that so there's no place to put them in the web-platform-tests

There are a few tests for animation timeline which could be landed in
web-platform-tests (and then removed from dom/animation/tests) but we need to
discuss with Google if this is the desired behavior or not first. For the time
being I have a branch setup for that and I'm leaving the tests in
dom/animation/tests so we continue to test what *we* think the behavior should
be in the meantime. That branch is here:
Attachment #8590624 - Flags: review?(james)

Comment 2

3 years ago
Created attachment 8590627 [details] [diff] [review]
part 2 - Rename AnimationTimeline to DocumentTimeline

And then re-add AnimationTimeline has an abstract super-class of
Attachment #8590627 - Flags: review?(bugs)

Comment 3

3 years ago
Created attachment 8590628 [details] [diff] [review]
part 3 - Update web-platform-tests expectations

Until the renaming PR (
lands upstream and we update, this test will fail.
Attachment #8590628 - Flags: review?(james)

Comment 4

3 years ago
Try run:
Comment on attachment 8590624 [details] [diff] [review]
part 1 - Remove AnimationTimeline IDL tests from dom/animation/tests

Review of attachment 8590624 [details] [diff] [review]:

I think Ms2ger should review things in imptest
Attachment #8590624 - Flags: review?(james) → review?(Ms2ger)
Attachment #8590627 - Flags: review?(bugs) → review+
Attachment #8590628 - Flags: review?(james) → review+
Comment on attachment 8590624 [details] [diff] [review]
part 1 - Remove AnimationTimeline IDL tests from dom/animation/tests

Review of attachment 8590624 [details] [diff] [review]:

Attachment #8590624 - Flags: review?(Ms2ger) → review+

Comment 7

3 years ago
Last Resolved: 3 years ago
status-firefox40: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla40


3 years ago
Blocks: 1154615


3 years ago
Keywords: dev-doc-needed
You need to log in before you can comment on or make changes to this bug.