Add code attribute into onKeyEvent argument.

Add code attribute into onKeyEvent argument.

The "code" holds the identification of physical key.
Unlike to "key" value, this value is not affected on current keyboard layout or modifier state.

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

[Seigo Nonaka, 2013-02-08 07:38:10]

ping? just in case.

[Zachary Kuznia, 2013-02-08 07:43:52]

lgtm

[Seigo Nonaka, 2013-02-08 07:57:02]

zork: Thank you for your review.

Adding Matt as the owner of Extension.
Could you take a look?

Thank you.

[Seigo Nonaka, 2013-02-11 18:12:47]

Matt: ping? just in case.
I'm sorry to rush you.

[Matt Perry, 2013-02-11 19:38:51]

LGTM

Note that TEST= should be instructions for QA. If you don't have instructions,
just leave it out.

https://codereview.chromium.org/12224031/diff/1/chrome/common/extensions/api/...
File chrome/common/extensions/api/input_ime.json (right):

https://codereview.chromium.org/12224031/diff/1/chrome/common/extensions/api/...
chrome/common/extensions/api/input_ime.json:18: "code": {"type": "string",
"description": "Value of the physical key being pressed. The value is not
affected with current keyboard layout or modifier state."},
affected by*

[Seigo Nonaka, 2013-02-12 02:22:00]

Sure,

Thank you.

https://codereview.chromium.org/12224031/diff/1/chrome/common/extensions/api/...
File chrome/common/extensions/api/input_ime.json (right):

https://codereview.chromium.org/12224031/diff/1/chrome/common/extensions/api/...
chrome/common/extensions/api/input_ime.json:18: "code": {"type": "string",
"description": "Value of the physical key being pressed. The value is not
affected with current keyboard layout or modifier state."},
On 2013/02/11 19:38:51, Matt Perry wrote:
> affected by*

Done.

[commit-bot: I haz the power, 2013-02-12 02:23:52]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nona@chromium.org/12224031/5002

[commit-bot: I haz the power, 2013-02-12 03:31:52]

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

[commit-bot: I haz the power, 2013-02-12 03:47:14]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nona@chromium.org/12224031/5002

[commit-bot: I haz the power, 2013-02-12 04:46:09]

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