1841967 - Stop using context in React components where it isn't relevant

Status: RESOLVED FIXED

(DevTools :: Debugger, task)

Description

[Alexandre Poirot [:ochameau] [PTO back Jan 5th]]

Yet another clone of bug 1841041 to continue this work while helping keep track of patches landed in which version.

Comment 1

[Alexandre Poirot [:ochameau] [PTO back Jan 5th]]

Attachment: Bug 1841967 - [devtools] Remove context from override source actions.

These actions take a source as input, but at the end we end up setting overrides
against URLs which aren't specific to any particular page/pause location.

Comment 2

[Alexandre Poirot [:ochameau] [PTO back Jan 5th]]

Attachment: Bug 1841967 - [devtools] Remove leftover "cx" variable from preview and command actions.

Comment 3

[Alexandre Poirot [:ochameau] [PTO back Jan 5th]]

Attachment: Bug 1841967 - [devtools] Align breakOnNext with step and resume actions.

Step and resume actions (command action) are acting on the currently selected thread,
so that there is no need to pass a context. This action could also query the selectors
to guess what is the currently selected thread.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0490f94b41d9
[devtools] Remove context from override source actions. r=devtools-reviewers,bomsy
https://hg.mozilla.org/integration/autoland/rev/05db01d0d1b9
[devtools] Remove leftover "cx" variable from preview and command actions. r=devtools-reviewers,bomsy
https://hg.mozilla.org/integration/autoland/rev/c5910091179c
[devtools] Align breakOnNext with step and resume actions. r=devtools-reviewers,bomsy

Comment 5

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/0490f94b41d9
https://hg.mozilla.org/mozilla-central/rev/05db01d0d1b9
https://hg.mozilla.org/mozilla-central/rev/c5910091179c