Closed Bug 17959 Opened 25 years ago Closed 25 years ago

[mlk] nsBrowserInstance::ClearHistoryPopup() leaks nsXULElements

Categories

(SeaMonkey :: General, defect, P1)

Product:
SeaMonkey
Component:
General
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: waterson, Assigned: waterson)

References

(
URL
)

Details

Attachments

(1 file)

proposed fix
(deleted), patch
Details | Diff | Splinter Review
Fixed by replacing implementation with nsCOMPtr's. See attached diffs. This accounts for the remaining content nodes that were leaked on startup/shutdown. Yay! radha/law, please review!
Status: NEW → ASSIGNED
Priority: P3 → P1
Target Milestone: M12
Attached patch proposed fix (deleted) — — Splinter Review
These are OK, also. Good work, Chris.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Adding verifyme keyword.
Keywords: verifyme
Status: RESOLVED → VERIFIED
Marking Verified.
Product: Browser → Seamonkey
Keywords: verifyme
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: