1829688 - Don't always select original source on pause

Status: RESOLVED FIXED

(DevTools :: Debugger, defect, P3)

Description

[Alexandre Poirot [:ochameau]]

For now, when we pause, the debugger frontend would always force selecting the original source. That, even if we were previously selecting a generated file.
This goes against the overall goal being done elsewhere to "keep context", and select the same type of source as the currently selected one (original if original is selected, or generatd otherwise).

We could still try to open the original source if no source is currently selected,
but when a source is already selected, try to keep the same context.

Comment 1

[Alexandre Poirot [:ochameau]]

Attachment: Bug 1829688 - [devtools] Avoid selecting the original source everytime we hit a breakpoint.

Instead, if we were selecting a generated source, we would keep selecting generated source.
If there is no selected source, we would still try to select the original source.

Many tests were relying on this behavior.

WASM test was failing because of errors with the parser worker.
I added early bailout in order to prevent trying to use the parser worker for WASM.

shared-head test helper were also tweaked to better support select WASM binary sources.

Comment 2

[Alexandre Poirot [:ochameau]]

Attachment: Bug 1829688 - [devtools] Avoid querying the parser worker for unsupported sources.

The worker itself ignores already the WASM sources,
but we could save some cycle to avoid calling the parserWorker entirely
when we know the source won't be supported by it.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/81fb6fc69cca
[devtools] Avoid querying the parser worker for unsupported sources. r=nchevobbe,devtools-reviewers
https://hg.mozilla.org/integration/autoland/rev/bb12f557adeb
[devtools] Avoid selecting the original source everytime we hit a breakpoint. r=devtools-reviewers,nchevobbe

Comment 4

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/81fb6fc69cca
https://hg.mozilla.org/mozilla-central/rev/bb12f557adeb

Comment 5

[Alexandre Poirot [:ochameau]]

Alongside bug 1634721 this improved stepping through original files and regular js modules. You context will be better preserved.