[KeyboardEvent] DOM |Key| support in dispatchKeyEvent

[KeyboardEvent] DOM |Key| support in dispatchKeyEvent

Updated dispatchKeyEvent in the inputHandler as protocol.json is updated with additional parameter for the domKey.

R= dgozman, pfeldman
BUG=227231
Committed: https://crrev.com/bb338c90f33f7f9ce67b56b293f255bb473c1c5d
Cr-Commit-Position: refs/heads/master@{#332406}

[Habib Virji, 2015-06-02 12:15:31]

Had dependency on the blink side: https://codereview.chromium.org/1157563008/

This commit is based on the comment from dgozman in 
https://codereview.chromium.org/933323002/.

[dgozman, 2015-06-02 12:30:46]

Note that due to code generation and different repos, you will need 3 patches:
- implementation in chromium which leaves current version intact (just add
similarly named method with more parameters);
- protocol and usage change in blink;
- cleanup in chromium to remove old method.

[Habib Virji, 2015-06-02 12:40:00]

Done. Will upload cleanup once the code lands. So the process of landing will be
first chromium side changes will land and then blink, is it right?

[dgozman, 2015-06-02 13:22:01]

> So the process of landing will be first chromium side
> changes will land and then blink, is it right?
First chromium patch, then blink, after that wait for blink roll, then land
chromium cleanup.

https://codereview.chromium.org/1143073007/diff/40001/content/browser/devtool...
File content/browser/devtools/protocol/input_handler.cc (right):

https://codereview.chromium.org/1143073007/diff/40001/content/browser/devtool...
content/browser/devtools/protocol/input_handler.cc:215: Response
InputHandler::DispatchKeyEvent(
Please implement one of the methods using another instead of copying.

[Habib Virji, 2015-06-02 14:15:20]

https://codereview.chromium.org/1143073007/diff/40001/content/browser/devtool...
File content/browser/devtools/protocol/input_handler.cc (right):

https://codereview.chromium.org/1143073007/diff/40001/content/browser/devtool...
content/browser/devtools/protocol/input_handler.cc:215: Response
InputHandler::DispatchKeyEvent(
On 2015/06/02 13:22:01, dgozman wrote:
> Please implement one of the methods using another instead of copying.

Done.

[dgozman, 2015-06-02 15:25:39]

lgtm

[Habib Virji, 2015-06-02 15:26:23]

The CQ bit was checked by habib.virji@samsung.com to run a CQ dry run

[commit-bot: I haz the power, 2015-06-02 15:27:21]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1143073007/60001

[commit-bot: I haz the power, 2015-06-02 16:25:20]

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

[commit-bot: I haz the power, 2015-06-02 16:25:21]

Dry run: This issue passed the CQ dry run.

[Habib Virji, 2015-06-02 16:26:32]

The CQ bit was checked by habib.virji@samsung.com

[commit-bot: I haz the power, 2015-06-02 16:27:11]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1143073007/60001

[commit-bot: I haz the power, 2015-06-02 16:32:31]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-06-02 16:33:19]

Patchset 4 (id:??) landed as
https://crrev.com/bb338c90f33f7f9ce67b56b293f255bb473c1c5d
Cr-Commit-Position: refs/heads/master@{#332406}