CC AnimationHost: Fix ProgrammaticScrollAnimator detach scope.

CC AnimationHost: Fix ProgrammaticScrollAnimator detach scope.

In a new world with external cc::AnimationHost
blink::WebCompositorAnimationPlayer objects may outsurvive the compositor
(LayerTreeView and AnimationHost).
In this case, they become detached.
(See RenderWidget::Close())

In general, any subsystem can react on willCloseLayerTreeView if some actions
required before detach.

See the related fix for LinkHighlights here:
https://codereview.chromium.org/1413373007/

BUG=541069
Committed: https://crrev.com/f59fdc1d0bc0eb35382f59f7e5f89a66bd0c325d
Cr-Commit-Position: refs/heads/master@{#357551}

[loyso (OOO), 2015-11-03 04:29:53]

loyso@chromium.org changed reviewers:
+ chrishtr@chromium.org, jochen@chromium.org, pdr@chromium.org,
vollick@chromium.org

[loyso (OOO), 2015-11-03 04:31:27]

Description was changed from

==========
CC AnimationHost: Fix ProgrammaticScrollAnimator detach scope.

In a new world with external cc::AnimationHost
blink::WebCompositorAnimationPlayer objects may outsurvive the compositor
(LayerTreeView).
In this case, they become detached.

In general, any subsystem can react on willCloseLayerTreeView if some actions
required before detach.

BUG=541069
==========

to

==========
CC AnimationHost: Fix ProgrammaticScrollAnimator detach scope.

In a new world with external cc::AnimationHost
blink::WebCompositorAnimationPlayer objects may outsurvive the compositor
(LayerTreeView).
In this case, they become detached.

In general, any subsystem can react on willCloseLayerTreeView if some actions
required before detach.

See the related fix for LinkHighlights here:
https://codereview.chromium.org/1413373007/

BUG=541069
==========

[loyso (OOO), 2015-11-03 04:33:23]

A LayoutTests OWNER needed for review. Any recommendations, please?

[loyso (OOO), 2015-11-03 04:43:29]

Description was changed from

==========
CC AnimationHost: Fix ProgrammaticScrollAnimator detach scope.

In a new world with external cc::AnimationHost
blink::WebCompositorAnimationPlayer objects may outsurvive the compositor
(LayerTreeView).
In this case, they become detached.

In general, any subsystem can react on willCloseLayerTreeView if some actions
required before detach.

See the related fix for LinkHighlights here:
https://codereview.chromium.org/1413373007/

BUG=541069
==========

to

==========
CC AnimationHost: Fix ProgrammaticScrollAnimator detach scope.

In a new world with external cc::AnimationHost
blink::WebCompositorAnimationPlayer objects may outsurvive the compositor
(LayerTreeView).
In this case, they become detached.
(See RenderWidget::Close())

In general, any subsystem can react on willCloseLayerTreeView if some actions
required before detach.

See the related fix for LinkHighlights here:
https://codereview.chromium.org/1413373007/

BUG=541069
==========

[loyso (OOO), 2015-11-03 05:36:00]

Description was changed from

==========
CC AnimationHost: Fix ProgrammaticScrollAnimator detach scope.

In a new world with external cc::AnimationHost
blink::WebCompositorAnimationPlayer objects may outsurvive the compositor
(LayerTreeView).
In this case, they become detached.
(See RenderWidget::Close())

In general, any subsystem can react on willCloseLayerTreeView if some actions
required before detach.

See the related fix for LinkHighlights here:
https://codereview.chromium.org/1413373007/

BUG=541069
==========

to

==========
CC AnimationHost: Fix ProgrammaticScrollAnimator detach scope.

In a new world with external cc::AnimationHost
blink::WebCompositorAnimationPlayer objects may outsurvive the compositor
(LayerTreeView and AnimationHost).
In this case, they become detached.
(See RenderWidget::Close())

In general, any subsystem can react on willCloseLayerTreeView if some actions
required before detach.

See the related fix for LinkHighlights here:
https://codereview.chromium.org/1413373007/

BUG=541069
==========

[jochen (gone - plz use gerrit), 2015-11-03 07:25:04]

layout tests are owned by everybody:
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit...

I'm not a good reviewer for the actual change here.

[chrishtr, 2015-11-03 16:18:08]

The CQ bit was checked by chrishtr@chromium.org

[chrishtr, 2015-11-03 16:18:09]

lgtm

[commit-bot: I haz the power, 2015-11-03 16:19:50]

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

[commit-bot: I haz the power, 2015-11-03 18:16:38]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-11-03 18:18:07]

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