Reset the touch-action on the touch gesture end sequence.

Reset the touch-action on the touch gesture end sequence.

This makes the property that the touch-action is always adjusted on
receiving touch ack. Change https://codereview.chromium.org/2715623002
broke the property that a single touch action was outstanding. This caused
issues with a delayed touch-end handler possibly and another touchstart
being issued the touch-action would have been reset intermitently there.

There is already a test that covers this and it will be apparent when
the test is modified to support rAF aligned input enabled (which then
turns on the pass through touch event queue).

BUG=624368
Review-Url: https://codereview.chromium.org/2724743004
Cr-Commit-Position: refs/heads/master@{#454036}
Committed: https://chromium.googlesource.com/chromium/src/+/783059297a429d72f45dc835a738233139d822f7

[dtapuska, 2017-03-01 16:23:15]

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

[dtapuska, 2017-03-01 16:23:23]

Description was changed from

==========
Reset the touch-action on the touch gesture end sequence.

This makes the property that the touch-action is always adjusted on
receiving touch ack. Change https://codereview.chromium.org/2715623002
broke the property that a single touch action was outstanding. This caused
issues with a delayed touch-end handler possibly and another touchstart
being issued the touch-action would have been reset intermitently there.

There is already a test that covers this and it will be apparent when
the test is modified to support rAF aligned input enabled (which then
turns on the pass through touch event queue).

BUG=624368
==========

to

==========
Reset the touch-action on the touch gesture end sequence.

This makes the property that the touch-action is always adjusted on
receiving touch ack. Change https://codereview.chromium.org/2715623002
broke the property that a single touch action was outstanding. This caused
issues with a delayed touch-end handler possibly and another touchstart
being issued the touch-action would have been reset intermitently there.

There is already a test that covers this and it will be apparent when
the test is modified to support rAF aligned input enabled (which then
turns on the pass through touch event queue).

BUG=624368
==========

[dtapuska, 2017-03-01 16:23:23]

dtapuska@chromium.org changed reviewers:
+ rbyers@chromium.org, tdresser@chromium.org

[commit-bot: I haz the power, 2017-03-01 16:23:32]

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

[dtapuska, 2017-03-01 16:23:52]

I discussed this with Rick as a potential fallout of my change earlier this
week.

[tdresser, 2017-03-01 16:45:36]

Could we update the test for this as part of this patch?

[dtapuska, 2017-03-01 16:52:00]

On 2017/03/01 16:45:36, tdresser wrote:
> Could we update the test for this as part of this patch?

It is here: https://codereview.chromium.org/2727683002/patch/1/10001

The test doesn't actually get updated; but it is the activation of the pass
through touch event queue that was failing without this fix.

[commit-bot: I haz the power, 2017-03-01 17:21:29]

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

[commit-bot: I haz the power, 2017-03-01 17:21:30]

Dry run: This issue passed the CQ dry run.

[dtapuska, 2017-03-01 20:23:13]

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

[dtapuska, 2017-03-01 20:23:35]

On 2017/03/01 16:52:00, dtapuska wrote:
> On 2017/03/01 16:45:36, tdresser wrote:
> > Could we update the test for this as part of this patch?
> 
> It is here: https://codereview.chromium.org/2727683002/patch/1/10001
> 
> The test doesn't actually get updated; but it is the activation of the pass
> through touch event queue that was failing without this fix.

Test case added. PTAL.

[commit-bot: I haz the power, 2017-03-01 20:23:51]

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

[tdresser, 2017-03-01 20:30:03]

LGTM, thanks.

[dtapuska, 2017-03-01 20:34:16]

The CQ bit was unchecked by dtapuska@chromium.org

[dtapuska, 2017-03-01 20:34:16]

The CQ bit was checked by dtapuska@chromium.org

[commit-bot: I haz the power, 2017-03-01 20:34:30]

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-01 21:25:20]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1488400456721160,
"parent_rev": "612b3dc0d5ab6aaf839cc33e71d5e4f1e49e2bf8", "commit_rev":
"783059297a429d72f45dc835a738233139d822f7"}

[commit-bot: I haz the power, 2017-03-01 21:25:53]

Description was changed from

==========
Reset the touch-action on the touch gesture end sequence.

This makes the property that the touch-action is always adjusted on
receiving touch ack. Change https://codereview.chromium.org/2715623002
broke the property that a single touch action was outstanding. This caused
issues with a delayed touch-end handler possibly and another touchstart
being issued the touch-action would have been reset intermitently there.

There is already a test that covers this and it will be apparent when
the test is modified to support rAF aligned input enabled (which then
turns on the pass through touch event queue).

BUG=624368
==========

to

==========
Reset the touch-action on the touch gesture end sequence.

This makes the property that the touch-action is always adjusted on
receiving touch ack. Change https://codereview.chromium.org/2715623002
broke the property that a single touch action was outstanding. This caused
issues with a delayed touch-end handler possibly and another touchstart
being issued the touch-action would have been reset intermitently there.

There is already a test that covers this and it will be apparent when
the test is modified to support rAF aligned input enabled (which then
turns on the pass through touch event queue).

BUG=624368

Review-Url: https://codereview.chromium.org/2724743004
Cr-Commit-Position: refs/heads/master@{#454036}
Committed:
https://chromium.googlesource.com/chromium/src/+/783059297a429d72f45dc835a738...
==========

[commit-bot: I haz the power, 2017-03-01 21:25:54]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/783059297a429d72f45dc835a738...