Properly forward queued touch events

Properly forward queued touch events

If a touch-event is added to an empty TouchEventQueue and should not be
forwarded to the renderer, it is ack'ed to the client immediately.  However,
this ack may trigger follow-up events synchronously, which would be queued but
never forwarded to the renderer.  Properly forward the follow-up events when the
first event reaches an empty queue but was never forwarded.

BUG=230735
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=224936

[jdduke (slow), 2013-09-23 19:50:38]

PTAL for a sanity check.  I'll add some tests that hit this corner case, thanks.

[jdduke (slow), 2013-09-23 20:50:53]

@sadrul: PTAL.

Yufeng brough up a corner case that neither platform currently hits, but that
should be properly handled, particularly as the touch pipeline evolves.

[Yufeng Shen (Slow to review), 2013-09-23 21:13:28]

On 2013/09/23 19:50:38, jdduke wrote:
> PTAL for a sanity check.  I'll add some tests that hit this corner case,
thanks.

LGTM

[sadrul, 2013-09-23 22:37:40]

Yep. This seems reasonable to me.

[commit-bot: I haz the power, 2013-09-23 22:47:25]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a lowly provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[sadrul, 2013-09-23 22:48:40]

LGTM

[jdduke (slow), 2013-09-23 22:49:35]

aelias@ for owners.  Thanks!

[jdduke (slow), 2013-09-23 22:53:29]

Oops, aelias@ please disregard review (unless you want to :)... for whatever
reason I thought sadrul@ had already LGTM'ed and it wasn't enough.

[commit-bot: I haz the power, 2013-09-23 22:58:47]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jdduke@chromium.org/24312009/4001

[commit-bot: I haz the power, 2013-09-24 03:48:50]

Retried try job too often on win_rel for step(s) sync_integration_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2013-09-24 04:27:57]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jdduke@chromium.org/24312009/4001

[commit-bot: I haz the power, 2013-09-24 07:10:44]

Change committed as 224936