Closed
Bug 328344
Opened 19 years ago
Closed 19 years ago
action module elements leak
Categories
(Core Graveyard :: XForms, defect)
Core Graveyard
XForms
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: smaug, Assigned: smaug)
References
Details
(Keywords: fixed1.8.0.4, fixed1.8.1)
Attachments
(1 file)
5.54 KB,
patch
|
aaronr
:
review+
doronr
:
review+
|
Details | Diff | Splinter Review |
Currently action module elements use strong refs to DOM elements.
That is wrong. Should be weak ref.
Patch coming
Assignee | ||
Comment 1•19 years ago
|
||
Attachment #212931 -
Flags: review?(aaronr)
Assignee | ||
Updated•19 years ago
|
Attachment #212931 -
Flags: review?(doronr)
Updated•19 years ago
|
Attachment #212931 -
Flags: review?(doronr) → review+
Attachment #212931 -
Flags: review?(aaronr) → review+
Assignee | ||
Comment 2•19 years ago
|
||
Checked in to trunk
Checking in nsXFormsActionElement.cpp;
/cvsroot/mozilla/extensions/xforms/nsXFormsActionElement.cpp,v <-- nsXFormsActionElement.cpp
new revision: 1.5; previous revision: 1.4
done
Checking in nsXFormsActionElement.h;
/cvsroot/mozilla/extensions/xforms/nsXFormsActionElement.h,v <-- nsXFormsActionElement.h
new revision: 1.3; previous revision: 1.2
done
Checking in nsXFormsActionModuleBase.cpp;
/cvsroot/mozilla/extensions/xforms/nsXFormsActionModuleBase.cpp,v <-- nsXFormsActionModuleBase.cpp
new revision: 1.3; previous revision: 1.2
done
Checking in nsXFormsActionModuleBase.h;
/cvsroot/mozilla/extensions/xforms/nsXFormsActionModuleBase.h,v <-- nsXFormsActionModuleBase.h
new revision: 1.2; previous revision: 1.1
done
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Whiteboard: xf-to-branch
Assignee | ||
Updated•19 years ago
|
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Updated•19 years ago
|
Status: REOPENED → RESOLVED
Closed: 19 years ago → 19 years ago
Resolution: --- → FIXED
Updated•19 years ago
|
Keywords: fixed1.8.0.3,
fixed1.8.1
Updated•19 years ago
|
Whiteboard: xf-to-branch
Updated•9 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•