Makes ~InputMethodMus ack any in flight events

Makes ~InputMethodMus ack any in flight events

It's entirely possible for InputMethodMus to be awaiting a response
from the ime server when deleted. If this happens the event ack to mus
for the key event is never acked and mus won't process the next event
until it times out. This patch makes ~InputMethodMus ack any in flight
events immediately with unhandled.

BUG=690709
TEST=covered by test
R=moshayedi@chromium.org
TBR=sadrul@chromium.org

Review-Url: https://codereview.chromium.org/2685183005
Cr-Commit-Position: refs/heads/master@{#450039}
Committed: https://chromium.googlesource.com/chromium/src/+/7a0d90d6351c104b896cb2ecef771dab00d68323

[sky, 2017-02-11 00:30:21]

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

[commit-bot: I haz the power, 2017-02-11 00:31:11]

Dry run: 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-02-11 01:24:40]

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

[commit-bot: I haz the power, 2017-02-11 01:24:40]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[sky, 2017-02-11 01:38:56]

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

[commit-bot: I haz the power, 2017-02-11 01:39:56]

Dry run: 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-02-11 07:41:06]

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

[commit-bot: I haz the power, 2017-02-11 07:41:07]

Dry run: Try jobs failed on following builders:
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_TIMED_OUT,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_TIMED_OUT,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[Hadi, 2017-02-13 14:36:20]

Thanks for the CL. LGTM.

[sky, 2017-02-13 16:21:20]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2017-02-13 16:21: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, 2017-02-13 16:21:42]

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

[commit-bot: I haz the power, 2017-02-13 16:21:43]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
Committers are members of the group "project-chromium-committers".
Note that this has nothing to do with OWNERS files.

[sky, 2017-02-13 17:29:34]

Description was changed from

==========
Makes ~InputMethodMus ack any in flight events

It's entirely possible for InputMethodMus to be awaiting a response
from the ime server when deleted. If this happens the event ack to mus
for the key event is never acked and mus won't process the next event
until it times out. This patch makes ~InputMethodMus ack any in flight
events immediately with unhandled.

BUG=690709
TEST=covered by test
R=moshayedi@chromium.org
==========

to

==========
Makes ~InputMethodMus ack any in flight events

It's entirely possible for InputMethodMus to be awaiting a response
from the ime server when deleted. If this happens the event ack to mus
for the key event is never acked and mus won't process the next event
until it times out. This patch makes ~InputMethodMus ack any in flight
events immediately with unhandled.

BUG=690709
TEST=covered by test
R=moshayedi@chromium.org
TBR=sadrul@chromium.org
==========

[sky, 2017-02-13 17:29:45]

sky@chromium.org changed reviewers:
+ sadrul@chromium.org

[sky, 2017-02-13 17:29:59]

Hadi isn't a committer, so TBRing Sadrul.

[sky, 2017-02-13 17:30:04]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2017-02-13 17:30:48]

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

[sadrul, 2017-02-13 17:30:59]

lgtm

[commit-bot: I haz the power, 2017-02-13 19:22:30]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1487007004212160,
"parent_rev": "610b2e126ebb150af06f8400c73e6341c32cddb7", "commit_rev":
"7a0d90d6351c104b896cb2ecef771dab00d68323"}

[commit-bot: I haz the power, 2017-02-13 19:23:45]

Description was changed from

==========
Makes ~InputMethodMus ack any in flight events

It's entirely possible for InputMethodMus to be awaiting a response
from the ime server when deleted. If this happens the event ack to mus
for the key event is never acked and mus won't process the next event
until it times out. This patch makes ~InputMethodMus ack any in flight
events immediately with unhandled.

BUG=690709
TEST=covered by test
R=moshayedi@chromium.org
TBR=sadrul@chromium.org
==========

to

==========
Makes ~InputMethodMus ack any in flight events

It's entirely possible for InputMethodMus to be awaiting a response
from the ime server when deleted. If this happens the event ack to mus
for the key event is never acked and mus won't process the next event
until it times out. This patch makes ~InputMethodMus ack any in flight
events immediately with unhandled.

BUG=690709
TEST=covered by test
R=moshayedi@chromium.org
TBR=sadrul@chromium.org

Review-Url: https://codereview.chromium.org/2685183005
Cr-Commit-Position: refs/heads/master@{#450039}
Committed:
https://chromium.googlesource.com/chromium/src/+/7a0d90d6351c104b896cb2ecef77...
==========

[commit-bot: I haz the power, 2017-02-13 19:23:46]

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