PBackgroundChild actors leak other process HANDLEs

NEW
Unassigned

Status

()

Core
IPC
--
minor
4 years ago
4 years ago

People

(Reporter: Ben Turner (not reading bugmail, use the needinfo flag!), Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Created attachment 8544896 [details] [diff] [review]
Patch, v1

This wasn't a problem really before because PBackground was primarily used on the main thread but now that workers create PBackgroundChild actors we need to make sure that they clean up the HANDLEs to the parent process.
Attachment #8544896 - Flags: review?(mrbkap)

Updated

4 years ago
Attachment #8544896 - Flags: review?(mrbkap) → review+
Backed out in https://hg.mozilla.org/integration/mozilla-inbound/rev/b460db55cd53 - impressive just how badly Windows debug hates that patch, from the make check destruction of some xpcshell harness test to every mochitest going crazy crashing and ABORTing as many times as it can manage.
You need to log in before you can comment on or make changes to this bug.