Closed Bug 1177772 Opened 4 years ago Closed 4 years ago

Improve MessagePort skippability

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla41
Tracking Status
firefox41 --- fixed
firefox42 --- affected

People

(Reporter: mccr8, Assigned: baku)

Details

Attachments

(1 file)

It sounds like there's some kind of central thing (PBackground?) that keeps alive MessageChannel. When a MessageChannel is kept alive in this way, we should treat it and whatever window or whatever it keeps alive as being definitely alive (and thus not visited by the CC), so that we won't cause hideous CC pauses in cases like bug 1177727.
Err I think I mean MessagePort not MessageChannel.
Summary: Improve MessageChannel skippability → Improve MessagePort skippability
MessagePort extends DETH and DETH is already skippable, so overriding skippable methods in MessagePort should be rather easy - similar to XMLHttpRequest.
Assignee: nobody → amarchesini
Attached patch mp.patchSplinter Review
Attachment #8626625 - Flags: review?(bugs)
Attachment #8626625 - Flags: review?(bugs) → review+
https://hg.mozilla.org/mozilla-central/rev/ca58200f87f4
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.