Closed
Bug 205104
Opened 21 years ago
Closed 21 years ago
Inspector leaks nsIAtoms
Categories
(Other Applications :: DOM Inspector, defect)
Other Applications
DOM Inspector
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: caillon, Assigned: caillon)
Details
(Keywords: memory-leak)
Attachments
(1 file)
|
876 bytes,
patch
|
jst
:
review+
jst
:
superreview+
sspitzer
:
approval1.4+
|
Details | Diff | Splinter Review |
|
Assignee | |
Comment 1•21 years ago
|
||
Also fixes us to not be silly and try and construct the binding uri ourselves, when we can just get it right from the binding itself (less virtual calls).
|
|
Assignee | |
Updated•21 years ago
|
Attachment #122895 -
Flags: superreview?(jst)
Attachment #122895 -
Flags: review?(jst)
|
||
Comment 2•21 years ago
|
||
Comment on attachment 122895 [details] [diff] [review] Patch r+sr=jst
Attachment #122895 -
Flags: superreview?(jst)
Attachment #122895 -
Flags: superreview+
Attachment #122895 -
Flags: review?(jst)
Attachment #122895 -
Flags: review+
|
|
Assignee | |
Comment 3•21 years ago
|
||
Comment on attachment 122895 [details] [diff] [review] Patch Inspector-only, this small patch fixes some leaks and cleans up our logic a little bit. Very little risk.
Attachment #122895 -
Flags: approval1.4?
|
||
Comment 4•21 years ago
|
||
Comment on attachment 122895 [details] [diff] [review] Patch a=sspitzer
Attachment #122895 -
Flags: approval1.4? → approval1.4+
|
|
Assignee | |
Comment 5•21 years ago
|
||
Fixed
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
|
||
Updated•20 years ago
|
Product: Core → Other Applications
|
||
Updated•17 years ago
|
QA Contact: timeless → dom-inspector
You need to log in
before you can comment on or make changes to this bug.
Description
•