Don't always select original source on pause
Categories
(DevTools :: Debugger, defect, P3)
Tracking
(firefox114 fixed)
Tracking | Status | |
---|---|---|
firefox114 | --- | fixed |
People
(Reporter: ochameau, Assigned: ochameau)
References
(Blocks 2 open bugs)
Details
(Whiteboard: [devtools:relnote])
Attachments
(2 files)
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.
Assignee | ||
Comment 1•8 months ago
|
||
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.
Updated•8 months ago
|
Assignee | ||
Comment 2•8 months ago
|
||
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.
Updated•8 months ago
|
Pushed by apoirot@mozilla.com: 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•8 months ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/81fb6fc69cca
https://hg.mozilla.org/mozilla-central/rev/bb12f557adeb
Assignee | ||
Comment 5•6 months ago
|
||
Alongside bug 1634721 this improved stepping through original files and regular js modules. You context will be better preserved.
Description
•