Closed
Bug 1532309
Opened 6 years ago
Closed 6 years ago
Log points should show multiple values
Categories
(DevTools :: Debugger, enhancement, P3)
DevTools
Debugger
Tracking
(firefox67 fixed)
RESOLVED
FIXED
Firefox 67
Tracking | Status | |
---|---|---|
firefox67 | --- | fixed |
People
(Reporter: jlast, Assigned: jlast)
References
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
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}
Assignee | ||
Updated•6 years ago
|
Severity: normal → enhancement
Priority: -- → P3
Assignee | ||
Comment 1•6 years ago
|
||
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•6 years ago
|
||
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.
Assignee | ||
Comment 3•6 years ago
|
||
Pushed by jlaster@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/8c6fc5c010f4
Log points should show multiple values. r=bhackett
Comment 5•6 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox67:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 67
Updated•6 years ago
|
Assignee: nobody → jlaster
Updated•6 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•