Release message manager related data structures more aggressively

RESOLVED FIXED

Status

()

defect
RESOLVED FIXED
8 years ago
3 months ago

People

(Reporter: smaug, Assigned: smaug)

Tracking

12 Branch
x86_64
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

Assignee

Description

8 years ago
Posted patch patchSplinter Review
This is part of my reduce-CC-times effort.
Attachment #585288 - Flags: review?(jst)
Attachment #585288 - Flags: review?(jst) → review+
Assignee

Comment 1

8 years ago
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+
Assignee

Comment 2

8 years ago
https://hg.mozilla.org/mozilla-central/rev/39a83f2080dc
https://hg.mozilla.org/mozilla-central/rev/4052f4936e1c
Status: NEW → RESOLVED
Closed: 8 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.