Only overwrite DomKey of keydown event if it's a printable character

Only overwrite DomKey of keydown event if it's a printable character

InputMethodWin will combine the WM_KEY* and WM_CHAR messages, and will
overwrite DomKey for |keydown| with the DomKey of |keypress|.

We want to only overwite it if the new DomKey is not a control
character.

BUG=601882
Committed: https://crrev.com/19ec26442db050a055cf9f9486aa69db4bd70cf1
Cr-Commit-Position: refs/heads/master@{#386691}

[chongz, 2016-04-08 19:29:40]

chongz@chromium.org changed reviewers:
+ dtapuska@chromium.org, shuchen@chromium.org

[chongz, 2016-04-08 19:29:41]

I'm not sure what IME is expecting... PTAL, thanks!

[Shu Chen, 2016-04-11 21:00:04]

lgtm

[Shu Chen, 2016-04-11 21:00:37]

CC'ing azurewei@ who may have met the similar issue.

[chongz, 2016-04-12 14:17:16]

The CQ bit was checked by chongz@chromium.org

[commit-bot: I haz the power, 2016-04-12 14:17:39]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1875763003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1875763003/1

[commit-bot: I haz the power, 2016-04-12 15:27:07]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-12 15:27:57]

Description was changed from

==========
Only overwrite DomKey of keydown event if it's a printable character

InputMethodWin will combine the WM_KEY* and WM_CHAR messages, and will
overwrite DomKey for |keydown| with the DomKey of |keypress|.

We want to only overwite it if the new DomKey is not a control
character.

BUG=601882
==========

to

==========
Only overwrite DomKey of keydown event if it's a printable character

InputMethodWin will combine the WM_KEY* and WM_CHAR messages, and will
overwrite DomKey for |keydown| with the DomKey of |keypress|.

We want to only overwite it if the new DomKey is not a control
character.

BUG=601882

Committed: https://crrev.com/19ec26442db050a055cf9f9486aa69db4bd70cf1
Cr-Commit-Position: refs/heads/master@{#386691}
==========

[commit-bot: I haz the power, 2016-04-12 15:27:58]

Patchset 1 (id:??) landed as
https://crrev.com/19ec26442db050a055cf9f9486aa69db4bd70cf1
Cr-Commit-Position: refs/heads/master@{#386691}