1497470 - Create a ContentProcessTargetFront front to match ContentProcessTargetActor

Status: RESOLVED FIXED

(DevTools :: Framework, enhancement, P2)

Description

[Alexandre Poirot [:ochameau]]

ContentProcessTarget codepath is very special on the client side.
The is the only one target where we never create any Client/Front class.
We always for throught generic methods of DebuggerClient.attachConsole/attachThread/... in order to call any of its method.

It is slightly similar to AddonTarget. We do instantiate an AddonClient from attachAddon, but we never really use it.

These two cases: content process and addon targets are the only two one where TabTarget is having a null `activeTab` property.

I think it will be easier to think about merging the target fronts with TabTarget, once TabTarget is always going to have a reference to the target front.

So, the goal of this bug would be to first introduce the ContentProcessTargetFront and have it somehow referenced by the TabTarget.
Bug 1497467 will so similarily with a convertion of AddonClient to AddonTargetFront.

Comment 1

[Alexandre Poirot [:ochameau]]

Attachment: Bug 1497470 - Create a ContentProcessTargetFront front to match ContentProcessTargetActor.

MozReview-Commit-ID: LFfG7WRHu3a

Comment 2

[Alexandre Poirot [:ochameau]]

Attachment: Bug 1497470 - Improve error logging when the browser content toolbox fails loading.

MozReview-Commit-ID: 2QJfxOMmmOl

Depends on D10189

Comment 3

[Alexandre Poirot [:ochameau]]

Try run:
https://treeherder.mozilla.org/#/jobs?repo=try&selectedJob=208433444&revision=401aa0e6e1aa5244aa0875d4a544ba4d8c692b02

This is based on top of bug 1499022, itself being on top of bug 1485661.

This patch adds one more case where activeTab isn't a "tab" but a target front.
This is only temporary. activeTab/target's front is meant to be merged with TabTarget. And TabTarget being merged with WorkerTarget.
Note that this is not the first time activeTab isn't a BrowsingContextTargetFront.
WorkerTarget also exposes an `activeTab` attribute which is a WorkerTargetFront.

The next steps here are:
* try merging WorkerTarget into TabTarget
* also set an activeTab for legacy addons, so that all targets are having a front being set in attach.

Then, we should be able to instantiate the target fronts directly from RootFront and pass the front as argument of TabTarget (instead of passing its "form"). And finally, we should be able to merge TabTarget with the fronts.

Comment 4

[Alexandre Poirot [:ochameau]]

Attachment: Bug 1497470 - Correctly type Targets.listWorkers request.

MozReview-Commit-ID: BfjRhhIDxiF

Depends on D10189

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e6e16b119518
Create a ContentProcessTargetFront front to match ContentProcessTargetActor. r=yulia
https://hg.mozilla.org/integration/autoland/rev/8b0695286be6
Correctly type Targets.listWorkers request. r=yulia
https://hg.mozilla.org/integration/autoland/rev/1ca9d01b116a
Improve error logging when the browser content toolbox fails loading. r=yulia

Comment 6

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/e6e16b119518
https://hg.mozilla.org/mozilla-central/rev/8b0695286be6
https://hg.mozilla.org/mozilla-central/rev/1ca9d01b116a