content/events/test/test_bug238987.html ran additional tests after finish() was called

RESOLVED FIXED in mozilla15

Status

()

Core
Event Handling
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: mats, Assigned: mats)

Tracking

Trunk
mozilla15
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Fallout from the diagnostic patch in bug 677964.

59333 INFO TEST-END | /tests/content/events/test/test_bug238987.html | finished in 600ms
59334 INFO TEST-START | /tests/content/events/test/test_bug288392.html
59335 INFO TEST-PASS | /tests/content/events/test/test_bug288392.html | |window| should not have got a blur event, target [object HTMLHeadingElement], id="begin"
59336 ERROR TEST-UNEXPECTED-FAIL | /tests/content/events/test/test_bug288392.html | [onunload] /tests/content/events/test/test_bug238987.html ran additional tests after finish() was called.
(Assignee)

Comment 1

5 years ago
Created attachment 628331 [details] [diff] [review]
fix

Remove the focus/blur event listeners that was added in start():
http://mxr.mozilla.org/mozilla-central/source/content/events/test/test_bug238987.html?force=1#143
Attachment #628331 - Flags: review?(philringnalda)
Comment on attachment 628331 [details] [diff] [review]
fix

Sweet.
Attachment #628331 - Flags: review?(philringnalda) → review+
(Assignee)

Comment 3

5 years ago
https://tbpl.mozilla.org/?usebuildbot=1&tree=Try&rev=b94c04bcd331
(Assignee)

Comment 4

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/3de76c58c5d1
Target Milestone: --- → mozilla15
https://hg.mozilla.org/mozilla-central/rev/3de76c58c5d1
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Why not just document.querySelectorAll("[id]")?
(Assignee)

Comment 7

5 years ago
No reason, I just copy-pasted the loop with the addEventListener calls.
You need to log in before you can comment on or make changes to this bug.