Closed
Bug 705272
Opened 13 years ago
Closed 13 years ago
[CC] Don't traverse the hidden window
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla11
People
(Reporter: mccr8, Assigned: smaug)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
|
(deleted),
patch
|
jst
:
review+
mccr8
:
feedback+
|
Details | Diff | Splinter Review |
It looks to me like the hidden window is created on startup, and lasts until shutdown. Thus we should not traverse it. This contains things (looks like maybe 500 DOM nodes?) and is traversed on OSX. It contains things on Windows, even though it is a different type of document, but I don't know if it is really traversed there. Hopefully this is doable by modifying nsCCUncollectableMarker. I'm not sure why it isn't caught by that already.
|
Assignee | |
Comment 1•13 years ago
|
||
Does the patch help+
Attachment #576929 -
Flags: feedback?(continuation)
|
Reporter | |
Comment 2•13 years ago
|
||
Comment on attachment 576929 [details] [diff] [review]
patch
This prevents the hidden window from being traversed, thanks!
Pushed to try: https://tbpl.mozilla.org/?tree=Try&rev=d1c5a9775853
It looked like that all documents were being traversed in the CC before the shutdown CC, which is odd, but I can't imagine this patch is at fault. I'll have to keep an eye on that.
Attachment #576929 -
Flags: feedback?(continuation) → feedback+
|
|
Assignee | |
Updated•13 years ago
|
Attachment #576929 -
Flags: review?(jst)
|
||
Updated•13 years ago
|
Attachment #576929 -
Flags: review?(jst) → review+
|
|
Assignee | |
Comment 3•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Updated•13 years ago
|
Assignee: nobody → bugs
|
|
Reporter | |
Updated•13 years ago
|
Target Milestone: --- → mozilla11
You need to log in
before you can comment on or make changes to this bug.
Description
•