Send a GSB to main thread before switching to it in the middle of scrolling.

Send a GSB to main thread before switching to it in the middle of
scrolling.

When wheel scroll latching is enabled, GSB occurs only once in the
beginning of a scroll sequence. So, it is possible to need to switch
to the main thread scroll handling in the middle of a scroll scroll
sequence (e.g. due to a change in main thread scrolling reasons).
To have a complete scroll sequence on main thread side and do the scroll
chain computation only while handling GSB, this cl sends a GSB to the main thread before sending the GSU that is not handled in the impl thread
side.


BUG=526463
TEST=AnimateInput/InputHandlerProxyTest.GestureScrollHandlingSwitchedToMainThread/*

Review-Url: https://codereview.chromium.org/2854683002
Cr-Commit-Position: refs/heads/master@{#471006}
Committed: https://chromium.googlesource.com/chromium/src/+/42f43cb31dd1ab13aa264e62d1967a4dcbd0dfa7

[sahel, 2017-05-01 19:54:53]

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

[commit-bot: I haz the power, 2017-05-01 19:55:35]

Dry run: CQ is trying da patch.

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

[sahel, 2017-05-01 20:22:48]

Description was changed from

==========
send a GSB to main thread before switching to it in the middle of scrolling.


Send a GSB before the first GSU after switching to the main thread.

BUG=526463
==========

to

==========
Send a GSB to main thread before switching to it in the middle of
scrolling.

When wheel scroll latching is enabled, GSB occurs only once in the
beginning of a scroll sequence. So, it is possible to need to switch
to the main thread scroll handling in the middle of a scroll scroll
sequence (e.g. due to a change in main thread scrolling reasons).
To have a complete scroll sequence on main thread side and do the scroll
chain computation only while handling GSB, this cl sends a GSB to the main
thread before sending the GSU that is not handled in the impl thread
side.


BUG=526463
TEST=AnimateInput/InputHandlerProxyTest.GestureScrollHandlingSwitchedToMainThread/*
==========

[commit-bot: I haz the power, 2017-05-01 20:53:38]

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

[commit-bot: I haz the power, 2017-05-01 20:53:39]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[sahel, 2017-05-01 21:05:59]

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

[commit-bot: I haz the power, 2017-05-01 21:06:53]

Dry run: CQ is trying da patch.

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

[sahel, 2017-05-01 21:24:33]

Patchset #2 (id:20001) has been deleted

[commit-bot: I haz the power, 2017-05-01 21:28:01]

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

[sahel, 2017-05-01 21:40:10]

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

[commit-bot: I haz the power, 2017-05-01 21:41:12]

Dry run: CQ is trying da patch.

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

[sahel, 2017-05-01 22:21:59]

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

[commit-bot: I haz the power, 2017-05-01 22:51:50]

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

[commit-bot: I haz the power, 2017-05-01 22:51:51]

Dry run: This issue passed the CQ dry run.

[dtapuska, 2017-05-09 19:16:29]

On 2017/05/01 22:21:59, sahel wrote:

lgtm

[sahel, 2017-05-09 19:18:45]

The CQ bit was checked by sahel@chromium.org

[commit-bot: I haz the power, 2017-05-09 19:19:46]

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-05-09 19:53:57]

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

[commit-bot: I haz the power, 2017-05-09 19:53:58]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[sahel, 2017-05-11 14:51:30]

The CQ bit was checked by sahel@chromium.org

[commit-bot: I haz the power, 2017-05-11 14:51:56]

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-05-11 16:35:57]

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

[commit-bot: I haz the power, 2017-05-11 16:35:59]

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

[sahel, 2017-05-11 17:50:06]

The CQ bit was checked by sahel@chromium.org

[commit-bot: I haz the power, 2017-05-11 17:51:18]

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-05-11 17:58:42]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1494525006993330,
"parent_rev": "a1692d2da9038b8515e47f4590f24df81b56c521", "commit_rev":
"42f43cb31dd1ab13aa264e62d1967a4dcbd0dfa7"}

[commit-bot: I haz the power, 2017-05-11 17:58:54]

Description was changed from

==========
Send a GSB to main thread before switching to it in the middle of
scrolling.

When wheel scroll latching is enabled, GSB occurs only once in the
beginning of a scroll sequence. So, it is possible to need to switch
to the main thread scroll handling in the middle of a scroll scroll
sequence (e.g. due to a change in main thread scrolling reasons).
To have a complete scroll sequence on main thread side and do the scroll
chain computation only while handling GSB, this cl sends a GSB to the main
thread before sending the GSU that is not handled in the impl thread
side.


BUG=526463
TEST=AnimateInput/InputHandlerProxyTest.GestureScrollHandlingSwitchedToMainThread/*
==========

to

==========
Send a GSB to main thread before switching to it in the middle of
scrolling.

When wheel scroll latching is enabled, GSB occurs only once in the
beginning of a scroll sequence. So, it is possible to need to switch
to the main thread scroll handling in the middle of a scroll scroll
sequence (e.g. due to a change in main thread scrolling reasons).
To have a complete scroll sequence on main thread side and do the scroll
chain computation only while handling GSB, this cl sends a GSB to the main
thread before sending the GSU that is not handled in the impl thread
side.


BUG=526463
TEST=AnimateInput/InputHandlerProxyTest.GestureScrollHandlingSwitchedToMainThread/*

Review-Url: https://codereview.chromium.org/2854683002
Cr-Commit-Position: refs/heads/master@{#471006}
Committed:
https://chromium.googlesource.com/chromium/src/+/42f43cb31dd1ab13aa264e62d196...
==========

[commit-bot: I haz the power, 2017-05-11 17:58:55]

Committed patchset #2 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/42f43cb31dd1ab13aa264e62d196...