Document the Web Animation interfaces with spec links and TODOs

Document the Web Animation interfaces with spec links and TODOs

Almost all of these interfaces are out of sync with the latest spec,
only the two dictionaries were close enough to make it worthwhile to
document the differences on a per-member level.

BUG=460722
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=194042

[philipj_slow, 2015-04-18 21:54:25]

philipj@opera.com changed reviewers:
+ dstockwell@chromium.org

[philipj_slow, 2015-04-18 21:54:25]

PTAL. This is part of the effort to document where our IDL files don't match the
specs, but in this case it became somewhat odd with so many differences from the
latest spec. I boldly put your name on the TODOs based on the history of the
files, but I'll revert to my own name if you don't think you'll actually be the
one to work on this.

[dstockwell, 2015-04-20 05:42:34]

On 2015/04/18 at 21:54:25, philipj wrote:
> PTAL. This is part of the effort to document where our IDL files don't match
the specs, but in this case it became somewhat odd with so many differences from
the latest spec. I boldly put your name on the TODOs based on the history of the
files, but I'll revert to my own name if you don't think you'll actually be the
one to work on this.

Thanks, these are indeed my todos. lgtm with a couple of minor clarifications.

[philipj_slow, 2015-04-20 09:07:56]

On 2015/04/20 05:42:34, dstockwell wrote:
> On 2015/04/18 at 21:54:25, philipj wrote:
> > PTAL. This is part of the effort to document where our IDL files don't match
> the specs, but in this case it became somewhat odd with so many differences
from
> the latest spec. I boldly put your name on the TODOs based on the history of
the
> files, but I'll revert to my own name if you don't think you'll actually be
the
> one to work on this.
> 
> Thanks, these are indeed my todos. lgtm with a couple of minor clarifications.

Thanks! I think you forgot to submit the bits in need of clarification :)

[philipj_slow, 2015-04-20 13:10:53]

adjust URLs that have been fixed in spec

[philipj_slow, 2015-04-20 13:14:26]

fix a few typos

[dstockwell, 2015-04-20 13:32:24]

Trying to send comments again...

https://codereview.chromium.org/1092203002/diff/1/Source/core/animation/Anima...
File Source/core/animation/Animation.idl (right):

https://codereview.chromium.org/1092203002/diff/1/Source/core/animation/Anima...
Source/core/animation/Animation.idl:34: //
https://w3c.github.io/web-animations/#the-animation-interface
This is renamed KeyframeEffect in the latest spec.

https://codereview.chromium.org/1092203002/diff/1/Source/core/animation/Anima...
File Source/core/animation/AnimationPlayer.idl (right):

https://codereview.chromium.org/1092203002/diff/1/Source/core/animation/Anima...
Source/core/animation/AnimationPlayer.idl:34: // spec, where it is no longer an
EventTarget and has other changes:
It will be an EventTarget again: https://github.com/w3c/web-animations/issues/71

[philipj_slow, 2015-04-20 15:17:14]

update as per dstockwell's feedback

[philipj_slow, 2015-04-20 15:17:54]

Thanks, I've updated those two comments to be more accurate and skipped
mentioning EventTarget. Landing now, but just comment if I got something wrong
and I'll fix it.

[philipj_slow, 2015-04-20 15:17:59]

The CQ bit was checked by philipj@opera.com

[philipj_slow, 2015-04-20 15:17:59]

The patchset sent to the CQ was uploaded after l-g-t-m from
dstockwell@chromium.org
Link to the patchset: https://codereview.chromium.org/1092203002/#ps50001
(title: "update as per dstockwell's feedback")

[commit-bot: I haz the power, 2015-04-20 15:18:34]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1092203002/50001

[commit-bot: I haz the power, 2015-04-20 18:11:39]

Committed patchset #4 (id:50001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=194042