Blink CompositorAnimations: Do not crash if no compositor animation player.

Blink CompositorAnimations: Do not crash if no compositor animation player.

blink::Animation can loose compositorPlayer if Animation::m_content detached via
Animation::setEffect(nullptr)

If KeyframeEffect::cancelAnimationOnCompositor wants to clear m_compositorAnimationIds
then it shouldn't crash.

Also, note the existing comment in CompositorAnimations::cancelAnimationOnCompositor (above changed lines)

BUG=571504
R=dstockwell@chromium.org

Committed: https://crrev.com/48abeaf3f8886a99e68722d55050547482863fb7
Cr-Commit-Position: refs/heads/master@{#367055}

[loyso (OOO), 2015-12-23 03:23:09]

Description was changed from

==========
Blink CompositorAnimations: Do not crash if no compositor animation player.

blink::Animation can lose compositorPlayer if Animation::m_content detached via
Animation::setEffect(nullptr)

If KeyframeEffect::cancelAnimationOnCompositor wants to clear
m_compositorAnimationIds
then it shoudln't crash.

Also, note the comment in CompositorAnimations::cancelAnimationOnCompositor.

BUG=571504
R=dstockwell@chromium.org
==========

to

==========
Blink CompositorAnimations: Do not crash if no compositor animation player.

blink::Animation can lose compositorPlayer if Animation::m_content detached via
Animation::setEffect(nullptr)

If KeyframeEffect::cancelAnimationOnCompositor wants to clear
m_compositorAnimationIds
then it shoudln't crash.

Also, note the comment in CompositorAnimations::cancelAnimationOnCompositor.

BUG=571504
R=dstockwell@chromium.org
==========

[loyso (OOO), 2015-12-23 03:23:09]

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

[loyso (OOO), 2015-12-23 03:29:17]

Description was changed from

==========
Blink CompositorAnimations: Do not crash if no compositor animation player.

blink::Animation can lose compositorPlayer if Animation::m_content detached via
Animation::setEffect(nullptr)

If KeyframeEffect::cancelAnimationOnCompositor wants to clear
m_compositorAnimationIds
then it shoudln't crash.

Also, note the comment in CompositorAnimations::cancelAnimationOnCompositor.

BUG=571504
R=dstockwell@chromium.org
==========

to

==========
Blink CompositorAnimations: Do not crash if no compositor animation player.

blink::Animation can lose compositorPlayer if Animation::m_content detached via
Animation::setEffect(nullptr)

If KeyframeEffect::cancelAnimationOnCompositor wants to clear
m_compositorAnimationIds
then it shouldn't crash.

Also, note the comment in CompositorAnimations::cancelAnimationOnCompositor.

BUG=571504
R=dstockwell@chromium.org
==========

[loyso (OOO), 2015-12-23 03:51:02]

Description was changed from

==========
Blink CompositorAnimations: Do not crash if no compositor animation player.

blink::Animation can lose compositorPlayer if Animation::m_content detached via
Animation::setEffect(nullptr)

If KeyframeEffect::cancelAnimationOnCompositor wants to clear
m_compositorAnimationIds
then it shouldn't crash.

Also, note the comment in CompositorAnimations::cancelAnimationOnCompositor.

BUG=571504
R=dstockwell@chromium.org
==========

to

==========
Blink CompositorAnimations: Do not crash if no compositor animation player.

blink::Animation can lose compositorPlayer if Animation::m_content detached via
Animation::setEffect(nullptr)

If KeyframeEffect::cancelAnimationOnCompositor wants to clear
m_compositorAnimationIds
then it shouldn't crash.

Also, note the existing comment in
CompositorAnimations::cancelAnimationOnCompositor (above changed lines)

BUG=571504
R=dstockwell@chromium.org
==========

[loyso (OOO), 2015-12-23 03:52:13]

Description was changed from

==========
Blink CompositorAnimations: Do not crash if no compositor animation player.

blink::Animation can lose compositorPlayer if Animation::m_content detached via
Animation::setEffect(nullptr)

If KeyframeEffect::cancelAnimationOnCompositor wants to clear
m_compositorAnimationIds
then it shouldn't crash.

Also, note the existing comment in
CompositorAnimations::cancelAnimationOnCompositor (above changed lines)

BUG=571504
R=dstockwell@chromium.org
==========

to

==========
Blink CompositorAnimations: Do not crash if no compositor animation player.

blink::Animation can loose compositorPlayer if Animation::m_content detached via
Animation::setEffect(nullptr)

If KeyframeEffect::cancelAnimationOnCompositor wants to clear
m_compositorAnimationIds
then it shouldn't crash.

Also, note the existing comment in
CompositorAnimations::cancelAnimationOnCompositor (above changed lines)

BUG=571504
R=dstockwell@chromium.org
==========

[loyso (OOO), 2015-12-23 03:54:05]

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

[alancutter (OOO until 2018), 2015-12-23 04:48:43]

lgtm, please add a test case before landing.

[loyso (OOO), 2015-12-29 04:00:58]

On 2015/12/23 04:48:43, alancutter wrote:
> lgtm, please add a test case before landing.
See the test in dependent patchset. It requires a whole new subsystem.

[loyso (OOO), 2015-12-29 04:01:03]

The CQ bit was checked by loyso@chromium.org

[commit-bot: I haz the power, 2015-12-29 04:02:47]

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

[commit-bot: I haz the power, 2015-12-29 05:56:31]

Description was changed from

==========
Blink CompositorAnimations: Do not crash if no compositor animation player.

blink::Animation can loose compositorPlayer if Animation::m_content detached via
Animation::setEffect(nullptr)

If KeyframeEffect::cancelAnimationOnCompositor wants to clear
m_compositorAnimationIds
then it shouldn't crash.

Also, note the existing comment in
CompositorAnimations::cancelAnimationOnCompositor (above changed lines)

BUG=571504
R=dstockwell@chromium.org
==========

to

==========
Blink CompositorAnimations: Do not crash if no compositor animation player.

blink::Animation can loose compositorPlayer if Animation::m_content detached via
Animation::setEffect(nullptr)

If KeyframeEffect::cancelAnimationOnCompositor wants to clear
m_compositorAnimationIds
then it shouldn't crash.

Also, note the existing comment in
CompositorAnimations::cancelAnimationOnCompositor (above changed lines)

BUG=571504
R=dstockwell@chromium.org
==========

[commit-bot: I haz the power, 2015-12-29 05:56:33]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-29 05:57:31]

Description was changed from

==========
Blink CompositorAnimations: Do not crash if no compositor animation player.

blink::Animation can loose compositorPlayer if Animation::m_content detached via
Animation::setEffect(nullptr)

If KeyframeEffect::cancelAnimationOnCompositor wants to clear
m_compositorAnimationIds
then it shouldn't crash.

Also, note the existing comment in
CompositorAnimations::cancelAnimationOnCompositor (above changed lines)

BUG=571504
R=dstockwell@chromium.org
==========

to

==========
Blink CompositorAnimations: Do not crash if no compositor animation player.

blink::Animation can loose compositorPlayer if Animation::m_content detached via
Animation::setEffect(nullptr)

If KeyframeEffect::cancelAnimationOnCompositor wants to clear
m_compositorAnimationIds
then it shouldn't crash.

Also, note the existing comment in
CompositorAnimations::cancelAnimationOnCompositor (above changed lines)

BUG=571504
R=dstockwell@chromium.org

Committed: https://crrev.com/48abeaf3f8886a99e68722d55050547482863fb7
Cr-Commit-Position: refs/heads/master@{#367055}
==========

[commit-bot: I haz the power, 2015-12-29 05:57:35]

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