The default bug view has changed. See this FAQ.

Release message manager related data structures more aggressively

RESOLVED FIXED

Status

()

Core
DOM
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: smaug, Assigned: smaug)

Tracking

12 Branch
x86_64
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

Created attachment 585288 [details] [diff] [review]
patch

This is part of my reduce-CC-times effort.
Attachment #585288 - Flags: review?(jst)

Updated

5 years ago
Attachment #585288 - Flags: review?(jst) → review+
Created attachment 586486 [details] [diff] [review]
additional patch

Silly me, I should read my own code better.

The JSContext swap is actually missing there even now, but it wasn't
causing real problems since mContext was set to null in Disconnect
and that just makes MM to use the implicit context or SafeCx.

RemoveFromParent() is added to keep the old behavior for tab swapping case.

https://tbpl.mozilla.org/?tree=Try&rev=ed99f63154c6
Attachment #586486 - Flags: review?(jst)

Updated

5 years ago
Attachment #586486 - Flags: review?(jst) → review+
https://hg.mozilla.org/mozilla-central/rev/39a83f2080dc
https://hg.mozilla.org/mozilla-central/rev/4052f4936e1c
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.