Promises leak AsyncErrorReporter

RESOLVED FIXED in mozilla29

Status

()

defect
RESOLVED FIXED
6 years ago
4 months ago

People

(Reporter: nsm, Assigned: nsm)

Tracking

unspecified
mozilla29
x86_64
Linux
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [qa-])

Attachments

(1 attachment)

The fix for Bug 887687 preserves wrappers, so that Promises that are around on shutdown will be destroyed during XPCOM shutdown. If the promise has been rejected, but the rejection not handled, the NS_DispatchToMainThread() call in Promise::MaybeReportRejected() fails, and the AsyncErrorReporter leaks.
Assignee: nobody → nsm.nikhil
Attachment #8358590 - Flags: review?(continuation) → review+
https://hg.mozilla.org/mozilla-central/rev/e7b3360cd07a
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
Whiteboard: [qa-]
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.