Closed Bug 1152171 Opened 5 years ago Closed 5 years ago

Rename AnimationTimeline to DocumentTimeline

Categories

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

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla40
Tracking Status
firefox40 --- fixed

People

(Reporter: birtles, Assigned: birtles)

References

Details

(Keywords: dev-doc-needed)

Attachments

(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
AnimationTimeline.

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.

  https://github.com/w3c/web-platform-tests/pull/1748

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
repository.

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:

  https://github.com/birtles/web-platform-tests/compare/rename-animation-timeline...birtles:add-hidden-iframe-tests
Attachment #8590624 - Flags: review?(james)
And then re-add AnimationTimeline has an abstract super-class of
DocumentTimeline.
Attachment #8590627 - Flags: review?(bugs)
Until the renaming PR (https://github.com/w3c/web-platform-tests/pull/1748)
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]:
-----------------------------------------------------------------

rs=me
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.