Remove death test in CompositorAnimationsTest

Remove death test in CompositorAnimationsTest

AnimationCompositorAnimationsTest.ConvertTimingForCompositorIterationCount
contains an EXPECT_DEATH test which checks that
CompositorAnimations::convertTimingForCompositor hits an assertion failure if
passed a Timing object with a value of -1 for iterationCount. This test is
probably included because -1 is a valid value for the equivalent field in the
output CompositorTiming object.

However, EXPECT_DEATH is flaky and dangerous in a multi-threaded context. Since
we're not gaining much by having this test, this patch removes it.

BUG=631021
Committed: https://crrev.com/0bc11b2ac870eb60d74f0f56337703ff6d4f1c80
Cr-Commit-Position: refs/heads/master@{#407674}

[suzyh_UTC10 (ex-contributor), 2016-07-26 00:18:32]

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

[suzyh_UTC10 (ex-contributor), 2016-07-26 00:18:39]

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

[commit-bot: I haz the power, 2016-07-26 00:19:31]

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

[alancutter (OOO until 2018), 2016-07-26 00:23:09]

lgtm

[suzyh_UTC10 (ex-contributor), 2016-07-26 00:28:32]

The CQ bit was unchecked by suzyh@chromium.org

[suzyh_UTC10 (ex-contributor), 2016-07-26 00:28:34]

The CQ bit was checked by suzyh@chromium.org

[commit-bot: I haz the power, 2016-07-26 00:29:32]

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-07-26 01:27:26]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-26 01:29:30]

Description was changed from

==========
Remove death test in CompositorAnimationsTest

AnimationCompositorAnimationsTest.ConvertTimingForCompositorIterationCount
contains an EXPECT_DEATH test which checks that
CompositorAnimations::convertTimingForCompositor hits an assertion failure if
passed a Timing object with a value of -1 for iterationCount. This test is
probably included because -1 is a valid value for the equivalent field in the
output CompositorTiming object.

However, EXPECT_DEATH is flaky and dangerous in a multi-threaded context. Since
we're not gaining much by having this test, this patch removes it.

BUG=631021
==========

to

==========
Remove death test in CompositorAnimationsTest

AnimationCompositorAnimationsTest.ConvertTimingForCompositorIterationCount
contains an EXPECT_DEATH test which checks that
CompositorAnimations::convertTimingForCompositor hits an assertion failure if
passed a Timing object with a value of -1 for iterationCount. This test is
probably included because -1 is a valid value for the equivalent field in the
output CompositorTiming object.

However, EXPECT_DEATH is flaky and dangerous in a multi-threaded context. Since
we're not gaining much by having this test, this patch removes it.

BUG=631021

Committed: https://crrev.com/0bc11b2ac870eb60d74f0f56337703ff6d4f1c80
Cr-Commit-Position: refs/heads/master@{#407674}
==========

[commit-bot: I haz the power, 2016-07-26 01:29:32]

Patchset 1 (id:??) landed as
https://crrev.com/0bc11b2ac870eb60d74f0f56337703ff6d4f1c80
Cr-Commit-Position: refs/heads/master@{#407674}