Closed
Bug 266118
Opened 20 years ago
Closed 20 years ago
Comment and code do not agree in nsXFormsInputElement
Categories
(Core Graveyard :: XForms, defect)
Core Graveyard
XForms
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: allan, Assigned: bryner)
References
()
Details
Attachments
(1 file)
|
805 bytes,
patch
|
smaug
:
review+
darin.moz
:
superreview+
|
Details | Diff | Splinter Review |
nsXFormsInputElement::OnCreated() states: --- CUT --- // It's ok to keep a weak pointer to mElement. mElement will have an // owning reference to this object, so as long as we null out mElement in // OnDestroyed, it will always be valid. --- CUT --- But nsXFormsInputElement::OnDestroyed() does not contain |mElement = nsnull;| ...
|
Reporter | |
Comment 1•20 years ago
|
||
The patch in bug 271737, includes a fix for this
Depends on: 271737
|
|
Reporter | |
Updated•20 years ago
|
Status: NEW → ASSIGNED
|
|
Reporter | |
Comment 2•20 years ago
|
||
|
|
Reporter | |
Comment 3•20 years ago
|
||
Comment on attachment 168609 [details] [diff] [review] Fix Newest path for bug 271737, does not include patch for this anymore.
Attachment #168609 -
Flags: superreview?(darin)
Attachment #168609 -
Flags: review?(smaug)
|
||
Comment 4•20 years ago
|
||
Comment on attachment 168609 [details] [diff] [review] Fix This is ok. Another thing is that RemoveEventListener should actually happen in WillChangeDocument.
Attachment #168609 -
Flags: review?(smaug) → review+
|
||
Updated•20 years ago
|
Attachment #168609 -
Flags: superreview?(darin) → superreview+
|
|
Reporter | |
Comment 5•20 years ago
|
||
Checked in.
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
|
||
Updated•8 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•