Unhandled promise rejections are no longer reported for DOM exceptions

RESOLVED FIXED in Firefox 63

Status

()

enhancement
P2
normal
RESOLVED FIXED
11 months ago
4 months ago

People

(Reporter: kmag, Assigned: kmag)

Tracking

unspecified
mozilla63
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox63 fixed)

Details

Attachments

(1 attachment)

Apparently this was fixed once before, but regressed when we switched to Spidermonkey promises, which I unfortunately did not notice until after I fixed it again.

Our unhandled promise rejection reporter tries to avoid undefined behavior by only reporting rejections for native error objects, so that it doesn't have to access JS properties which might run arbitrary code or bring dying objects back to life. Unfortunately, it only currently handles native JS ErrorObjects, and not DOM/XPC Exceptions.

We probably don't really need this anti-sniffing behavior anymore, since we no longer report unhandled rejections during finalization. But it's just as easy to special case DOM/XPC Exceptions as it is to remove those guards, so I'm going the safe route for now.
Priority: -- → P2
Comment on attachment 9003912 [details]
Bug 1486130: Fix reporting of unhandled DOMException rejection values. r=smaug

Olli Pettay [:smaug] has approved the revision.
Attachment #9003912 - Flags: review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/7ccc1c8a7abeefd54bc4cf9a5d0eef23681c924a
Bug 1486130: Fix reporting of unhandled DOMException rejection values. r=smaug
https://hg.mozilla.org/integration/mozilla-inbound/rev/ee4af6d31e69542f53aee91cb126e3d2a85691de
Bug 1486130: Follow-up: Ignore extra console message on OS-X. r=bustage,test-only
https://hg.mozilla.org/mozilla-central/rev/7ccc1c8a7abe
https://hg.mozilla.org/mozilla-central/rev/ee4af6d31e69
Status: NEW → RESOLVED
Closed: 11 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Depends on: 1528319
No longer depends on: 1528319
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.