1928510 - Remove now-unused codepath related to debugging WebExtension via WebExtensionDescriptor.getTarget

Status: RESOLVED FIXED

(DevTools :: General, task)

Description

[Alexandre Poirot [:ochameau]]

Bug 1754452 implemented a new way of debugging WebExtension via the Watcher actor instead of via a unique Target fetched via Descriptor.getTarget.
We can remove a few functions thanks to this.

Comment 1

[Alexandre Poirot [:ochameau]]

Attachment: Bug 1928510 - [devtools] Remove now-unused WebExtensionTargetActor.

WebExtensionTargetActor.getTarget is being removed and removes most of the now-useless
codebase involving connectToFrame and message manager.

This allows to simplify a bit the ParentProcessTargetActor which required
some tweaks to be subclassed correctly.

We can also remove the consoleAPIListenerOptions which was specific to
this special target actor retrieving all document's console messages.
Now each individual target actor, for each document will handle the messages distinctly.

Note that we weren't using WebExtensionDescriptorFront.connect.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8170c986c851
[devtools] Remove now-unused WebExtensionTargetActor. r=devtools-reviewers,devtools-backward-compat-reviewers,nchevobbe

Comment 3

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/8170c986c851