Closed Bug 1530554 Opened 2 years ago Closed 2 years ago

Evaluating expressions no longer works in worker threads


(DevTools :: Debugger, defect)

Not set


(firefox67 fixed)

Firefox 67
Tracking Status
firefox67 --- fixed


(Reporter: bhackett1024, Assigned: ystartsev)




(2 files)

Attached patch mochitest patchSplinter Review

I think bug 1485664 broke the ability to evaluate expressions in workers. The debugger uses the console's evaluateJSAsync request for this. Before these patches, each worker was associated with its own console actor which could evaluate expressions in that worker's thread, but after the patch in each worker is associated with the same console actor which the main thread uses.

Unfortunately there was no test for this functionality (mea culpa, windowless workers are pretty new and the tests are in poor shape). I've attached a patch which updates a mochitest to exercise this. This test passes before these patches, and fails afterwards.

I'm not sure how to go about fixing this. Is there still a separate console actor associated with each thread? From the changes in D20150 it seems like there isn't, but it's hard for me to follow the changes that are happening in this bug.

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

Hi Brian,

Thanks for the patch, the issue is here:

it should be workerTargetFront.getFront("console") instead of tabTarget.getFront("console")

I am currently in a workshop but I can correct it when i have some free minutes

Flags: needinfo?(ystartsev)

I have a patch that addresses the test case.

Flags: needinfo?(poirot.alex)
Assignee: nobody → ystartsev
Pushed by
Fix expression evaluation in worker threads; r=bhackett
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 67
You need to log in before you can comment on or make changes to this bug.