1532309 - Log points should show multiple values

Status: RESOLVED FIXED

(DevTools :: Debugger, enhancement, P3)

Description

[Jason Laster [:jlast]]

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}

Comment 1

[Jason Laster [:jlast]]

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.

Comment 2

[:Harald Kirschner :digitarald]

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.

Comment 3

[Jason Laster [:jlast]]

Attachment: Bug 1532309 - Log points should show multiple values. r=bhackett

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8c6fc5c010f4
Log points should show multiple values. r=bhackett

Comment 5

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/8c6fc5c010f4