Mark sub-frame root layer non-fast scrollable when wheel handlers present.

Mark sub-frame root layer non-fast scrollable when wheel handlers present.

Just like with touch event handlers, at present a sub-frame's
compositor does not receive regions rects for non-fast scrollable
regions that accompany MouseWheel handlers. Until ScrollingCoordinator
is made per-frame, we should mark the entire root layer rect as
non-fast scrollable in the presence of blocking wheel handlers.

The primary changes are all in render_widget_compositor.cc; other
files are touched only to give the function a more inclusive name.

BUG=698195
Review-Url: https://codereview.chromium.org/2740133002
Cr-Commit-Position: refs/heads/master@{#456131}
Committed: https://chromium.googlesource.com/chromium/src/+/47b1b819e0e239b1c753ae4e736f77ac9ccafcdd

[wjmaclean, 2017-03-09 15:21:32]

Description was changed from

==========
Mark sub-frame root layer non-fast scrollable when wheel handlers present.

Just like with touch event handlers, at present a sub-frame's
compositor does not receive regions rects for non-fast scrollable
regions that accompany MouseWheel handlers. Until ScrollingCoordinator
is made per-frame, we should mark the entire root layer rect as
non-fast scrollable in the presence of blocking wheel handlers.

BUG=698195
==========

to

==========
Mark sub-frame root layer non-fast scrollable when wheel handlers present.

Just like with touch event handlers, at present a sub-frame's
compositor does not receive regions rects for non-fast scrollable
regions that accompany MouseWheel handlers. Until ScrollingCoordinator
is made per-frame, we should mark the entire root layer rect as
non-fast scrollable in the presence of blocking wheel handlers.

The primary changes are all in render_widget_compositor.cc; other
files are touched only to give the function a more inclusive name.

BUG=698195
==========

[wjmaclean, 2017-03-09 15:21:50]

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

[commit-bot: I haz the power, 2017-03-09 15:22:00]

Dry run: 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-09 17:17:03]

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

[commit-bot: I haz the power, 2017-03-09 17:17:04]

Dry run: This issue passed the CQ dry run.

[wjmaclean, 2017-03-10 01:48:21]

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

[commit-bot: I haz the power, 2017-03-10 01:48:44]

Dry run: 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-10 02:15:17]

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

[commit-bot: I haz the power, 2017-03-10 02:15:18]

Dry run: Try jobs failed on following builders:
  linux_chromium_tsan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[wjmaclean, 2017-03-10 12:58:02]

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

[commit-bot: I haz the power, 2017-03-10 12:58:17]

Dry run: 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-10 14:49:05]

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

[commit-bot: I haz the power, 2017-03-10 14:49:06]

Dry run: This issue passed the CQ dry run.

[wjmaclean, 2017-03-10 16:19:56]

wjmaclean@chromium.org changed reviewers:
+ avi@chromium.org

[wjmaclean, 2017-03-10 16:20:32]

avi@ - This is a relatively straightforward CL (except perhaps the test) ... do
you have time to review?

[wjmaclean, 2017-03-10 16:38:27]

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

[commit-bot: I haz the power, 2017-03-10 16:38:53]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Avi (use Gerrit), 2017-03-10 16:40:54]

lgtm

[commit-bot: I haz the power, 2017-03-10 18:38:43]

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

[commit-bot: I haz the power, 2017-03-10 18:38:44]

Dry run: This issue passed the CQ dry run.

[wjmaclean, 2017-03-10 19:06:24]

The CQ bit was checked by wjmaclean@chromium.org

[commit-bot: I haz the power, 2017-03-10 19:07:17]

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-10 19:14:14]

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

[commit-bot: I haz the power, 2017-03-10 19:14:15]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[haraken, 2017-03-10 19:15:55]

WebKit owner LGTM

[wjmaclean, 2017-03-10 19:17:44]

On 2017/03/10 19:15:55, haraken wrote:
> WebKit owner LGTM

Thanks haraken@ !

[wjmaclean, 2017-03-10 19:17:53]

The CQ bit was checked by wjmaclean@chromium.org

[commit-bot: I haz the power, 2017-03-10 19:18:50]

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-10 19:24:28]

CQ is committing da patch.

Bot data: {"patchset_id": 50001, "attempt_start_ts": 1489173473103580,
"parent_rev": "9f86464794416e81abfe5921e9fac892716c610b", "commit_rev":
"47b1b819e0e239b1c753ae4e736f77ac9ccafcdd"}

[commit-bot: I haz the power, 2017-03-10 19:25:19]

Description was changed from

==========
Mark sub-frame root layer non-fast scrollable when wheel handlers present.

Just like with touch event handlers, at present a sub-frame's
compositor does not receive regions rects for non-fast scrollable
regions that accompany MouseWheel handlers. Until ScrollingCoordinator
is made per-frame, we should mark the entire root layer rect as
non-fast scrollable in the presence of blocking wheel handlers.

The primary changes are all in render_widget_compositor.cc; other
files are touched only to give the function a more inclusive name.

BUG=698195
==========

to

==========
Mark sub-frame root layer non-fast scrollable when wheel handlers present.

Just like with touch event handlers, at present a sub-frame's
compositor does not receive regions rects for non-fast scrollable
regions that accompany MouseWheel handlers. Until ScrollingCoordinator
is made per-frame, we should mark the entire root layer rect as
non-fast scrollable in the presence of blocking wheel handlers.

The primary changes are all in render_widget_compositor.cc; other
files are touched only to give the function a more inclusive name.

BUG=698195

Review-Url: https://codereview.chromium.org/2740133002
Cr-Commit-Position: refs/heads/master@{#456131}
Committed:
https://chromium.googlesource.com/chromium/src/+/47b1b819e0e239b1c753ae4e736f...
==========

[commit-bot: I haz the power, 2017-03-10 19:25:20]

Committed patchset #4 (id:50001) as
https://chromium.googlesource.com/chromium/src/+/47b1b819e0e239b1c753ae4e736f...