Closed
Bug 759745
Opened 13 years ago
Closed 13 years ago
content/events/test/test_bug238987.html ran additional tests after finish() was called
Categories
(Core :: DOM: UI Events & Focus Handling, defect)
Core
DOM: UI Events & Focus Handling
Tracking
()
RESOLVED
FIXED
mozilla15
People
(Reporter: MatsPalmgren_bugz, Assigned: MatsPalmgren_bugz)
Details
Attachments
(1 file)
|
1.56 KB,
patch
|
philor
:
review+
|
Details | Diff | Splinter Review |
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•13 years ago
|
||
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 2•13 years ago
|
||
Comment on attachment 628331 [details] [diff] [review]
fix
Sweet.
Attachment #628331 -
Flags: review?(philringnalda) → review+
|
|
Assignee | |
Comment 3•13 years ago
|
||
|
|
Assignee | |
Comment 4•13 years ago
|
||
Target Milestone: --- → mozilla15
|
||
Comment 5•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
|
||
Comment 6•13 years ago
|
||
Why not just document.querySelectorAll("[id]")?
|
|
Assignee | |
Comment 7•13 years ago
|
||
No reason, I just copy-pasted the loop with the addEventListener calls.
|
||
Updated•6 years ago
|
Component: Event Handling → User events and focus handling
You need to log in
before you can comment on or make changes to this bug.
Description
•