Expose readonly animation timeline

Expose readonly animation timeline

This patch exposes the Animation.timeline attribute in Animation.idl, as a
readonly attribute with mutability left as a TODO. By exposing this feature, a
few more of the Web Animations web platform tests pass.

Two tests go from failing to flaky:
- imported/wpt/web-animations/interfaces/AnimationEffectTiming/endDelay.html
- imported/wpt/web-animations/interfaces/Animation/playbackRate.html
Where previously these tests failed because timeline was undefined, they now
fail due to values of the timeline sometimes not being as expected, which is a
separate issue.

Something very similar to this patch had previously been submitted in
crrev.com/1942683002 but was reverted due to these flakes.

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

[suzyh_UTC10 (ex-contributor), 2016-06-27 06:53:40]

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

[suzyh_UTC10 (ex-contributor), 2016-06-27 06:54:01]

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

[suzyh_UTC10 (ex-contributor), 2016-06-27 06:54:46]

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

[commit-bot: I haz the power, 2016-06-27 06:54:58]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[suzyh_UTC10 (ex-contributor), 2016-06-27 07:32:20]

Description was changed from

==========
Expose readonly animation timeline

This patch exposes the Animation.timeline attribute in Animation.idl, as a
readonly attribute with mutability left as a TODO. By exposing this feature, a
few more of the Web Animations web platform tests pass.

Two tests go from failing to flaky:
- imported/wpt/web-animations/interfaces/AnimationEffectTiming/endDelay.html
- imported/wpt/web-animations/interfaces/Animation/playbackRate.html
Where previously these tests failed because timeline was undefined, they now
fail due to values of the timeline sometimes not being as expected, which is a
separate issue.

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

to

==========
Expose readonly animation timeline

This patch exposes the Animation.timeline attribute in Animation.idl, as a
readonly attribute with mutability left as a TODO. By exposing this feature, a
few more of the Web Animations web platform tests pass.

Two tests go from failing to flaky:
- imported/wpt/web-animations/interfaces/AnimationEffectTiming/endDelay.html
- imported/wpt/web-animations/interfaces/Animation/playbackRate.html
Where previously these tests failed because timeline was undefined, they now
fail due to values of the timeline sometimes not being as expected, which is a
separate issue.

Something very similar to this patch had previously been submitted in
crrev.com/1942683002 but was reverted due to these flakes.

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

[suzyh_UTC10 (ex-contributor), 2016-06-27 07:34:31]

I just realised that we had attempted to land (something very similar to) this
patch before: https://codereview.chromium.org/1942683002/ but it was reverted
for causing test failures. The reported test failure is covered by the
additional TestExpectations flake line.

[commit-bot: I haz the power, 2016-06-27 08:12:27]

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

[commit-bot: I haz the power, 2016-06-27 08:12:28]

Dry run: This issue passed the CQ dry run.

[Eric Willigers, 2016-06-28 00:15:33]

lgtm

[suzyh_UTC10 (ex-contributor), 2016-06-28 00:17:48]

The CQ bit was checked by suzyh@chromium.org

[commit-bot: I haz the power, 2016-06-28 00:18:46]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-06-28 00:34:05]

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

[commit-bot: I haz the power, 2016-06-28 00:34:06]

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

[suzyh_UTC10 (ex-contributor), 2016-06-28 00:52:08]

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

[suzyh_UTC10 (ex-contributor), 2016-06-28 00:52:17]

+alancutter for OWNERS approval

[alancutter (OOO until 2018), 2016-06-29 06:04:50]

lgtm

[suzyh_UTC10 (ex-contributor), 2016-06-29 06:05:40]

The CQ bit was checked by suzyh@chromium.org

[commit-bot: I haz the power, 2016-06-29 06:06:01]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-06-29 07:33:32]

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

[commit-bot: I haz the power, 2016-06-29 07:33:33]

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

[suzyh_UTC10 (ex-contributor), 2016-06-29 23:23:30]

On 2016/06/29 at 07:33:33, commit-bot wrote:
> Try jobs failed on following builders:
>   linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

The web-platform-tests were updated a couple of days ago and there's a new test
with expectations affected by this patch. It's interesting that the test passed
when animation.timeline was undefined but failed when it was defined but
readonly.

PTAL

[Eric Willigers, 2016-06-29 23:31:51]

lgtm

[suzyh_UTC10 (ex-contributor), 2016-06-29 23:38:20]

The CQ bit was checked by suzyh@chromium.org

[suzyh_UTC10 (ex-contributor), 2016-06-29 23:38:21]

The patchset sent to the CQ was uploaded after l-g-t-m from
alancutter@chromium.org
Link to the patchset: https://codereview.chromium.org/2094213002/#ps20001
(title: "Update test expectation for new WPT test")

[commit-bot: I haz the power, 2016-06-29 23:39:44]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-06-30 01:14:39]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-30 01:14:50]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-06-30 01:16:36]

Description was changed from

==========
Expose readonly animation timeline

This patch exposes the Animation.timeline attribute in Animation.idl, as a
readonly attribute with mutability left as a TODO. By exposing this feature, a
few more of the Web Animations web platform tests pass.

Two tests go from failing to flaky:
- imported/wpt/web-animations/interfaces/AnimationEffectTiming/endDelay.html
- imported/wpt/web-animations/interfaces/Animation/playbackRate.html
Where previously these tests failed because timeline was undefined, they now
fail due to values of the timeline sometimes not being as expected, which is a
separate issue.

Something very similar to this patch had previously been submitted in
crrev.com/1942683002 but was reverted due to these flakes.

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

to

==========
Expose readonly animation timeline

This patch exposes the Animation.timeline attribute in Animation.idl, as a
readonly attribute with mutability left as a TODO. By exposing this feature, a
few more of the Web Animations web platform tests pass.

Two tests go from failing to flaky:
- imported/wpt/web-animations/interfaces/AnimationEffectTiming/endDelay.html
- imported/wpt/web-animations/interfaces/Animation/playbackRate.html
Where previously these tests failed because timeline was undefined, they now
fail due to values of the timeline sometimes not being as expected, which is a
separate issue.

Something very similar to this patch had previously been submitted in
crrev.com/1942683002 but was reverted due to these flakes.

BUG=600248

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

[commit-bot: I haz the power, 2016-06-30 01:16:37]

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