Errors thrown in workers are not being paused on
Categories
(DevTools :: Debugger, defect, P3)
Tracking
(firefox71 fixed)
Tracking | Status | |
---|---|---|
firefox71 | --- | fixed |
People
(Reporter: Harald, Assigned: bhackett1024)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
What were you doing?
- Open Debugger on https://firefox-devtools-tour.glitch.me/20190915/parity/#worker-debugging
- Make sure pause on exception is on
- Hit
Spawn
, thenMake Error
What happened?
Nothing.
What should have happened?
Thrown error should be paused on, as in Chrome.
Comment 1•5 years ago
|
||
Thanks Harald for nice test case!
I can easily repro that on my Win machine.
@Brian, could you please take a look at this?
Honza
Updated•5 years ago
|
Assignee | ||
Comment 2•5 years ago
|
||
There are a couple problems here. First, a small bug in the server (not having access to Cr in worker threads) causes us to not properly pause when the exception is thrown. Fixing that allows pausing on exceptions to work, but only when clicking the pause-on-exceptions checkbox after the worker has started. The second problem is that newly created workers are not given options reflecting the client's current pause-on-exceptions state (and other state, like skip-pausing). There is some code in toolbox.js with a bunch of options used when attaching to the main thread. I didn't want to copy-paste that stuff, so this patch adds plumbing so the debugger can get the default options for new threads from the toolbox. The latter uses the preferences state instead of reducer state, but since we update the prefs at the same time as the reducer this shouldn't be a problem, and workers should always be able to pause on exceptions as expected.
Assignee | ||
Comment 3•5 years ago
|
||
Here is the revision btw. I used moz-phab to submit this but the processing pipeline seems to have either failed somewhere or is just taking a really long time.
Assignee | ||
Comment 4•5 years ago
|
||
Pushed by bhackett@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/213ffedf9fc0 Allow pausing on exceptions in workers, r=jlast.
Comment 6•5 years ago
|
||
bugherder |
Updated•5 years ago
|
Description
•