Closed
Bug 367164
Opened 18 years ago
Closed 18 years ago
[FIX]DOMNodeInserted/DOMNodeRemoved listener checks use wrong node
Categories
(Core :: DOM: Events, defect)
Tracking
()
RESOLVED
FIXED
mozilla1.9alpha1
People
(Reporter: bzbarsky, Assigned: bzbarsky)
References
Details
(Keywords: dev-doc-complete)
Attachments
(2 files)
|
1.32 KB,
application/vnd.mozilla.xul+xml
|
Details | |
|
5.91 KB,
patch
|
sicking
:
review+
sicking
:
superreview+
|
Details | Diff | Splinter Review |
We should be looking for listeners on the kid, not the parent.
|
Assignee | |
Comment 1•18 years ago
|
||
Attachment #251665 -
Flags: superreview?(bugmail)
Attachment #251665 -
Flags: review?(bugmail)
Attachment #251665 -
Flags: superreview?(bugmail)
Attachment #251665 -
Flags: superreview+
Attachment #251665 -
Flags: review?(bugmail)
Attachment #251665 -
Flags: review+
|
|
Assignee | |
Comment 3•18 years ago
|
||
Fixed.
|
||
Comment 4•18 years ago
|
||
http://lxr.mozilla.org/mozilla/source/testing/mochitest/tests/test_bug367164.html
Flags: in-testsuite+
|
||
Comment 5•17 years ago
|
||
Testcase is not valid, because capturing listeners shouldn't fire on target (Bug 235441).
|
||
Comment 6•17 years ago
|
||
This appears to be a pretty straightforward bug fix in material that's not presently documented in a meaningful way anyway. What sort of documentation is needed for this (I'm curious since the dev-doc-needed keyword is set)?
|
|
||
Comment 7•17 years ago
|
||
I've added a note of this bug fix to the Fx3 for developers page; tagging as doc-complete.
Keywords: dev-doc-needed → dev-doc-complete
You need to log in
before you can comment on or make changes to this bug.
Description
•