1779284 - Fix duplicated call to thread actor method and avoid duplicated target front list in commands.js

Status: RESOLVED FIXED

(DevTools :: Debugger, defect, P3)

Description

[Alexandre Poirot [:ochameau]]

We store a duplicated list of target fronts in commands.js.
This duplicates the list already maintained in TargetCommand:
https://searchfox.org/mozilla-central/rev/9ad01b5eae48436e3bdb8df01be3973e683d5242/devtools/client/debugger/src/client/firefox/commands.js#388-398

This is even more confusing as the "current target" is using the TargetCommand's one:
https://searchfox.org/mozilla-central/rev/9ad01b5eae48436e3bdb8df01be3973e683d5242/devtools/client/debugger/src/client/firefox/commands.js#23-25

On top of this, this code is currently buggy and leads to call some thread actor methods twice for the top level target:
https://searchfox.org/mozilla-central/rev/9ad01b5eae48436e3bdb8df01be3973e683d5242/devtools/client/debugger/src/client/firefox/commands.js#99

Comment 1

[Alexandre Poirot [:ochameau]]

Attachment: Bug 1779284 - [devtools] Stop maintaining a duplicate of the target list in the debugger frontend.

Instead we can fully rely on TargetCommand.

This fixes duplicated calls made to top level thread actor.
This was only happening with toolbox not using the Watcher Actor.
i.e. the non-multiprocess Browser Toolbox, Browser Content Toolbox and may be worker toolboxes from about:debugging.

Comment 2

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:bomsy, could you have a look please?

For more information, please visit auto_nag documentation.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7fbc69cd1c9f
[devtools] Stop maintaining a duplicate of the target list in the debugger frontend. r=bomsy

Comment 4

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/7fbc69cd1c9f