Last Comment Bug 759745 - content/events/test/test_bug238987.html ran additional tests after finish() was called
: content/events/test/test_bug238987.html ran additional tests after finish() w...
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: Event Handling (show other bugs)
: Trunk
: All All
: -- normal (vote)
: mozilla15
Assigned To: Mats Palmgren (:mats)
:
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-05-30 07:23 PDT by Mats Palmgren (:mats)
Modified: 2012-06-02 14:05 PDT (History)
3 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
fix (1.56 KB, patch)
2012-05-30 07:26 PDT, Mats Palmgren (:mats)
philringnalda: review+
Details | Diff | Splinter Review

Description Mats Palmgren (:mats) 2012-05-30 07:23:10 PDT
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.
Comment 1 Mats Palmgren (:mats) 2012-05-30 07:26:53 PDT
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
Comment 2 Phil Ringnalda (:philor) 2012-05-30 08:41:56 PDT
Comment on attachment 628331 [details] [diff] [review]
fix

Sweet.
Comment 6 :Ms2ger (⌚ UTC+1/+2) 2012-06-02 13:31:54 PDT
Why not just document.querySelectorAll("[id]")?
Comment 7 Mats Palmgren (:mats) 2012-06-02 14:05:25 PDT
No reason, I just copy-pasted the loop with the addEventListener calls.

Note You need to log in before you can comment on or make changes to this bug.