Port views::KeyEvent functions to aura::KeyEvent.

Port views::KeyEvent functions to aura::KeyEvent.

They will be used for adding IME support to Aura.

BUG=chromium:97261
TEST=try


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=110288

[James Su, 2011-11-15 11:51:10]

LGTM

http://codereview.chromium.org/8567021/diff/6001/ui/aura/event.cc
File ui/aura/event.cc (right):

http://codereview.chromium.org/8567021/diff/6001/ui/aura/event.cc#newcode165
ui/aura/event.cc:165: if (!native_event()) {
nit: no { } for single line if statement.

http://codereview.chromium.org/8567021/diff/6001/ui/aura/event.cc#newcode190
ui/aura/event.cc:190: return ui::GetCharacterFromKeyCode(key_code_, flags() &
ui::EF_SHIFT_DOWN);
ditto.

[Yusuke Sato, 2011-11-15 12:20:59]

http://codereview.chromium.org/8567021/diff/6001/ui/aura/event.cc
File ui/aura/event.cc (right):

http://codereview.chromium.org/8567021/diff/6001/ui/aura/event.cc#newcode165
ui/aura/event.cc:165: if (!native_event()) {
On 2011/11/15 11:51:10, James Su wrote:
> nit: no { } for single line if statement.

Done.

http://codereview.chromium.org/8567021/diff/6001/ui/aura/event.cc#newcode190
ui/aura/event.cc:190: return ui::GetCharacterFromKeyCode(key_code_, flags() &
ui::EF_SHIFT_DOWN);
On 2011/11/15 11:51:10, James Su wrote:
> ditto.

Done.

[sky, 2011-11-15 16:39:31]

LGTM

[commit-bot: I haz the power, 2011-11-16 06:47:41]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yusukes@chromium.org/8567021/9007

[commit-bot: I haz the power, 2011-11-16 10:19:27]

Commit queue rejected this change because the description was changed
between the time the change entered the commit queue and the time it
was ready to commit. You can safely check the commit box again.

[commit-bot: I haz the power, 2011-11-16 10:26:33]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yusukes@chromium.org/8567021/9007

[commit-bot: I haz the power, 2011-11-16 11:57:20]

Change committed as 110288