Rewrite the additive-transform-animations test.

Rewrite the additive-transform-animations test.

This should make it faster and not timeout.

BUG=521084
Committed: https://crrev.com/6b4fd8c7083a2a31786e4ee83ab909c00f5fdfdc
Cr-Commit-Position: refs/heads/master@{#379739}

[dsansome, 2016-03-07 04:49:44]

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

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

I suspect this test is flakily timing out because it's waiting on an
asynchronous webkitAnimationStart event before beginning. We can certainly
rewrite this test to be significantly simpler and not rely on events to work.

https://codereview.chromium.org/1768043002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/animations/additive-transform-animations.html
(left):

https://codereview.chromium.org/1768043002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/animations/additive-transform-animations.html:30:
runAnimationTest(expectedValues, null, null, disablePauseAnimationAPI,
doPixelTest);
If we change this to a text expectation test then we are no longer testing a
wide range of code paths, especially in virtual/threaded.

[dsansome, 2016-03-08 03:24:11]

Description was changed from

==========
Use the pauseAnimations API for additive-transform-animations.

This should make it faster and not timeout.

BUG=521084
==========

to

==========
Rewrite the additive-transform-animations test.

This should make it faster and not timeout.

BUG=521084
==========

[dsansome, 2016-03-08 03:24:53]

Oops my bad - I misunderstood what this was doing.
I've rewritten it again to be much simpler, PTAL.

[alancutter (OOO until 2018), 2016-03-08 03:41:21]

lgtm

[dsansome, 2016-03-08 03:43:28]

The CQ bit was checked by dsansome@chromium.org

[commit-bot: I haz the power, 2016-03-08 03:44:00]

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

[commit-bot: I haz the power, 2016-03-08 04:40:34]

Description was changed from

==========
Rewrite the additive-transform-animations test.

This should make it faster and not timeout.

BUG=521084
==========

to

==========
Rewrite the additive-transform-animations test.

This should make it faster and not timeout.

BUG=521084
==========

[commit-bot: I haz the power, 2016-03-08 04:40:35]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-03-08 04:41:44]

Description was changed from

==========
Rewrite the additive-transform-animations test.

This should make it faster and not timeout.

BUG=521084
==========

to

==========
Rewrite the additive-transform-animations test.

This should make it faster and not timeout.

BUG=521084

Committed: https://crrev.com/6b4fd8c7083a2a31786e4ee83ab909c00f5fdfdc
Cr-Commit-Position: refs/heads/master@{#379739}
==========

[commit-bot: I haz the power, 2016-03-08 04:41:44]

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