Makes InputMethodMus deal correctly with focus changes

Makes InputMethodMus deal correctly with focus changes

Calling DispatchKeyEventPostIME() may lead to changing the focused
text client, which means AckPendingCallbacksUnhandled() may be
called. If AckPendingCallbacksUnhandled() is called pending_callbacks_
is cleared. ProcessKeyEventCallback() should remove the callback from
the list, then call DispatchKeyEventPostIME() so that it doesn't
matter if the focus changes.

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

Review-Url: https://codereview.chromium.org/2767103004
Cr-Commit-Position: refs/heads/master@{#459109}
Committed: https://chromium.googlesource.com/chromium/src/+/ae34d020eb0cb8d4ea748151470ce031df6f8366

[sky, 2017-03-22 23:46:02]

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

[commit-bot: I haz the power, 2017-03-22 23:47:13]

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-03-23 01:04:48]

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

[commit-bot: I haz the power, 2017-03-23 01:04:49]

Dry run: This issue passed the CQ dry run.

[Hadi, 2017-03-23 13:42:51]

lgtm.

[sky, 2017-03-23 16:20:12]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2017-03-23 16:20:48]

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-23 16:25:19]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490286012417820, "parent_rev":
"8fe88eadb09b3307f897d17f1306fdaa78afe36f", "commit_rev":
"ae34d020eb0cb8d4ea748151470ce031df6f8366"}

[commit-bot: I haz the power, 2017-03-23 16:26:05]

Description was changed from

==========
Makes InputMethodMus deal correctly with focus changes

Calling DispatchKeyEventPostIME() may lead to changing the focused
text client, which means AckPendingCallbacksUnhandled() may be
called. If AckPendingCallbacksUnhandled() is called pending_callbacks_
is cleared. ProcessKeyEventCallback() should remove the callback from
the list, then call DispatchKeyEventPostIME() so that it doesn't
matter if the focus changes.

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

to

==========
Makes InputMethodMus deal correctly with focus changes

Calling DispatchKeyEventPostIME() may lead to changing the focused
text client, which means AckPendingCallbacksUnhandled() may be
called. If AckPendingCallbacksUnhandled() is called pending_callbacks_
is cleared. ProcessKeyEventCallback() should remove the callback from
the list, then call DispatchKeyEventPostIME() so that it doesn't
matter if the focus changes.

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

Review-Url: https://codereview.chromium.org/2767103004
Cr-Commit-Position: refs/heads/master@{#459109}
Committed:
https://chromium.googlesource.com/chromium/src/+/ae34d020eb0cb8d4ea748151470c...
==========

[commit-bot: I haz the power, 2017-03-23 16:26:06]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/ae34d020eb0cb8d4ea748151470c...