[DevTools] Save connected worker ids to send "detached" notification on restore.

[DevTools] Save connected worker ids to send "detached" notification on restore.

So far we used MainFrameNavigated for that, which is unfortunate.
Now we store all worker ids reported to frontend in backend state, so that we can
report them as disconnected on cross-process navigation.

BUG=none

Review-Url: https://codereview.chromium.org/2736763002
Cr-Commit-Position: refs/heads/master@{#455332}
Committed: https://chromium.googlesource.com/chromium/src/+/3ce2eb3e3bfb42a0147b651a3d4e191285c4a3ec

[dgozman, 2017-03-06 19:14:49]

dgozman@chromium.org changed reviewers:
+ pfeldman@chromium.org

[dgozman, 2017-03-06 19:14:50]

What do you think about this approach?

[pfeldman, 2017-03-06 19:25:21]

lgtm

[dgozman, 2017-03-06 19:30:10]

The CQ bit was checked by dgozman@chromium.org

[commit-bot: I haz the power, 2017-03-06 19:30:55]

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-06 21:05:42]

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

[commit-bot: I haz the power, 2017-03-06 21:05:43]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[dgozman, 2017-03-06 22:01:32]

The CQ bit was checked by dgozman@chromium.org

[dgozman, 2017-03-06 22:01:32]

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

[commit-bot: I haz the power, 2017-03-06 22:02:36]

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-07 00:00:16]

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

[commit-bot: I haz the power, 2017-03-07 00:00:16]

Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[dgozman, 2017-03-07 00:31:33]

The CQ bit was checked by dgozman@chromium.org

[commit-bot: I haz the power, 2017-03-07 00:33:20]

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-07 03:53:15]

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

[commit-bot: I haz the power, 2017-03-07 03:53:16]

Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[dgozman, 2017-03-07 06:08:26]

The CQ bit was checked by dgozman@chromium.org

[commit-bot: I haz the power, 2017-03-07 06:09:35]

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-07 08:38:44]

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

[commit-bot: I haz the power, 2017-03-07 08:38:45]

Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[dgozman, 2017-03-07 16:01:51]

The CQ bit was checked by dgozman@chromium.org

[commit-bot: I haz the power, 2017-03-07 16:02:50]

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-07 19:24:10]

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

[commit-bot: I haz the power, 2017-03-07 19:24:11]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[dgozman, 2017-03-07 19:31:02]

The CQ bit was checked by dgozman@chromium.org

[commit-bot: I haz the power, 2017-03-07 19:32:08]

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-08 01:46:11]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1488915062150610,
"parent_rev": "0573ada970c3acde0fcc51bf5f7a5c9be9e7629c", "commit_rev":
"3ce2eb3e3bfb42a0147b651a3d4e191285c4a3ec"}

[commit-bot: I haz the power, 2017-03-08 01:47:17]

Description was changed from

==========
[DevTools] Save connected worker ids to send "detached" notification on restore.

So far we used MainFrameNavigated for that, which is unfortunate.
Now we store all worker ids reported to frontend in backend state, so that we
can
report them as disconnected on cross-process navigation.

BUG=none
==========

to

==========
[DevTools] Save connected worker ids to send "detached" notification on restore.

So far we used MainFrameNavigated for that, which is unfortunate.
Now we store all worker ids reported to frontend in backend state, so that we
can
report them as disconnected on cross-process navigation.

BUG=none

Review-Url: https://codereview.chromium.org/2736763002
Cr-Commit-Position: refs/heads/master@{#455332}
Committed:
https://chromium.googlesource.com/chromium/src/+/3ce2eb3e3bfb42a0147b651a3d4e...
==========

[commit-bot: I haz the power, 2017-03-08 01:47:18]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/3ce2eb3e3bfb42a0147b651a3d4e...