Closed Bug 1526557 Opened 1 year ago Closed 1 year ago

Messages from log points should point to the right source/location


(DevTools :: Debugger, enhancement, P2)

67 Branch


(firefox67 fixed)

Firefox 67
Tracking Status
firefox67 --- fixed


(Reporter: Harald, Assigned: bhackett1024)



User Story

When seeing messages generated from a logpoint, I want to have the breakpoint’s source location referenced for the log, so that I can navigate to the file.


(1 file)

Logpoints right now generate logs with 'debugger eval code'.

They should reflect the source and location of the associated breakpoint.

Assignee: nobody → bhackett1024
Attached patch patchSplinter Review

This patch sends a console message with the right source information directly from the breakpoint handling code in the server, avoiding calling console.log directly.

Attachment #9046952 - Flags: review?(lsmyth)

Oh, doing things this way also fixes a bug I noticed where creating a logpoint that throws would pause execution as if it was a normal breakpoint. Instead, the thrown exception is logged.

Comment on attachment 9046952 [details] [diff] [review]

Review of attachment 9046952 [details] [diff] [review]:

It is unfortunate that this reaches through all these private fields to call `onConsoleAPICall`, but that's something we can always try to fix in the future.
Attachment #9046952 - Flags: review?(lsmyth) → review+
Pushed by
Specify correct source location in logpoint messages, r=lsmyth.

Backed out for failing xpcshell at test_framebindings-02.js

Push with failures:

Failure log:


Unfortunately, both bugs were pushed at once and I could not figure out which one is the real cause based on retriggers and revision info, reason I backed out both bugs.

Flags: needinfo?(bhackett1024)
Pushed by
Specify correct source location in logpoint messages, r=lsmyth.
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 67
Flags: needinfo?(bhackett1024)
You need to log in before you can comment on or make changes to this bug.