Make AnimationEffectReadOnly.getComputedTiming() consistent with Web Animations spec

Make AnimationEffectReadOnly.getComputedTiming() consistent with Web Animations spec

This patch changes the AnimationEffectReadOnly interface to have a
getComputedTiming() method instead of a computedTiming attribute:
https://w3c.github.io/web-animations/#dom-animationeffectreadonly-getcomputedtiming

This also updates the returned ComputedTimingProperties dictionary
members to match spec:
https://w3c.github.io/web-animations/#the-computedtimingproperties-dictionary

BUG=600248
Committed: https://crrev.com/cd3a7885f9c5c7ca6e209142af3f73be6ef5d375
Cr-Commit-Position: refs/heads/master@{#391750}

[alancutter (OOO until 2018), 2016-05-03 04:20:10]

alancutter@chromium.org changed reviewers:
+ suzyh@chromium.org

[alancutter (OOO until 2018), 2016-05-03 04:20:11]

https://codereview.chromium.org/1944603003/diff/20001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/web-animations-api/timed-item-expected.txt
(right):

https://codereview.chromium.org/1944603003/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/web-animations-api/timed-item-expected.txt:6:
FAIL TimedItem startTime, endTime, duration, activeDuration are sensible for
keyframeEffects with delays and iterations assert_equals: expected (number) 0
but got (undefined) undefined
startTime doesn't exist in the spec, our version of the web-platform-tests is
out of date.

[alancutter (OOO until 2018), 2016-05-03 04:20:20]

The CQ bit was checked by alancutter@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-05-03 04:20:27]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1944603003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1944603003/20001

[alancutter (OOO until 2018), 2016-05-03 04:23:41]

Description was changed from

==========
6# Enter a description of the change.
Make AnimationEffect.getComputedTiming() consistent with Web Animations spec

This patch changes the AnimationEffectReadOnly interface to have a
getComputedTiming() method instead of a computedTiming attribute:
https://w3c.github.io/web-animations/#dom-animationeffectreadonly-getcomputed...

This also updates the returned ComputedTimingProperties dictionary
members to match spec:
https://w3c.github.io/web-animations/#the-computedtimingproperties-dictionary


BUG=600248
==========

to

==========
Make AnimationEffect.getComputedTiming() consistent with Web Animations spec

This patch changes the AnimationEffectReadOnly interface to have a
getComputedTiming() method instead of a computedTiming attribute:
https://w3c.github.io/web-animations/#dom-animationeffectreadonly-getcomputed...

This also updates the returned ComputedTimingProperties dictionary
members to match spec:
https://w3c.github.io/web-animations/#the-computedtimingproperties-dictionary


BUG=600248
==========

[alancutter (OOO until 2018), 2016-05-03 05:07:34]

https://codereview.chromium.org/1944603003/diff/60001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/imported/web-platform-tests/web-animations/keyframe-effect/getComputedTiming-progress-expected.txt
(right):

https://codereview.chromium.org/1944603003/diff/60001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/imported/web-platform-tests/web-animations/keyframe-effect/getComputedTiming-progress-expected.txt:2:
FAIL Test progress during before and after phase when fill is none
assert_equals: expected (object) null but got (undefined) undefined
This seems to be an IDL bindings bug where T? members aren't being returned as
null to Javascript.
Created patch to fix: https://codereview.chromium.org/1946453002

[alancutter (OOO until 2018), 2016-05-03 05:07:40]

The CQ bit was checked by alancutter@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-05-03 05:08:17]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1944603003/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1944603003/60001

[commit-bot: I haz the power, 2016-05-03 06:38:13]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-05-03 06:38:14]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[alancutter (OOO until 2018), 2016-05-03 07:51:59]

Description was changed from

==========
Make AnimationEffect.getComputedTiming() consistent with Web Animations spec

This patch changes the AnimationEffectReadOnly interface to have a
getComputedTiming() method instead of a computedTiming attribute:
https://w3c.github.io/web-animations/#dom-animationeffectreadonly-getcomputed...

This also updates the returned ComputedTimingProperties dictionary
members to match spec:
https://w3c.github.io/web-animations/#the-computedtimingproperties-dictionary


BUG=600248
==========

to

==========
Make AnimationEffectReadOnly.getComputedTiming() consistent with Web Animations
spec

This patch changes the AnimationEffectReadOnly interface to have a
getComputedTiming() method instead of a computedTiming attribute:
https://w3c.github.io/web-animations/#dom-animationeffectreadonly-getcomputed...

This also updates the returned ComputedTimingProperties dictionary
members to match spec:
https://w3c.github.io/web-animations/#the-computedtimingproperties-dictionary


BUG=600248
==========

[alancutter (OOO until 2018), 2016-05-03 08:18:55]

The CQ bit was checked by alancutter@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-05-03 08:19:08]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1944603003/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1944603003/100001

[commit-bot: I haz the power, 2016-05-03 10:50:44]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-05-03 10:50:50]

Dry run: This issue passed the CQ dry run.

[suzyh_UTC10 (ex-contributor), 2016-05-03 23:38:03]

lgtm

[alancutter (OOO until 2018), 2016-05-03 23:41:49]

The CQ bit was checked by alancutter@chromium.org

[commit-bot: I haz the power, 2016-05-03 23:48:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1944603003/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1944603003/100001

[commit-bot: I haz the power, 2016-05-04 00:00:37]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-05-04 00:00:38]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[alancutter (OOO until 2018), 2016-05-04 00:22:53]

alancutter@chromium.org changed reviewers:
+ pfeldman@chromium.org

[alancutter (OOO until 2018), 2016-05-04 00:22:54]

+pfeldman for core/inspector/InspectorAnimationAgent.cpp change.

[pfeldman, 2016-05-04 23:02:03]

lgtm

[alancutter (OOO until 2018), 2016-05-05 01:02:31]

The CQ bit was checked by alancutter@chromium.org

[commit-bot: I haz the power, 2016-05-05 01:02:54]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1944603003/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1944603003/100001

[commit-bot: I haz the power, 2016-05-05 03:53:59]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-05-05 03:55:36]

Description was changed from

==========
Make AnimationEffectReadOnly.getComputedTiming() consistent with Web Animations
spec

This patch changes the AnimationEffectReadOnly interface to have a
getComputedTiming() method instead of a computedTiming attribute:
https://w3c.github.io/web-animations/#dom-animationeffectreadonly-getcomputed...

This also updates the returned ComputedTimingProperties dictionary
members to match spec:
https://w3c.github.io/web-animations/#the-computedtimingproperties-dictionary


BUG=600248
==========

to

==========
Make AnimationEffectReadOnly.getComputedTiming() consistent with Web Animations
spec

This patch changes the AnimationEffectReadOnly interface to have a
getComputedTiming() method instead of a computedTiming attribute:
https://w3c.github.io/web-animations/#dom-animationeffectreadonly-getcomputed...

This also updates the returned ComputedTimingProperties dictionary
members to match spec:
https://w3c.github.io/web-animations/#the-computedtimingproperties-dictionary

BUG=600248

Committed: https://crrev.com/cd3a7885f9c5c7ca6e209142af3f73be6ef5d375
Cr-Commit-Position: refs/heads/master@{#391750}
==========

[commit-bot: I haz the power, 2016-05-05 03:55:38]

Patchset 6 (id:??) landed as
https://crrev.com/cd3a7885f9c5c7ca6e209142af3f73be6ef5d375
Cr-Commit-Position: refs/heads/master@{#391750}