Remove KeyboardEvent.prototype.keyLocation (alias of location)

Remove KeyboardEvent.prototype.keyLocation (alias of location)

Intent to Remove:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/lqknEaUYCJM/UbNahDDMAwAJ

BUG=568261
Committed: https://crrev.com/263d94f9b2dc4a0a3cf5b3124c1cc8d2163a4410
Cr-Commit-Position: refs/heads/master@{#375759}

[philipj_slow, 2016-02-16 19:03:02]

The CQ bit was checked by philipj@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-02-16 19:06:49]

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

[philipj_slow, 2016-02-16 19:35:35]

philipj@opera.com changed reviewers:
+ tkent@chromium.org

[philipj_slow, 2016-02-16 19:35:37]

PTAL

[commit-bot: I haz the power, 2016-02-16 20:21:15]

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

[commit-bot: I haz the power, 2016-02-16 20:21:15]

Dry run: This issue passed the CQ dry run.

[tkent, 2016-02-16 23:33:54]

lgtm

https://codereview.chromium.org/1700233002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/testing/InternalDictionary.idl (right):

https://codereview.chromium.org/1700233002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/testing/InternalDictionary.idl:12:
[DeprecateAs=PrefixedStorageInfo, ImplementedAs=longMember] long
deprecateLongMember;
ok, anything to be deprecated is acceptable for this.

[philipj_slow, 2016-02-17 01:59:43]

https://codereview.chromium.org/1700233002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/testing/InternalDictionary.idl (right):

https://codereview.chromium.org/1700233002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/testing/InternalDictionary.idl:12:
[DeprecateAs=PrefixedStorageInfo, ImplementedAs=longMember] long
deprecateLongMember;
On 2016/02/16 23:33:54, tkent wrote:
> ok, anything to be deprecated is acceptable for this.

Yep, I was a bit surprised to find this. We'll just have to keep changing it as
deprecated things are removed, or come up with a better way for testing the same
thing.

[philipj_slow, 2016-02-17 01:59:50]

The CQ bit was checked by philipj@opera.com

[commit-bot: I haz the power, 2016-02-17 02:00:25]

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

[commit-bot: I haz the power, 2016-02-17 02:13:56]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-17 02:15:28]

Description was changed from

==========
Remove KeyboardEvent.prototype.keyLocation (alias of location)

Intent to Remove:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/lqknEaUYCJM/UbNahDDM...

BUG=568261
==========

to

==========
Remove KeyboardEvent.prototype.keyLocation (alias of location)

Intent to Remove:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/lqknEaUYCJM/UbNahDDM...

BUG=568261

Committed: https://crrev.com/263d94f9b2dc4a0a3cf5b3124c1cc8d2163a4410
Cr-Commit-Position: refs/heads/master@{#375759}
==========

[commit-bot: I haz the power, 2016-02-17 02:15:28]

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