Fix latency info tracking for coalesced events.

Fix latency info tracking for coalesced events.

Keep the oldest touch event id so the latency info will be associated
with that object. Don't adjust the dispatch type for an event that
gets coalesced since it it should carry the same type as the touch id
ack.

BUG=711325
Review-Url: https://codereview.chromium.org/2822293002
Cr-Commit-Position: refs/heads/master@{#465457}
Committed: https://chromium.googlesource.com/chromium/src/+/401d48085b5a5db8bab875c9a6732f61f23a42b1

[dtapuska, 2017-04-18 20:02:49]

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

[dtapuska, 2017-04-18 20:02:56]

Description was changed from

==========
Fix latency info tracking for coalesced events.

Keep the oldest touch event id so the latency info will be associated
with that object. Don't adjust the dispatch type for an event that
gets coalesced since it it should carry the same type as the touch id
ack.

BUG=711325
==========

to

==========
Fix latency info tracking for coalesced events.

Keep the oldest touch event id so the latency info will be associated
with that object. Don't adjust the dispatch type for an event that
gets coalesced since it it should carry the same type as the touch id
ack.

BUG=711325
==========

[dtapuska, 2017-04-18 20:02:56]

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

[dtapuska, 2017-04-18 20:03:05]

PTAL

[commit-bot: I haz the power, 2017-04-18 20:03:27]

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

[dtapuska, 2017-04-18 20:29:56]

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

[commit-bot: I haz the power, 2017-04-18 20:30:36]

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

[tdresser, 2017-04-18 20:43:39]

LGTM

https://codereview.chromium.org/2822293002/diff/40001/content/renderer/input/...
File content/renderer/input/main_thread_event_queue.cc (left):

https://codereview.chromium.org/2822293002/diff/40001/content/renderer/input/...
content/renderer/input/main_thread_event_queue.cc:95: originally_cancelable_ =
other_event.originally_cancelable_;
Is this correct? I would have thought this logic would be the same as
dispatch_type_.

[dtapuska, 2017-04-18 20:54:45]

On 2017/04/18 20:43:39, tdresser wrote:
> LGTM
> 
>
https://codereview.chromium.org/2822293002/diff/40001/content/renderer/input/...
> File content/renderer/input/main_thread_event_queue.cc (left):
> 
>
https://codereview.chromium.org/2822293002/diff/40001/content/renderer/input/...
> content/renderer/input/main_thread_event_queue.cc:95: originally_cancelable_ =
> other_event.originally_cancelable_;
> Is this correct? I would have thought this logic would be the same as
> dispatch_type_.

Yes it is correct. Cancelable != ACK_Dispatch_Type. This is the ack dispatch
type. Not the dispatch type on WebInputEvent. Yes very confusing. Need to fix
this sometime.

[commit-bot: I haz the power, 2017-04-18 21:29:23]

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

[commit-bot: I haz the power, 2017-04-18 21:29:24]

Dry run: This issue passed the CQ dry run.

[dtapuska, 2017-04-19 01:24:22]

The CQ bit was checked by dtapuska@chromium.org

[commit-bot: I haz the power, 2017-04-19 01:24:35]

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-04-19 01:28:42]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1492565062940060,
"parent_rev": "12c4f68ae8557ca0af22bbe6281a832e2ed29d9a", "commit_rev":
"401d48085b5a5db8bab875c9a6732f61f23a42b1"}

[commit-bot: I haz the power, 2017-04-19 01:28:52]

Description was changed from

==========
Fix latency info tracking for coalesced events.

Keep the oldest touch event id so the latency info will be associated
with that object. Don't adjust the dispatch type for an event that
gets coalesced since it it should carry the same type as the touch id
ack.

BUG=711325
==========

to

==========
Fix latency info tracking for coalesced events.

Keep the oldest touch event id so the latency info will be associated
with that object. Don't adjust the dispatch type for an event that
gets coalesced since it it should carry the same type as the touch id
ack.

BUG=711325

Review-Url: https://codereview.chromium.org/2822293002
Cr-Commit-Position: refs/heads/master@{#465457}
Committed:
https://chromium.googlesource.com/chromium/src/+/401d48085b5a5db8bab875c9a673...
==========

[commit-bot: I haz the power, 2017-04-19 01:28:53]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/401d48085b5a5db8bab875c9a673...