Check for touch event queue size before popping.

Check for touch event queue size before popping.

Don't allow the browser process to crash if for some reason the
renderer process sends too many ack events.

BUG=598483
Committed: https://crrev.com/9d7cdbdbc717c68e7ad51b43d282599ebb6af6c3
Cr-Commit-Position: refs/heads/master@{#421330}

[dtapuska, 2016-09-27 19:42:25]

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

[dtapuska, 2016-09-27 19:42:30]

Description was changed from

==========
Check for touch event queue size before popping.

Don't allow the browser process to crash if for some reason the
renderer process sends too many ack events.

BUG=598483
==========

to

==========
Check for touch event queue size before popping.

Don't allow the browser process to crash if for some reason the
renderer process sends too many ack events.

BUG=598483
==========

[dtapuska, 2016-09-27 19:42:30]

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

[commit-bot: I haz the power, 2016-09-27 19:42:51]

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

[mustaq, 2016-09-27 19:53:25]

lgtm

https://codereview.chromium.org/2367253006/diff/1/content/browser/renderer_ho...
File content/browser/renderer_host/input/touch_event_queue.cc (right):

https://codereview.chromium.org/2367253006/diff/1/content/browser/renderer_ho...
content/browser/renderer_host/input/touch_event_queue.cc:770: NOTREACHED() <<
"Incorrect number of acks";
Nit: "Too many acks"?

[dtapuska, 2016-09-27 19:56:17]

The CQ bit was checked by dtapuska@chromium.org

[dtapuska, 2016-09-27 19:56:17]

The patchset sent to the CQ was uploaded after l-g-t-m from mustaq@chromium.org
Link to the patchset: https://codereview.chromium.org/2367253006/#ps20001
(title: "Change not reached line")

[commit-bot: I haz the power, 2016-09-27 19:56:41]

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, 2016-09-27 20:54:43]

Description was changed from

==========
Check for touch event queue size before popping.

Don't allow the browser process to crash if for some reason the
renderer process sends too many ack events.

BUG=598483
==========

to

==========
Check for touch event queue size before popping.

Don't allow the browser process to crash if for some reason the
renderer process sends too many ack events.

BUG=598483
==========

[commit-bot: I haz the power, 2016-09-27 20:54:44]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-27 21:01:18]

Description was changed from

==========
Check for touch event queue size before popping.

Don't allow the browser process to crash if for some reason the
renderer process sends too many ack events.

BUG=598483
==========

to

==========
Check for touch event queue size before popping.

Don't allow the browser process to crash if for some reason the
renderer process sends too many ack events.

BUG=598483

Committed: https://crrev.com/9d7cdbdbc717c68e7ad51b43d282599ebb6af6c3
Cr-Commit-Position: refs/heads/master@{#421330}
==========

[commit-bot: I haz the power, 2016-09-27 21:01:19]

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