[Media Router WebUI] Update ESC handling.

[Media Router WebUI] Update ESC handling.

ESC should be handled when pressed independently from other keys. Otherwise, we'll be overriding combinations such as shift+esc, which are keyboard shortcuts.

BUG=592422
Committed: https://crrev.com/96b1b55fa2851c0dce144badc3e29c9b63e2fccd
Cr-Commit-Position: refs/heads/master@{#381827}

[apacible, 2016-03-15 20:48:25]

Patchset #1 (id:1) has been deleted

[apacible, 2016-03-15 20:49:17]

Description was changed from

==========
remove dialog close on esc

BUG=592422
==========

to

==========
[Media Router WebUI] Fix ESC handling.

BUG=592422
==========

[apacible, 2016-03-15 20:59:31]

Description was changed from

==========
[Media Router WebUI] Fix ESC handling.

BUG=592422
==========

to

==========
[Media Router WebUI] Update ESC handling.

ESC should be handled when pressed independently from other keys. Otherwise,
we'll be overriding combinations such as shift+esc, which are keyboard
shortcuts.

BUG=592422
==========

[apacible, 2016-03-15 20:59:32]

apacible@chromium.org changed reviewers:
+ imcheng@chromium.org

[apacible, 2016-03-15 21:01:46]

PTAL, thanks!

Tested on mac and linux.

[imcheng, 2016-03-17 18:26:12]

lgtm

[apacible, 2016-03-17 22:09:18]

The CQ bit was checked by apacible@chromium.org

[apacible, 2016-03-17 22:09:18]

The patchset sent to the CQ was uploaded after l-g-t-m from imcheng@chromium.org
Link to the patchset: https://codereview.chromium.org/1800213002/#ps40001
(title: "Rebase.")

[commit-bot: I haz the power, 2016-03-17 22:09:51]

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

[commit-bot: I haz the power, 2016-03-17 23:11:51]

Description was changed from

==========
[Media Router WebUI] Update ESC handling.

ESC should be handled when pressed independently from other keys. Otherwise,
we'll be overriding combinations such as shift+esc, which are keyboard
shortcuts.

BUG=592422
==========

to

==========
[Media Router WebUI] Update ESC handling.

ESC should be handled when pressed independently from other keys. Otherwise,
we'll be overriding combinations such as shift+esc, which are keyboard
shortcuts.

BUG=592422
==========

[commit-bot: I haz the power, 2016-03-17 23:11:52]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-03-17 23:13:08]

Description was changed from

==========
[Media Router WebUI] Update ESC handling.

ESC should be handled when pressed independently from other keys. Otherwise,
we'll be overriding combinations such as shift+esc, which are keyboard
shortcuts.

BUG=592422
==========

to

==========
[Media Router WebUI] Update ESC handling.

ESC should be handled when pressed independently from other keys. Otherwise,
we'll be overriding combinations such as shift+esc, which are keyboard
shortcuts.

BUG=592422

Committed: https://crrev.com/96b1b55fa2851c0dce144badc3e29c9b63e2fccd
Cr-Commit-Position: refs/heads/master@{#381827}
==========

[commit-bot: I haz the power, 2016-03-17 23:13:10]

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