[Devtools] Esc closes searchview if focused and open

[Devtools] Esc closes searchview if focused and open

If ESC key is pressed while search input box is focused and open
it will now always close the search input box and stop propagation.

BUG=613232
R=lushnikov

Committed: https://crrev.com/2c9c1f684d0376f3be57284da16ab5c70ba0ff9b
Cr-Commit-Position: refs/heads/master@{#396363}

[allada, 2016-05-24 18:45:31]

PTL

[lushnikov, 2016-05-24 22:44:13]

https://codereview.chromium.org/2013473003/diff/1/third_party/WebKit/Source/d...
File third_party/WebKit/Source/devtools/front_end/ui/SearchableView.js (right):

https://codereview.chromium.org/2013473003/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/ui/SearchableView.js:420: if
(isEscKey(event)) {
How does this work in other places? E.g. while searching in sources, esc closes
search, and the next esc triggers drawer.

We should use the same approach here

[allada, 2016-05-25 22:45:54]

PTL

[lushnikov, 2016-05-26 00:07:36]

lgtm

Please, make sure sources/elements panel searches do not break with this change
(focus is tricky)

[allada, 2016-05-26 21:24:50]

The CQ bit was checked by allada@chromium.org

[commit-bot: I haz the power, 2016-05-26 21:25:43]

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

[commit-bot: I haz the power, 2016-05-27 02:43:49]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-27 02:45:24]

Description was changed from

==========
[Devtools] Esc closes searchview if focused and open

If ESC key is pressed while search input box is focused and open
it will now always close the search input box and stop propagation.

BUG=613232
R=lushnikov
==========

to

==========
[Devtools] Esc closes searchview if focused and open

If ESC key is pressed while search input box is focused and open
it will now always close the search input box and stop propagation.

BUG=613232
R=lushnikov

Committed: https://crrev.com/2c9c1f684d0376f3be57284da16ab5c70ba0ff9b
Cr-Commit-Position: refs/heads/master@{#396363}
==========

[commit-bot: I haz the power, 2016-05-27 02:45:26]

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