[FIX]DOMNodeInserted/DOMNodeRemoved listener checks use wrong node

RESOLVED FIXED in mozilla1.9alpha1

Status

()

Core
DOM: Events
RESOLVED FIXED
11 years ago
10 years ago

People

(Reporter: bz, Assigned: bz)

Tracking

({dev-doc-complete})

Trunk
mozilla1.9alpha1
x86
Linux
dev-doc-complete
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Assignee)

Description

11 years ago
Created attachment 251662 [details]
Testcase -- there should be 4 alerts

We should be looking for listeners on the kid, not the parent.
(Assignee)

Comment 1

11 years ago
Created attachment 251665 [details] [diff] [review]
Fix
Attachment #251665 - Flags: superreview?(bugmail)
Attachment #251665 - Flags: review?(bugmail)
Duplicate of this bug: 367160
Attachment #251665 - Flags: superreview?(bugmail)
Attachment #251665 - Flags: superreview+
Attachment #251665 - Flags: review?(bugmail)
Attachment #251665 - Flags: review+
(Assignee)

Comment 3

11 years ago
Fixed.
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Keywords: dev-doc-needed
Resolution: --- → FIXED
http://lxr.mozilla.org/mozilla/source/testing/mochitest/tests/test_bug367164.html
Flags: in-testsuite+

Comment 5

10 years ago
Testcase is not valid, because capturing listeners shouldn't fire on target (Bug 235441).
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)?
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.