Closed Bug 1532309 Opened 1 year ago Closed 1 year ago

Log points should show multiple values

Categories

(DevTools :: Debugger, enhancement, P3)

enhancement

Tracking

(firefox67 fixed)

RESOLVED FIXED
Firefox 67
Tracking Status
firefox67 --- fixed

People

(Reporter: jlast, Assigned: jlast)

References

Details

Attachments

(1 file)

console.log(1,2,3) will show all three values. Similarly the log point 1,2,3 should show all three values.

1526557 changed the default behavior from using console.log to evaluating the expression and using the return value.

One of the common usecases for multiple values is a log with a label and a value. e.g. addBreakpoint ${bp}

Severity: normal → enhancement
Priority: -- → P3

we could get this behavior by using an array to capture all of the values [${log}]

https://gist.github.com/e07155cced2f5f906c64010c2cb781ba

This could potentially have some issues in terms of error messages, but overall is fine.

This is the behavior from other log point implementations as well and helps to avoid string concatenations/templating, so +1 to align this; maybe even making this a stretch goal for M1.

Pushed by jlaster@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/8c6fc5c010f4
Log points should show multiple values. r=bhackett
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 67
Blocks: logpoints-m1
No longer blocks: dbg-logpoints
You need to log in before you can comment on or make changes to this bug.