Open Bug 1749763 Opened 3 years ago Updated 2 years ago

Can't set breakpoints on generated source while sourcemap are loading

Categories

(DevTools :: Debugger, defect, P2)

defect

Tracking

(Not tracked)

People

(Reporter: ochameau, Assigned: ochameau)

References

(Blocks 3 open bugs)

Details

Attachments

(2 files)

I wanted to write an integration test to replace the following unit test:
https://searchfox.org/mozilla-central/source/devtools/client/debugger/src/actions/sources/tests/newSources.spec.js#134-172
Where we assert that a slow loading sourcemap doesn't delay loading the other ones.

But while writing this integration test, I realized that things were broken while sourcemap were loading. You could set a breakpoint on the generated source while the sourcemap was loading. And breakpoints weren't fully functional even if you set them beforehand.

This goal of this bug is to provide this integration test and try to fix the underlying issues, if possible.

Assignee: nobody → poirot.alex
Blocks: 1749767
Blocks: 1667026

We had some unit tests to cover slow-loading source maps,
but they were only covering sources management.
I'm converting them to an integration test, so that we can assert
that the debugger is fully function while the source maps are loading.

We were blocking breakpoint setting and hitting code on the calls to mapLocation/updateFrameLocation,
which were waiting for the sourcemap to load.
Instead we should consider setting the breakpoint only on the generated location.
And when we hit a breakpoint, we should only try to map the frame if
the generated source has an original source already registered.

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

For more information, please visit auto_nag documentation.

Flags: needinfo?(odvarko)
Severity: -- → S3
Flags: needinfo?(odvarko)
Priority: -- → P2
Attachment #9258698 - Attachment description: Bug 1749763 - [devtools] Cover slow loading source maps → Bug 1749763 - [devtools] Cover slow loading source maps.

There are some r+ patches which didn't land and no activity in this bug for 2 weeks.
:ochameau, could you have a look please?
If you still have some work to do, you can add an action "Plan Changes" in Phabricator.
For more information, please visit auto_nag documentation.

Flags: needinfo?(poirot.alex)
Flags: needinfo?(hmanilla)

Fixing this particular issue introduces more issues as highligted on phabricator.
We will get back to this after having simplified the debugger some more in bug 1749767 project.

Flags: needinfo?(poirot.alex)
Flags: needinfo?(hmanilla)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: