Remove "reportWheelOverscroll" from Blink settings.

Remove "reportWheelOverscroll" from Blink settings.

This flag complicates Blink event handling needlessly. It's only needed on Mac
where we use it to disable the elastic rubber banding effect on overscroll.

This patch removes the setting and always reports overscroll for wheel from
Blink. In the renderer content layer, we check if the ElasticOverscroll system
setting is enabled on Mac and block the overscroll if not (there's no touch
overscroll to worry about on Mac). Otherwise just allow the overscroll deltas to
be sent to the browser since they have no effect on other platforms.

BUG=505516
Committed: https://crrev.com/731ec38829ae017204d43337da90ea7ca2f15e51
Cr-Commit-Position: refs/heads/master@{#385642}

[bokan, 2016-04-06 14:13:24]

The CQ bit was checked by bokan@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-04-06 14:13:36]

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

[commit-bot: I haz the power, 2016-04-06 14:27:22]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-06 14:27:23]

Dry run: Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[bokan, 2016-04-06 14:29:44]

Patchset #2 (id:20001) has been deleted

[bokan, 2016-04-06 14:30:11]

The CQ bit was checked by bokan@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-04-06 14:30:37]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1861343002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1861343002/40001

[bokan, 2016-04-06 15:21:14]

Description was changed from

==========
Remove "reportWheelOverscroll" from Blink settings.

This flag complicates Blink event handling needlessly. It's only needed on Mac
where we use it to disable the elastic rubber banding effect on overscroll.

This patch removes the setting and always reports overscroll for wheel from
Blink. In the renderer content layer, we check if the ElasticOverscroll system
setting is enabled on Mac and block the overscroll if not (there's no touch
overscroll to worry about on Mac). Otherwise just allow the overscroll deltas to
be sent to the browser since they have no effect on other platforms.

BUG=505516
==========

to

==========
Remove "reportWheelOverscroll" from Blink settings.

This flag complicates Blink event handling needlessly. It's only needed on Mac
where we use it to disable the elastic rubber banding effect on overscroll.

This patch removes the setting and always reports overscroll for wheel from
Blink. In the renderer content layer, we check if the ElasticOverscroll system
setting is enabled on Mac and block the overscroll if not (there's no touch
overscroll to worry about on Mac). Otherwise just allow the overscroll deltas to
be sent to the browser since they have no effect on other platforms.

BUG=505516
==========

[bokan, 2016-04-06 15:21:14]

bokan@chromium.org changed reviewers:
+ dtapuska@chromium.org

[dtapuska, 2016-04-06 15:29:20]

On 2016/04/06 15:21:28, bokan wrote:

lgtm

[commit-bot: I haz the power, 2016-04-06 16:02:44]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-06 16:02:45]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[bokan, 2016-04-06 23:48:31]

bokan@chromium.org changed reviewers:
+ avi@chromium.org, rbyers@chromium.org

[bokan, 2016-04-06 23:48:31]

+rbyers@ for WebKit/public
+avi@ for content/

[Rick Byers, 2016-04-06 23:50:03]

public LGTM

[Avi (use Gerrit), 2016-04-07 01:51:21]

lgtm

[bokan, 2016-04-07 01:51:49]

The CQ bit was checked by bokan@chromium.org

[bokan, 2016-04-07 01:51:50]

The patchset sent to the CQ was uploaded after l-g-t-m from
dtapuska@chromium.org
Link to the patchset: https://codereview.chromium.org/1861343002/#ps60001
(title: "Fixed unit tests and moved check to RenderWidget")

[commit-bot: I haz the power, 2016-04-07 01:52:51]

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

[commit-bot: I haz the power, 2016-04-07 03:16:56]

Description was changed from

==========
Remove "reportWheelOverscroll" from Blink settings.

This flag complicates Blink event handling needlessly. It's only needed on Mac
where we use it to disable the elastic rubber banding effect on overscroll.

This patch removes the setting and always reports overscroll for wheel from
Blink. In the renderer content layer, we check if the ElasticOverscroll system
setting is enabled on Mac and block the overscroll if not (there's no touch
overscroll to worry about on Mac). Otherwise just allow the overscroll deltas to
be sent to the browser since they have no effect on other platforms.

BUG=505516
==========

to

==========
Remove "reportWheelOverscroll" from Blink settings.

This flag complicates Blink event handling needlessly. It's only needed on Mac
where we use it to disable the elastic rubber banding effect on overscroll.

This patch removes the setting and always reports overscroll for wheel from
Blink. In the renderer content layer, we check if the ElasticOverscroll system
setting is enabled on Mac and block the overscroll if not (there's no touch
overscroll to worry about on Mac). Otherwise just allow the overscroll deltas to
be sent to the browser since they have no effect on other platforms.

BUG=505516
==========

[commit-bot: I haz the power, 2016-04-07 03:16:57]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2016-04-07 03:20:04]

Description was changed from

==========
Remove "reportWheelOverscroll" from Blink settings.

This flag complicates Blink event handling needlessly. It's only needed on Mac
where we use it to disable the elastic rubber banding effect on overscroll.

This patch removes the setting and always reports overscroll for wheel from
Blink. In the renderer content layer, we check if the ElasticOverscroll system
setting is enabled on Mac and block the overscroll if not (there's no touch
overscroll to worry about on Mac). Otherwise just allow the overscroll deltas to
be sent to the browser since they have no effect on other platforms.

BUG=505516
==========

to

==========
Remove "reportWheelOverscroll" from Blink settings.

This flag complicates Blink event handling needlessly. It's only needed on Mac
where we use it to disable the elastic rubber banding effect on overscroll.

This patch removes the setting and always reports overscroll for wheel from
Blink. In the renderer content layer, we check if the ElasticOverscroll system
setting is enabled on Mac and block the overscroll if not (there's no touch
overscroll to worry about on Mac). Otherwise just allow the overscroll deltas to
be sent to the browser since they have no effect on other platforms.

BUG=505516

Committed: https://crrev.com/731ec38829ae017204d43337da90ea7ca2f15e51
Cr-Commit-Position: refs/heads/master@{#385642}
==========

[commit-bot: I haz the power, 2016-04-07 03:20:05]

Patchset 3 (id:??) landed as
https://crrev.com/731ec38829ae017204d43337da90ea7ca2f15e51
Cr-Commit-Position: refs/heads/master@{#385642}