1783813 - Unify debugger initialization and thread selection

Status: RESOLVED FIXED

(DevTools :: Debugger, enhancement)

Description

[Alexandre Poirot [:ochameau]]

We are currently updating more things when selecting a thread than when opening the debugger.
Let's use a unique codepath when it comes to handle a new top level target.

When we select a thread we do all this:
https://searchfox.org/mozilla-central/rev/c0bed29d643393af6ebe77aa31455f283f169202/devtools/client/debugger/src/actions/pause/commands.js#20-51
While we do only this when opening the debugger:
https://searchfox.org/mozilla-central/rev/c0bed29d643393af6ebe77aa31455f283f169202/devtools/client/debugger/src/actions/navigation.js#38-48

Also in the reducer we have two distinct action which should ideally have the same behavior, but this isn't guaranteed as they have distinct implementation:
https://searchfox.org/mozilla-central/rev/c0bed29d643393af6ebe77aa31455f283f169202/devtools/client/debugger/src/reducers/pause.js#236-242

Comment 1

[Alexandre Poirot [:ochameau]]

Attachment: Bug 1783813 - [devtools] Unify debugger initialization and thread selection.

We are updating more things when selecting a thread than when opening the debugger.
Let's use a unique codepath when it comes to handle a new top level target.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9a188205b2cb
[devtools] Unify debugger initialization and thread selection. r=bomsy

Comment 3

[Marian-Vasile Laza]

https://hg.mozilla.org/mozilla-central/rev/9a188205b2cb