Closed Bug 714633 Opened 13 years ago Closed 13 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+
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: