scheduler: Detect when a main thread touch stream turns into a compositor gesture

scheduler: Detect when a main thread touch stream turns into a compositor gesture

When a main thread gesture turns into a compositor driven gesture (e.g.,
because the touch handler didn't preventDefault(), allowing the
compositor to scroll), we previously failed to update the detected use
case from MAIN_THREAD_GESTURE to COMPOSITOR_GESTURE. This patch makes
sure we always update our policy whenever the use case detected from the
input stream hanges, ensuring an appropriate scheduling policy is
activated.

BUG=600202
Committed: https://crrev.com/1fd38c63f6df73d7d9d86f312b02a1c0d4b08d78
Cr-Commit-Position: refs/heads/master@{#393022}

[Sami, 2016-05-11 17:00:09]

Description was changed from

==========
scheduler: Detect when a main thread touch stream turns into a compositor
gesture

When a main thread gesture turns into a compositor driven gesture (e.g.,
because the touch handler didn't preventDefault(), allowing the
compositor to scroll), we previously failed to update the detected use
case from MAIN_THREAD_GESTURE to COMPOSITOR_GESTURE. This patch makes
sure we always update our policy whenever the use case detected from the
cinput stream hanges, ensuring an appropriate scheduling policy is
activated.

BUG=600202
==========

to

==========
scheduler: Detect when a main thread touch stream turns into a compositor
gesture

When a main thread gesture turns into a compositor driven gesture (e.g.,
because the touch handler didn't preventDefault(), allowing the
compositor to scroll), we previously failed to update the detected use
case from MAIN_THREAD_GESTURE to COMPOSITOR_GESTURE. This patch makes
sure we always update our policy whenever the use case detected from the
input stream hanges, ensuring an appropriate scheduling policy is
activated.

BUG=600202
==========

[Sami, 2016-05-11 17:00:18]

skyostil@chromium.org changed reviewers:
+ alexclarke@chromium.org

[alex clarke (OOO till 29th), 2016-05-11 17:06:52]

lgtm

https://codereview.chromium.org/1971763003/diff/1/components/scheduler/render...
File components/scheduler/renderer/renderer_scheduler_impl_unittest.cc (right):

https://codereview.chromium.org/1971763003/diff/1/components/scheduler/render...
components/scheduler/renderer/renderer_scheduler_impl_unittest.cc:394:
scheduler_->ForceUpdatePolicy();
Perhaps assert on the policy - to double check what we think is happen is
actually happening?

[Sami, 2016-05-11 18:00:10]

https://codereview.chromium.org/1971763003/diff/1/components/scheduler/render...
File components/scheduler/renderer/renderer_scheduler_impl_unittest.cc (right):

https://codereview.chromium.org/1971763003/diff/1/components/scheduler/render...
components/scheduler/renderer/renderer_scheduler_impl_unittest.cc:394:
scheduler_->ForceUpdatePolicy();
On 2016/05/11 17:06:52, alex clarke wrote:
> Perhaps assert on the policy - to double check what we think is happen is
> actually happening?

Good idea, done.

[Sami, 2016-05-11 18:00:14]

The CQ bit was checked by skyostil@chromium.org

[Sami, 2016-05-11 18:00:14]

The patchset sent to the CQ was uploaded after l-g-t-m from
alexclarke@chromium.org
Link to the patchset: https://codereview.chromium.org/1971763003/#ps20001
(title: "Review comments")

[commit-bot: I haz the power, 2016-05-11 18:00:42]

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

[commit-bot: I haz the power, 2016-05-11 19:23:38]

Description was changed from

==========
scheduler: Detect when a main thread touch stream turns into a compositor
gesture

When a main thread gesture turns into a compositor driven gesture (e.g.,
because the touch handler didn't preventDefault(), allowing the
compositor to scroll), we previously failed to update the detected use
case from MAIN_THREAD_GESTURE to COMPOSITOR_GESTURE. This patch makes
sure we always update our policy whenever the use case detected from the
input stream hanges, ensuring an appropriate scheduling policy is
activated.

BUG=600202
==========

to

==========
scheduler: Detect when a main thread touch stream turns into a compositor
gesture

When a main thread gesture turns into a compositor driven gesture (e.g.,
because the touch handler didn't preventDefault(), allowing the
compositor to scroll), we previously failed to update the detected use
case from MAIN_THREAD_GESTURE to COMPOSITOR_GESTURE. This patch makes
sure we always update our policy whenever the use case detected from the
input stream hanges, ensuring an appropriate scheduling policy is
activated.

BUG=600202
==========

[commit-bot: I haz the power, 2016-05-11 19:23:40]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-11 19:25:06]

Description was changed from

==========
scheduler: Detect when a main thread touch stream turns into a compositor
gesture

When a main thread gesture turns into a compositor driven gesture (e.g.,
because the touch handler didn't preventDefault(), allowing the
compositor to scroll), we previously failed to update the detected use
case from MAIN_THREAD_GESTURE to COMPOSITOR_GESTURE. This patch makes
sure we always update our policy whenever the use case detected from the
input stream hanges, ensuring an appropriate scheduling policy is
activated.

BUG=600202
==========

to

==========
scheduler: Detect when a main thread touch stream turns into a compositor
gesture

When a main thread gesture turns into a compositor driven gesture (e.g.,
because the touch handler didn't preventDefault(), allowing the
compositor to scroll), we previously failed to update the detected use
case from MAIN_THREAD_GESTURE to COMPOSITOR_GESTURE. This patch makes
sure we always update our policy whenever the use case detected from the
input stream hanges, ensuring an appropriate scheduling policy is
activated.

BUG=600202

Committed: https://crrev.com/1fd38c63f6df73d7d9d86f312b02a1c0d4b08d78
Cr-Commit-Position: refs/heads/master@{#393022}
==========

[commit-bot: I haz the power, 2016-05-11 19:25:07]

Patchset 2 (id:??) landed as
https://crrev.com/1fd38c63f6df73d7d9d86f312b02a1c0d4b08d78
Cr-Commit-Position: refs/heads/master@{#393022}