Closed Bug 1362 Opened 26 years ago Closed 26 years ago

Leaking WebShell/DocLoaderImpl/DocumentBindInfo

Categories

(Core Graveyard :: Embedding: APIs, defect, P1)

x86
Windows NT
defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: troy, Assigned: rpotts)

Details

Rick, on HTML documents we're leaking the web shell, the doc loader, and the doc bind info objects. Part of the problem may be that there are circular references: web shell -> doc loader doc loader -> doc bind info doc bind info -> web shell I suppose this might be okay if the doc bind info was freed and that would break the circular reference. Problem is that doesn't seem to happen. The nsDocLoaderImpl::LoadURLComplete() member function is called and that reduces the ref count on the bind info object by one, but that doesn't cause it to be freed because the nsStreamListenerProxy object still holds a reference and it isn't getting freed. I'm a little out of my league at that point and hopefully it's obvious to you why the nsStreamListenerProxy isn't getting freed
Status: NEW → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
how did this get fixed? who fixed it?
Status: RESOLVED → REOPENED
also -- who would be the best person to verify fix -- Troy?
Status: REOPENED → RESOLVED
Closed: 26 years ago26 years ago
Resolved bug as fixed. rpotts says this is cool.
Status: RESOLVED → VERIFIED
QA Contact: 3849
based on developer input, marking verified
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.