Removing UIEvent.keyCode and charCode.

Removing UIEvent.keyCode and charCode.

As discussed on this intent:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/pjohnl9znQQ

BUG=508316
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=199019

[Paritosh Kumar, 2015-07-13 12:44:35]

paritosh.in@samsung.com changed reviewers:
+ philipj@opera.com, rbyers@chromium.org

[Paritosh Kumar, 2015-07-13 12:44:35]

Please have a look.

[philipj_slow, 2015-07-13 13:58:59]

Looks good, but the thread on blink-dev is still missing two L-G-T-Ms.

https://codereview.chromium.org/1232403003/diff/40001/LayoutTests/fast/events...
File LayoutTests/fast/events/script-tests/init-events.js (right):

https://codereview.chromium.org/1232403003/diff/40001/LayoutTests/fast/events...
LayoutTests/fast/events/script-tests/init-events.js:103:
shouldBe("testInitEvent('Mouse', '\"a\", false, false, window, 1001, 1002, 1003,
1004, 1005, false, false, false, false, 1006, null').keyCode", "undefined");
There are tests for KeyboardEvent in this file to, so I think you can remove
tests of keyCode/charCode for non-KeyboardEvents instead. The webexposed test
already shows that it was successfully removed, and

https://codereview.chromium.org/1232403003/diff/40001/Source/core/events/UIEv...
File Source/core/events/UIEvent.idl (left):

https://codereview.chromium.org/1232403003/diff/40001/Source/core/events/UIEv...
Source/core/events/UIEvent.idl:38: [MeasureAs=UIEventCharCode] readonly
attribute long charCode;
Please also remove UIEventCharCode and UIEventKeyCode from UseCounter.h

[Paritosh Kumar, 2015-07-16 07:53:47]

Thanks Philip,

Removed tests of keyCode/charCode for non-KeyboardEvents. And We have 3 L-G-T-Ms
on intent thread.
Please have a look.

[philipj_slow, 2015-07-16 08:02:41]

The CQ bit was checked by philipj@opera.com

[philipj_slow, 2015-07-16 08:02:42]

lgtm

I edited the description slightly to not have such a long line.

[commit-bot: I haz the power, 2015-07-16 08:03:03]

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

[commit-bot: I haz the power, 2015-07-16 09:06:18]

Committed patchset #4 (id:60001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=199019