Merge the patch back into M57

Remove logic about recording style related main thread scroll reasons

The current recording incorrectly propagates the style related main thread scrolling reasons
of a scrollable overflow element to the main frame. It would make the main frame scroll on main if there were any
scrollers which needed to scroll on main which it shouldn't. It also caused 34.5% regression in
smoothness.key_mobile_sites_smooth benchmark (crbug.com/693527) .As the issue is blocking
release of M57 stable, it's better to remove the recording logic for now and work
on a correct one in a following patch.

BUG=701355
TBR=pdr@chromium.org
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2766893002
Cr-Commit-Position: refs/heads/master@{#458596}
(cherry picked from commit d528436cf0ab46778ccd56a439a250e41113fc46)

Review-Url: https://codereview.chromium.org/2766353002
Cr-Commit-Position: refs/branch-heads/2987@{#864}
Cr-Branched-From: ad51088c0e8776e8dcd963dbe752c4035ba6dab6-refs/heads/master@{#444943}
Committed: https://chromium.googlesource.com/chromium/src/+/c8352fb469276b9d2d1bcef3fc93f202aa5c2b75

[yigu, 2017-03-22 18:55:36]

Description was changed from

==========
Merge patch back into M57
==========

to

==========
Remove logic about recording style related main thread scroll reasons

The current recording incorrectly propagates the style related main thread
scrolling reasons
of a scrollable overflow element to the main frame. It would make the main frame
scroll on main if there were any
scrollers which needed to scroll on main which it shouldn't. It also caused
34.5% regression in
smoothness.key_mobile_sites_smooth benchmark (crbug.com/693527) .As the issue is
blocking
release of M57 stable, it's better to remove the recording logic for now and
work
on a correct one in a following patch.

BUG=701355
TBR=pdr@chromium.org
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2766893002
Cr-Commit-Position: refs/heads/master@{#458596}
(cherry picked from commit d528436cf0ab46778ccd56a439a250e41113fc46)
==========

[yigu, 2017-03-22 18:55:36]

yigu@chromium.org changed reviewers:
+ flackr@chromium.org, pdr@chromium.org

[yigu, 2017-03-22 18:56:47]

PTAL. Thanks!

[flackr, 2017-03-22 18:57:27]

lgtm

[yigu, 2017-03-22 18:57:42]

The CQ bit was checked by yigu@chromium.org

[commit-bot: I haz the power, 2017-03-22 18:58:40]

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-03-22 19:07:48]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490209062541420, "parent_rev":
"b02ab26e3de4d0faee3443ae34cf798b5748fe32", "commit_rev":
"c8352fb469276b9d2d1bcef3fc93f202aa5c2b75"}

[commit-bot: I haz the power, 2017-03-22 19:08:39]

Description was changed from

==========
Remove logic about recording style related main thread scroll reasons

The current recording incorrectly propagates the style related main thread
scrolling reasons
of a scrollable overflow element to the main frame. It would make the main frame
scroll on main if there were any
scrollers which needed to scroll on main which it shouldn't. It also caused
34.5% regression in
smoothness.key_mobile_sites_smooth benchmark (crbug.com/693527) .As the issue is
blocking
release of M57 stable, it's better to remove the recording logic for now and
work
on a correct one in a following patch.

BUG=701355
TBR=pdr@chromium.org
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2766893002
Cr-Commit-Position: refs/heads/master@{#458596}
(cherry picked from commit d528436cf0ab46778ccd56a439a250e41113fc46)
==========

to

==========
Remove logic about recording style related main thread scroll reasons

The current recording incorrectly propagates the style related main thread
scrolling reasons
of a scrollable overflow element to the main frame. It would make the main frame
scroll on main if there were any
scrollers which needed to scroll on main which it shouldn't. It also caused
34.5% regression in
smoothness.key_mobile_sites_smooth benchmark (crbug.com/693527) .As the issue is
blocking
release of M57 stable, it's better to remove the recording logic for now and
work
on a correct one in a following patch.

BUG=701355
TBR=pdr@chromium.org
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2766893002
Cr-Commit-Position: refs/heads/master@{#458596}
(cherry picked from commit d528436cf0ab46778ccd56a439a250e41113fc46)

Review-Url: https://codereview.chromium.org/2766353002
Cr-Commit-Position: refs/branch-heads/2987@{#864}
Cr-Branched-From:
ad51088c0e8776e8dcd963dbe752c4035ba6dab6-refs/heads/master@{#444943}
Committed:
https://chromium.googlesource.com/chromium/src/+/c8352fb469276b9d2d1bcef3fc93...
==========

[commit-bot: I haz the power, 2017-03-22 19:08:39]

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