Closed Bug 1152171 Opened 9 years ago Closed 9 years ago

Rename AnimationTimeline to DocumentTimeline


(Core :: DOM: Core & HTML, defect)

Not set



Tracking Status
firefox40 --- fixed


(Reporter: birtles, Assigned: birtles)



(Keywords: dev-doc-needed)


(3 files)

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.
Depends on: 1152619
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)
And then re-add AnimationTimeline has an abstract super-class of
Attachment #8590627 - Flags: review?(bugs)
Until the renaming PR (
lands upstream and we update, this test will fail.
Attachment #8590628 - Flags: review?(james)
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+
Blocks: 1154615
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.