DevTools: allow focus to be restored to the drawer when dialog closes

DevTools: allow focus to be restored to the drawer after resize

BUG=609013
Committed: https://crrev.com/8a6c91ecf6b62e973e5b9cd4fb6e41735f1e9b1f
Cr-Commit-Position: refs/heads/master@{#395449}

[luoe, 2016-05-21 01:03:42]

Description was changed from

==========
DevTools: allow focus to be restored to the drawer after resize

BUG=609013
==========

to

==========
DevTools: allow focus to be restored to the drawer after resize

BUG=609013
==========

[luoe, 2016-05-21 01:03:42]

luoe@chromium.org changed reviewers:
+ lushnikov@chromium.org

[luoe, 2016-05-21 01:03:53]

Tests coming soon in another patch

[lushnikov, 2016-05-23 18:19:24]

lgtm

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

https://codereview.chromium.org/2002803002/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/ui/InspectorView.js:412: if
(this._drawer.hasFocus()) {
style 1: the if blocks with return statements should not have "else" statement
style 2: one-line blocks should be formatted without {}

if (this._drawer.hasFocus())
    return this._drawer.deafultFocusedElement();
return this._currentPanel ? this._currentPanel.defaultFocusedElement() : null;

[luoe, 2016-05-23 19:18:41]

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

https://codereview.chromium.org/2002803002/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/ui/InspectorView.js:412: if
(this._drawer.hasFocus()) {
On 2016/05/23 18:19:24, lushnikov wrote:
> style 1: the if blocks with return statements should not have "else" statement
> style 2: one-line blocks should be formatted without {}
> 
> if (this._drawer.hasFocus())
>     return this._drawer.deafultFocusedElement();
> return this._currentPanel ? this._currentPanel.defaultFocusedElement() : null;

Done.

[luoe, 2016-05-23 19:18:46]

The CQ bit was checked by luoe@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-05-23 19:19:03]

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

[commit-bot: I haz the power, 2016-05-23 22:59:51]

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

[commit-bot: I haz the power, 2016-05-23 22:59:52]

Dry run: This issue passed the CQ dry run.

[luoe, 2016-05-23 23:01:16]

The CQ bit was checked by luoe@chromium.org

[luoe, 2016-05-23 23:01:16]

The patchset sent to the CQ was uploaded after l-g-t-m from
lushnikov@chromium.org
Link to the patchset: https://codereview.chromium.org/2002803002/#ps40001
(title: "Address patch comments")

[commit-bot: I haz the power, 2016-05-23 23:02:14]

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

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

Description was changed from

==========
DevTools: allow focus to be restored to the drawer after resize

BUG=609013
==========

to

==========
DevTools: allow focus to be restored to the drawer after resize

BUG=609013
==========

[commit-bot: I haz the power, 2016-05-23 23:08:15]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-05-23 23:09:58]

Description was changed from

==========
DevTools: allow focus to be restored to the drawer after resize

BUG=609013
==========

to

==========
DevTools: allow focus to be restored to the drawer after resize

BUG=609013

Committed: https://crrev.com/8a6c91ecf6b62e973e5b9cd4fb6e41735f1e9b1f
Cr-Commit-Position: refs/heads/master@{#395449}
==========

[commit-bot: I haz the power, 2016-05-23 23:09:59]

Patchset 3 (id:??) landed as
https://crrev.com/8a6c91ecf6b62e973e5b9cd4fb6e41735f1e9b1f
Cr-Commit-Position: refs/heads/master@{#395449}