Closed Bug 714633 Opened 9 years ago Closed 9 years ago

Release message manager related data structures more aggressively

Categories

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

12 Branch
x86_64
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: smaug, Assigned: smaug)

Details

Attachments

(2 files)

Attached patch patchSplinter Review
This is part of my reduce-CC-times effort.
Attachment #585288 - Flags: review?(jst)
Attachment #585288 - Flags: review?(jst) → review+
Attached patch additional patchSplinter Review
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)
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
Closed: 9 years ago
Resolution: --- → FIXED
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.