Fix keyboard commands on bookmarks page.

Fix keyboard commands on bookmarks page.

Appears I missed a set of hotkey commands on the bookmark page when
removing keyIdentifier and supporting KeyboardEvent.key. I've
verified U+007F is no longer used via a grep.

Remove some dead code too.

BUG=627364
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/7185ed0ef16da5c97142491d4c6d07df5c4e250b
Cr-Commit-Position: refs/heads/master@{#404902}

[dtapuska, 2016-07-12 13:12:02]

Description was changed from

==========
Fix keyboard commands on bookmarks page.

Appears I missed a set of hotkey commands on the bookmark page when
removing keyIdentifier and supporting KeyboardEvent.key. I've
verified U+007F is no longer used via a grep.

Remove some dead code too.

BUG=627364
==========

to

==========
Fix keyboard commands on bookmarks page.

Appears I missed a set of hotkey commands on the bookmark page when
removing keyIdentifier and supporting KeyboardEvent.key. I've
verified U+007F is no longer used via a grep.

Remove some dead code too.

BUG=627364
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[dtapuska, 2016-07-12 13:12:44]

dtapuska@chromium.org changed reviewers:
+ dbeam@chromium.org

[Dan Beam, 2016-07-12 19:59:30]

https://codereview.chromium.org/2140853003/diff/1/chrome/browser/resources/md...
File chrome/browser/resources/md_downloads/crisper.js (left):

https://codereview.chromium.org/2140853003/diff/1/chrome/browser/resources/md...
chrome/browser/resources/md_downloads/crisper.js:1702: * Chrome uses an older
version of DOM Level 3 Keyboard Events
this is just gonna get replaced the next time we vulcanize the downloads page. 
this is from polymer somewhere.  it'd be hard to remove it.

[dtapuska, 2016-07-12 20:04:05]

On 2016/07/12 19:59:30, Dan Beam wrote:
>
https://codereview.chromium.org/2140853003/diff/1/chrome/browser/resources/md...
> File chrome/browser/resources/md_downloads/crisper.js (left):
> 
>
https://codereview.chromium.org/2140853003/diff/1/chrome/browser/resources/md...
> chrome/browser/resources/md_downloads/crisper.js:1702: * Chrome uses an older
> version of DOM Level 3 Keyboard Events
> this is just gonna get replaced the next time we vulcanize the downloads page.

> this is from polymer somewhere.  it'd be hard to remove it.

That is fine the polymer code looks for key property first. I guess the change
before shouldn't have been done to this file that made the orphaned code. What
do you want me to do? Just remove that change from this CL?

[Dan Beam, 2016-07-12 21:03:08]

yeah, just revert that.  it's auto pulled + generated from polymer code :/

[dtapuska, 2016-07-12 21:17:08]

On 2016/07/12 21:03:08, Dan Beam wrote:
> yeah, just revert that.  it's auto pulled + generated from polymer code :/

done

[Dan Beam, 2016-07-12 22:33:56]

lgtm

[dtapuska, 2016-07-12 22:49:01]

The CQ bit was checked by dtapuska@chromium.org

[commit-bot: I haz the power, 2016-07-12 22:50:47]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-13 02:19:28]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-07-13 02:21:21]

Description was changed from

==========
Fix keyboard commands on bookmarks page.

Appears I missed a set of hotkey commands on the bookmark page when
removing keyIdentifier and supporting KeyboardEvent.key. I've
verified U+007F is no longer used via a grep.

Remove some dead code too.

BUG=627364
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
Fix keyboard commands on bookmarks page.

Appears I missed a set of hotkey commands on the bookmark page when
removing keyIdentifier and supporting KeyboardEvent.key. I've
verified U+007F is no longer used via a grep.

Remove some dead code too.

BUG=627364
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/7185ed0ef16da5c97142491d4c6d07df5c4e250b
Cr-Commit-Position: refs/heads/master@{#404902}
==========

[commit-bot: I haz the power, 2016-07-13 02:21:23]

Patchset 2 (id:??) landed as
https://crrev.com/7185ed0ef16da5c97142491d4c6d07df5c4e250b
Cr-Commit-Position: refs/heads/master@{#404902}