Revert of Run smooth scroll animations on the compositor when possible

Revert of Run smooth scroll animations on the compositor when possible (patchset #4 id:60001 of https://codereview.chromium.org/1534813004/ )

Reason for revert:
Appears to have made Linux debug builds fail the following tests:

virtual/threaded/fast/scroll-behavior/overflow-scroll-root-frame-animates.html
virtual/threaded/fast/scroll-behavior/overflow-scroll-animates.html

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20%28dbg%29/builds/5360

Reverting to green the Blink tree.

Original issue's description:
> Run smooth scroll animations on the compositor when possible
>
> This CL implements "immediate retargeting" shown in the following diagram:
> https://docs.google.com/drawings/d/1z76m-GMDgLI-clkfroLJ-5yhsHkmriZr_swP0kn5iw4
>
> This CL does the following
> - Add a new state to ScrollAnimatorCompositorCoordinator to update the target
> offset on the compositor
> - Use the ScrollAnimatorCompositorCoordinator to schedule anim
> - Add plumbing from the animators to the animation controller to abort
> animations
>
> Sample test pages:
> yashmalik.com/nested_scroll_mousewheel_listener.html
> yashmalik.com/nested_scroll_mousewheel_listener_15ms_janky.html (tight loop in RAF)
>
> BUG=552556
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
>
> Committed: https://crrev.com/5534a2b2972534b523014c803e1e0e0541855cc7
> Cr-Commit-Position: refs/heads/master@{#366685}

TBR=skobes@chromium.org,ajuma@chromium.org,vollick@chromium.org,rbyers@chromium.org,ymalik@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=552556
Committed: https://crrev.com/51059d9d038aa255afdddfcbd3d418eb42b718ff
Cr-Commit-Position: refs/heads/master@{#366733}

[Noel Gordon, 2015-12-23 09:13:50]

Created Revert of Run smooth scroll animations on the compositor when possible

[commit-bot: I haz the power, 2015-12-23 09:14:03]

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

[Noel Gordon, 2015-12-23 09:14:10]

Description was changed from

==========
Revert of Run smooth scroll animations on the compositor when possible (patchset
#4 id:60001 of https://codereview.chromium.org/1534813004/ )

Reason for revert:
Appears to have made Linux debug builds fail the following tests:

virtual/threaded/fast/scroll-behavior/overflow-scroll-root-frame-animates.html
virtual/threaded/fast/scroll-behavior/overflow-scroll-animates.html

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20%28dbg...

Reverting to green the Blink tree.




Original issue's description:
> Run smooth scroll animations on the compositor when possible
> 
> This CL implements "immediate retargeting" shown in the following diagram:
>
https://docs.google.com/drawings/d/1z76m-GMDgLI-clkfroLJ-5yhsHkmriZr_swP0kn5iw4
> 
> This CL does the following
> - Add a new state to ScrollAnimatorCompositorCoordinator to update the target
> offset on the compositor
> - Use the ScrollAnimatorCompositorCoordinator to schedule anim
> - Add plumbing from the animators to the animation controller to abort
> animations
> 
> Sample test pages:
> yashmalik.com/nested_scroll_mousewheel_listener.html
> yashmalik.com/nested_scroll_mousewheel_listener_15ms_janky.html (tight loop in
RAF)
> 
> BUG=552556
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
> 
> Committed: https://crrev.com/5534a2b2972534b523014c803e1e0e0541855cc7
> Cr-Commit-Position: refs/heads/master@{#366685}

TBR=skobes@chromium.org,ajuma@chromium.org,vollick@chromium.org,rbyers@chromi...
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=552556
==========

to

==========
Revert of Run smooth scroll animations on the compositor when possible (patchset
#4 id:60001 of https://codereview.chromium.org/1534813004/ )

Reason for revert:
Appears to have made Linux debug builds fail the following tests:

virtual/threaded/fast/scroll-behavior/overflow-scroll-root-frame-animates.html
virtual/threaded/fast/scroll-behavior/overflow-scroll-animates.html

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20%28dbg...

Reverting to green the Blink tree.

Original issue's description:
> Run smooth scroll animations on the compositor when possible
> 
> This CL implements "immediate retargeting" shown in the following diagram:
>
https://docs.google.com/drawings/d/1z76m-GMDgLI-clkfroLJ-5yhsHkmriZr_swP0kn5iw4
> 
> This CL does the following
> - Add a new state to ScrollAnimatorCompositorCoordinator to update the target
> offset on the compositor
> - Use the ScrollAnimatorCompositorCoordinator to schedule anim
> - Add plumbing from the animators to the animation controller to abort
> animations
> 
> Sample test pages:
> yashmalik.com/nested_scroll_mousewheel_listener.html
> yashmalik.com/nested_scroll_mousewheel_listener_15ms_janky.html (tight loop in
RAF)
> 
> BUG=552556
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
> 
> Committed: https://crrev.com/5534a2b2972534b523014c803e1e0e0541855cc7
> Cr-Commit-Position: refs/heads/master@{#366685}

TBR=skobes@chromium.org,ajuma@chromium.org,vollick@chromium.org,rbyers@chromi...
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=552556
==========

[Noel Gordon, 2015-12-23 09:14:41]

Description was changed from

==========
Revert of Run smooth scroll animations on the compositor when possible (patchset
#4 id:60001 of https://codereview.chromium.org/1534813004/ )

Reason for revert:
Appears to have made Linux debug builds fail the following tests:

virtual/threaded/fast/scroll-behavior/overflow-scroll-root-frame-animates.html
virtual/threaded/fast/scroll-behavior/overflow-scroll-animates.html

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20%28dbg...

Reverting to green the Blink tree.

Original issue's description:
> Run smooth scroll animations on the compositor when possible
> 
> This CL implements "immediate retargeting" shown in the following diagram:
>
https://docs.google.com/drawings/d/1z76m-GMDgLI-clkfroLJ-5yhsHkmriZr_swP0kn5iw4
> 
> This CL does the following
> - Add a new state to ScrollAnimatorCompositorCoordinator to update the target
> offset on the compositor
> - Use the ScrollAnimatorCompositorCoordinator to schedule anim
> - Add plumbing from the animators to the animation controller to abort
> animations
> 
> Sample test pages:
> yashmalik.com/nested_scroll_mousewheel_listener.html
> yashmalik.com/nested_scroll_mousewheel_listener_15ms_janky.html (tight loop in
RAF)
> 
> BUG=552556
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
> 
> Committed: https://crrev.com/5534a2b2972534b523014c803e1e0e0541855cc7
> Cr-Commit-Position: refs/heads/master@{#366685}

TBR=skobes@chromium.org,ajuma@chromium.org,vollick@chromium.org,rbyers@chromi...
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=552556
==========

to

==========
Revert of Run smooth scroll animations on the compositor when possible (patchset
#4 id:60001 of https://codereview.chromium.org/1534813004/ )

Reason for revert:
Appears to have made Linux debug builds fail the following tests:

virtual/threaded/fast/scroll-behavior/overflow-scroll-root-frame-animates.html
virtual/threaded/fast/scroll-behavior/overflow-scroll-animates.html

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20%28dbg...

Original issue's description:
> Run smooth scroll animations on the compositor when possible
> 
> This CL implements "immediate retargeting" shown in the following diagram:
>
https://docs.google.com/drawings/d/1z76m-GMDgLI-clkfroLJ-5yhsHkmriZr_swP0kn5iw4
> 
> This CL does the following
> - Add a new state to ScrollAnimatorCompositorCoordinator to update the target
> offset on the compositor
> - Use the ScrollAnimatorCompositorCoordinator to schedule anim
> - Add plumbing from the animators to the animation controller to abort
> animations
> 
> Sample test pages:
> yashmalik.com/nested_scroll_mousewheel_listener.html
> yashmalik.com/nested_scroll_mousewheel_listener_15ms_janky.html (tight loop in
RAF)
> 
> BUG=552556
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
> 
> Committed: https://crrev.com/5534a2b2972534b523014c803e1e0e0541855cc7
> Cr-Commit-Position: refs/heads/master@{#366685}

TBR=skobes@chromium.org,ajuma@chromium.org,vollick@chromium.org,rbyers@chromi...
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=552556
==========

[commit-bot: I haz the power, 2015-12-23 09:14:51]

Description was changed from

==========
Revert of Run smooth scroll animations on the compositor when possible (patchset
#4 id:60001 of https://codereview.chromium.org/1534813004/ )

Reason for revert:
Appears to have made Linux debug builds fail the following tests:

virtual/threaded/fast/scroll-behavior/overflow-scroll-root-frame-animates.html
virtual/threaded/fast/scroll-behavior/overflow-scroll-animates.html

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20%28dbg...

Original issue's description:
> Run smooth scroll animations on the compositor when possible
> 
> This CL implements "immediate retargeting" shown in the following diagram:
>
https://docs.google.com/drawings/d/1z76m-GMDgLI-clkfroLJ-5yhsHkmriZr_swP0kn5iw4
> 
> This CL does the following
> - Add a new state to ScrollAnimatorCompositorCoordinator to update the target
> offset on the compositor
> - Use the ScrollAnimatorCompositorCoordinator to schedule anim
> - Add plumbing from the animators to the animation controller to abort
> animations
> 
> Sample test pages:
> yashmalik.com/nested_scroll_mousewheel_listener.html
> yashmalik.com/nested_scroll_mousewheel_listener_15ms_janky.html (tight loop in
RAF)
> 
> BUG=552556
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
> 
> Committed: https://crrev.com/5534a2b2972534b523014c803e1e0e0541855cc7
> Cr-Commit-Position: refs/heads/master@{#366685}

TBR=skobes@chromium.org,ajuma@chromium.org,vollick@chromium.org,rbyers@chromi...
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=552556
==========

to

==========
Revert of Run smooth scroll animations on the compositor when possible (patchset
#4 id:60001 of https://codereview.chromium.org/1534813004/ )

Reason for revert:
Appears to have made Linux debug builds fail the following tests:

virtual/threaded/fast/scroll-behavior/overflow-scroll-root-frame-animates.html
virtual/threaded/fast/scroll-behavior/overflow-scroll-animates.html

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20%28dbg...

Reverting to green the Blink tree.

Original issue's description:
> Run smooth scroll animations on the compositor when possible
> 
> This CL implements "immediate retargeting" shown in the following diagram:
>
https://docs.google.com/drawings/d/1z76m-GMDgLI-clkfroLJ-5yhsHkmriZr_swP0kn5iw4
> 
> This CL does the following
> - Add a new state to ScrollAnimatorCompositorCoordinator to update the target
> offset on the compositor
> - Use the ScrollAnimatorCompositorCoordinator to schedule anim
> - Add plumbing from the animators to the animation controller to abort
> animations
> 
> Sample test pages:
> yashmalik.com/nested_scroll_mousewheel_listener.html
> yashmalik.com/nested_scroll_mousewheel_listener_15ms_janky.html (tight loop in
RAF)
> 
> BUG=552556
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
> 
> Committed: https://crrev.com/5534a2b2972534b523014c803e1e0e0541855cc7
> Cr-Commit-Position: refs/heads/master@{#366685}

TBR=skobes@chromium.org,ajuma@chromium.org,vollick@chromium.org,rbyers@chromi...
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=552556
==========

[commit-bot: I haz the power, 2015-12-23 09:14:52]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-23 09:15:43]

Description was changed from

==========
Revert of Run smooth scroll animations on the compositor when possible (patchset
#4 id:60001 of https://codereview.chromium.org/1534813004/ )

Reason for revert:
Appears to have made Linux debug builds fail the following tests:

virtual/threaded/fast/scroll-behavior/overflow-scroll-root-frame-animates.html
virtual/threaded/fast/scroll-behavior/overflow-scroll-animates.html

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20%28dbg...

Reverting to green the Blink tree.

Original issue's description:
> Run smooth scroll animations on the compositor when possible
> 
> This CL implements "immediate retargeting" shown in the following diagram:
>
https://docs.google.com/drawings/d/1z76m-GMDgLI-clkfroLJ-5yhsHkmriZr_swP0kn5iw4
> 
> This CL does the following
> - Add a new state to ScrollAnimatorCompositorCoordinator to update the target
> offset on the compositor
> - Use the ScrollAnimatorCompositorCoordinator to schedule anim
> - Add plumbing from the animators to the animation controller to abort
> animations
> 
> Sample test pages:
> yashmalik.com/nested_scroll_mousewheel_listener.html
> yashmalik.com/nested_scroll_mousewheel_listener_15ms_janky.html (tight loop in
RAF)
> 
> BUG=552556
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
> 
> Committed: https://crrev.com/5534a2b2972534b523014c803e1e0e0541855cc7
> Cr-Commit-Position: refs/heads/master@{#366685}

TBR=skobes@chromium.org,ajuma@chromium.org,vollick@chromium.org,rbyers@chromi...
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=552556
==========

to

==========
Revert of Run smooth scroll animations on the compositor when possible (patchset
#4 id:60001 of https://codereview.chromium.org/1534813004/ )

Reason for revert:
Appears to have made Linux debug builds fail the following tests:

virtual/threaded/fast/scroll-behavior/overflow-scroll-root-frame-animates.html
virtual/threaded/fast/scroll-behavior/overflow-scroll-animates.html

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20%28dbg...

Reverting to green the Blink tree.

Original issue's description:
> Run smooth scroll animations on the compositor when possible
>
> This CL implements "immediate retargeting" shown in the following diagram:
>
https://docs.google.com/drawings/d/1z76m-GMDgLI-clkfroLJ-5yhsHkmriZr_swP0kn5iw4
>
> This CL does the following
> - Add a new state to ScrollAnimatorCompositorCoordinator to update the target
> offset on the compositor
> - Use the ScrollAnimatorCompositorCoordinator to schedule anim
> - Add plumbing from the animators to the animation controller to abort
> animations
>
> Sample test pages:
> yashmalik.com/nested_scroll_mousewheel_listener.html
> yashmalik.com/nested_scroll_mousewheel_listener_15ms_janky.html (tight loop in
RAF)
>
> BUG=552556
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
>
> Committed: https://crrev.com/5534a2b2972534b523014c803e1e0e0541855cc7
> Cr-Commit-Position: refs/heads/master@{#366685}

TBR=skobes@chromium.org,ajuma@chromium.org,vollick@chromium.org,rbyers@chromi...
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=552556

Committed: https://crrev.com/51059d9d038aa255afdddfcbd3d418eb42b718ff
Cr-Commit-Position: refs/heads/master@{#366733}
==========

[commit-bot: I haz the power, 2015-12-23 09:15:44]

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

[Noel Gordon, 2015-12-23 11:35:20]

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20%28dbg...
the revert did not help.  Test is just a lot slower and appears to be hitting
timeout limits.

[Noel Gordon, 2015-12-23 11:36:52]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1543043004/ by noel@chromium.org.

The reason for reverting is: The revert did not help, the tests

virtual/threaded/fast/scroll-behavior/overflow-scroll-root-frame-animates.html
virtual/threaded/fast/scroll-behavior/overflow-scroll-animates.html

are now slower and hitting test timeout limits.

.