DevTools: keep selected tab as persistence binding comes in

DevTools: keep selected tab as persistence binding comes in

This patch fixes a bug in the TabbedEditorContainer's handler for
persistence binding.

As the binding comes in, we should retain selected tab no matter
what.

R=dgozman

Review-Url: https://codereview.chromium.org/2740053002
Cr-Commit-Position: refs/heads/master@{#456162}
Committed: https://chromium.googlesource.com/chromium/src/+/d9ab107ef6724e5fe194e6b7402086d41c187063

[lushnikov, 2017-03-09 08:10:19]

please, take a look

[lushnikov, 2017-03-09 08:45:31]

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

[commit-bot: I haz the power, 2017-03-09 08:45:51]

Dry run: 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, 2017-03-09 10:10:18]

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

[commit-bot: I haz the power, 2017-03-09 10:10:19]

Dry run: This issue passed the CQ dry run.

[lushnikov, 2017-03-10 19:16:39]

please, take another look!

[lushnikov, 2017-03-10 19:16:44]

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

[commit-bot: I haz the power, 2017-03-10 19:17:17]

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

[dgozman, 2017-03-10 19:23:38]

https://codereview.chromium.org/2740053002/diff/20001/third_party/WebKit/Sour...
File
third_party/WebKit/Source/devtools/front_end/sources/TabbedEditorContainer.js
(right):

https://codereview.chromium.org/2740053002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/devtools/front_end/sources/TabbedEditorContainer.js:94:
if (!fileSystemTabId) {
Do we have to move filsystem in place of network otherwise?

[dgozman, 2017-03-10 19:57:53]

lgtm

[lushnikov, 2017-03-10 20:06:51]

The CQ bit was unchecked by lushnikov@chromium.org

[lushnikov, 2017-03-10 20:06:51]

The CQ bit was checked by lushnikov@chromium.org

[commit-bot: I haz the power, 2017-03-10 20:07:38]

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, 2017-03-10 21:12:36]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1489176411781440,
"parent_rev": "e2a3dd0d63dd994aaad58f8818f0ed075b453834", "commit_rev":
"d9ab107ef6724e5fe194e6b7402086d41c187063"}

[commit-bot: I haz the power, 2017-03-10 21:13:22]

Description was changed from

==========
DevTools: keep selected tab as persistence binding comes in

This patch fixes a bug in the TabbedEditorContainer's handler for
persistence binding.

As the binding comes in, we should retain selected tab no matter
what.

R=dgozman
==========

to

==========
DevTools: keep selected tab as persistence binding comes in

This patch fixes a bug in the TabbedEditorContainer's handler for
persistence binding.

As the binding comes in, we should retain selected tab no matter
what.

R=dgozman

Review-Url: https://codereview.chromium.org/2740053002
Cr-Commit-Position: refs/heads/master@{#456162}
Committed:
https://chromium.googlesource.com/chromium/src/+/d9ab107ef6724e5fe194e6b74020...
==========

[commit-bot: I haz the power, 2017-03-10 21:13:23]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/d9ab107ef6724e5fe194e6b74020...