cc: Fix animation bounds computation for multiple players

cc: Fix animation bounds computation for multiple players

When one ElementAnimations has multiple AnimationPlayers, the
ElementAnimations can have an animation that inflates bounds without
every player having such an animation. This CL makes ElementAnimations
correctly handle this case, fixing a DCHECK that was previously
being hit in AnimationPlayer::TransformAnimationBoundsForBox.

BUG=675545
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2655993002
Cr-Commit-Position: refs/heads/master@{#447812}
Committed: https://chromium.googlesource.com/chromium/src/+/bd39ee6aee08c93647e9174347d7926e58b11012

[ajuma, 2017-01-25 22:07:10]

Description was changed from

==========
cc: Fix animation bounds computation for multiple players

When one ElementAnimations has multiple AnimationPlayers, the
ElementAnimations can have an animation that inflates bounds without
every player having such an animation. This CL makes ElementAnimations
correctly handle this case, fixing a DCHECK that was previously
being hit in AnimationPlayer::TransformAnimationBoundsForBox.

BUG=675545
==========

to

==========
cc: Fix animation bounds computation for multiple players

When one ElementAnimations has multiple AnimationPlayers, the
ElementAnimations can have an animation that inflates bounds without
every player having such an animation. This CL makes ElementAnimations
correctly handle this case, fixing a DCHECK that was previously
being hit in AnimationPlayer::TransformAnimationBoundsForBox.

BUG=675545
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[ajuma, 2017-01-25 22:07:39]

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

[commit-bot: I haz the power, 2017-01-25 22:08:28]

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

[ajuma, 2017-01-25 22:35:10]

Description was changed from

==========
cc: Fix animation bounds computation for multiple players

When one ElementAnimations has multiple AnimationPlayers, the
ElementAnimations can have an animation that inflates bounds without
every player having such an animation. This CL makes ElementAnimations
correctly handle this case, fixing a DCHECK that was previously
being hit in AnimationPlayer::TransformAnimationBoundsForBox.

BUG=675545
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
cc: Fix animation bounds computation for multiple players

When one ElementAnimations has multiple AnimationPlayers, the
ElementAnimations can have an animation that inflates bounds without
every player having such an animation. This CL makes ElementAnimations
correctly handle this case, fixing a DCHECK that was previously
being hit in AnimationPlayer::TransformAnimationBoundsForBox.

BUG=675545
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[ajuma, 2017-01-25 22:35:10]

ajuma@chromium.org changed reviewers:
+ loyso@chromium.org

[commit-bot: I haz the power, 2017-01-25 23:47:29]

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

[commit-bot: I haz the power, 2017-01-25 23:47:29]

Dry run: This issue passed the CQ dry run.

[loyso (OOO), 2017-02-01 00:25:44]

LGTM

[ajuma, 2017-02-02 18:16:10]

The CQ bit was checked by ajuma@chromium.org

[commit-bot: I haz the power, 2017-02-02 18:16: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, 2017-02-02 19:14:43]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1486059370264920, "parent_rev":
"101569dac2a9cee88240aa87fc695b9b6fb93aa1", "commit_rev":
"bd39ee6aee08c93647e9174347d7926e58b11012"}

[commit-bot: I haz the power, 2017-02-02 19:15:16]

Description was changed from

==========
cc: Fix animation bounds computation for multiple players

When one ElementAnimations has multiple AnimationPlayers, the
ElementAnimations can have an animation that inflates bounds without
every player having such an animation. This CL makes ElementAnimations
correctly handle this case, fixing a DCHECK that was previously
being hit in AnimationPlayer::TransformAnimationBoundsForBox.

BUG=675545
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
cc: Fix animation bounds computation for multiple players

When one ElementAnimations has multiple AnimationPlayers, the
ElementAnimations can have an animation that inflates bounds without
every player having such an animation. This CL makes ElementAnimations
correctly handle this case, fixing a DCHECK that was previously
being hit in AnimationPlayer::TransformAnimationBoundsForBox.

BUG=675545
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2655993002
Cr-Commit-Position: refs/heads/master@{#447812}
Committed:
https://chromium.googlesource.com/chromium/src/+/bd39ee6aee08c93647e9174347d7...
==========

[commit-bot: I haz the power, 2017-02-02 19:15:16]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/bd39ee6aee08c93647e9174347d7...