Revert of Implement Element.scrollIntoView for scroll-behavior: smooth.

Revert of Implement Element.scrollIntoView for scroll-behavior: smooth. (patchset #19 id:550001 of https://codereview.chromium.org/2650343008/ )

Reason for revert:
Looks this CL made the "WebKit Linux Trusty Leak" bot red.

See https://codereview.chromium.org/2650343008/#msg131

Original issue's description:
> Implement Element.scrollIntoView for scroll-behavior: smooth.
> 
> Currently in Chrome, a call to Element.scrollIntoView will calculate the amount
> each scroller needs to scroll to align the Element as specified and instantly
> set the scroll position on that scroller (going from the innermost to the
> outermost scroller). 
> 
> If Element.scrollIntoView is called with scroll-behavior: smooth, instead of
> scrolling instantly, we should animate to the desired position.
> 
> We can’t simply call setScrollPosition in PaintLayerScrollableArea and
> FrameView with ScrollBehaviorSmooth because of the following reasons:
> - We want to run the animation from the outermost scroller to the
>   innermost scroller
> - We want to run the animation one after another. That is, the scroll animation
>   on the child scroller should start after the animation on the parent scroller
>   is finished.
> 
> This patch adds a new class, ProgrammaticScrollCoordinator, that manages
> programmatic scroll animations.
> 
> BUG=648446
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> 
> Review-Url: https://codereview.chromium.org/2650343008
> Cr-Commit-Position: refs/heads/master@{#475387}
> Committed: https://chromium.googlesource.com/chromium/src/+/bbf870d971d95af7ae1ee688a7ed100e3787d02b

TBR=jbroman@chromium.org,bokan@chromium.org,ikilpatrick@chromium.org,sigbjornf@opera.com,sunyunjia@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=648446
Review-Url: https://codereview.chromium.org/2911103002
Cr-Commit-Position: refs/heads/master@{#475402}
Committed: https://chromium.googlesource.com/chromium/src/+/50ad23a34ff5e7d2a041c98991ef067209a8f41f

[tyoshino (SeeGerritForStatus), 2017-05-30 02:08:50]

The CQ bit was checked by tyoshino@chromium.org

[tyoshino (SeeGerritForStatus), 2017-05-30 02:08:50]

Created Revert of Implement Element.scrollIntoView for scroll-behavior: smooth.

[commit-bot: I haz the power, 2017-05-30 02:09:04]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[tyoshino (SeeGerritForStatus), 2017-05-30 02:09:39]

http://crbug.com/727446 is the bug for this.

[commit-bot: I haz the power, 2017-05-30 02:09:39]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1496110130068650, "parent_rev":
"d560ed865c053d87594aca8a72637593117d5d1b", "commit_rev":
"50ad23a34ff5e7d2a041c98991ef067209a8f41f"}

[commit-bot: I haz the power, 2017-05-30 02:09:54]

Description was changed from

==========
Revert of Implement Element.scrollIntoView for scroll-behavior: smooth.
(patchset #19 id:550001 of https://codereview.chromium.org/2650343008/ )

Reason for revert:
Looks this CL made the "WebKit Linux Trusty Leak" bot red.

See https://codereview.chromium.org/2650343008/#msg131

Original issue's description:
> Implement Element.scrollIntoView for scroll-behavior: smooth.
> 
> Currently in Chrome, a call to Element.scrollIntoView will calculate the
amount
> each scroller needs to scroll to align the Element as specified and instantly
> set the scroll position on that scroller (going from the innermost to the
> outermost scroller). 
> 
> If Element.scrollIntoView is called with scroll-behavior: smooth, instead of
> scrolling instantly, we should animate to the desired position.
> 
> We can’t simply call setScrollPosition in PaintLayerScrollableArea and
> FrameView with ScrollBehaviorSmooth because of the following reasons:
> - We want to run the animation from the outermost scroller to the
>   innermost scroller
> - We want to run the animation one after another. That is, the scroll
animation
>   on the child scroller should start after the animation on the parent
scroller
>   is finished.
> 
> This patch adds a new class, ProgrammaticScrollCoordinator, that manages
> programmatic scroll animations.
> 
> BUG=648446
>
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> 
> Review-Url: https://codereview.chromium.org/2650343008
> Cr-Commit-Position: refs/heads/master@{#475387}
> Committed:
https://chromium.googlesource.com/chromium/src/+/bbf870d971d95af7ae1ee688a7ed...

TBR=jbroman@chromium.org,bokan@chromium.org,ikilpatrick@chromium.org,sigbjorn...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=648446
==========

to

==========
Revert of Implement Element.scrollIntoView for scroll-behavior: smooth.
(patchset #19 id:550001 of https://codereview.chromium.org/2650343008/ )

Reason for revert:
Looks this CL made the "WebKit Linux Trusty Leak" bot red.

See https://codereview.chromium.org/2650343008/#msg131

Original issue's description:
> Implement Element.scrollIntoView for scroll-behavior: smooth.
> 
> Currently in Chrome, a call to Element.scrollIntoView will calculate the
amount
> each scroller needs to scroll to align the Element as specified and instantly
> set the scroll position on that scroller (going from the innermost to the
> outermost scroller). 
> 
> If Element.scrollIntoView is called with scroll-behavior: smooth, instead of
> scrolling instantly, we should animate to the desired position.
> 
> We can’t simply call setScrollPosition in PaintLayerScrollableArea and
> FrameView with ScrollBehaviorSmooth because of the following reasons:
> - We want to run the animation from the outermost scroller to the
>   innermost scroller
> - We want to run the animation one after another. That is, the scroll
animation
>   on the child scroller should start after the animation on the parent
scroller
>   is finished.
> 
> This patch adds a new class, ProgrammaticScrollCoordinator, that manages
> programmatic scroll animations.
> 
> BUG=648446
>
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> 
> Review-Url: https://codereview.chromium.org/2650343008
> Cr-Commit-Position: refs/heads/master@{#475387}
> Committed:
https://chromium.googlesource.com/chromium/src/+/bbf870d971d95af7ae1ee688a7ed...

TBR=jbroman@chromium.org,bokan@chromium.org,ikilpatrick@chromium.org,sigbjorn...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=648446

Review-Url: https://codereview.chromium.org/2911103002
Cr-Commit-Position: refs/heads/master@{#475402}
Committed:
https://chromium.googlesource.com/chromium/src/+/50ad23a34ff5e7d2a041c98991ef...
==========

[commit-bot: I haz the power, 2017-05-30 02:09:55]

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