Closed Bug 1638533 Opened 5 years ago Closed 5 years ago

Show (in promise) for uncaught promise rejections

Categories

(DevTools :: Console, enhancement)

enhancement

Tracking

(firefox78 fixed)

RESOLVED FIXED
Firefox 78
Tracking Status
firefox78 --- fixed

People

(Reporter: evilpie, Assigned: nchevobbe)

References

Details

Attachments

(1 file)

Nicolas said he wanted to add "(in promise)" to uncaught promise rejections logged to the console like Chrome. With my patch in bug 1636590, nsIScriptError now has isPromiseRejection that could be used by the devtools team to implement this.

Assignee: nobody → nchevobbe
Status: NEW → ASSIGNED

Now that nsIScriptError have a isPromiseRejection flag, we can consume it
in the console to indicate to the user that we're dealing with a promise rejection.
The exception message is then: "Uncaught (in promise) XXX".
A few tests needed to be updated, and we take this as an opportunity to
ensure we have a stacktrace for uncaught promise rejection.

Pushed by nchevobbe@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/bc1d9c3a3167 Add (in promise) suffix for uncaught rejected promise. r=davidwalsh.
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 78
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: