content/media/test/test_playback.html ran additional tests after finish() was called.

RESOLVED FIXED in mozilla15

Status

()

Core
Audio/Video
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Mats Palmgren (vacation - back in August), Assigned: Mats Palmgren (vacation - back in August))

Tracking

Trunk
mozilla15
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Fallout from the diagnostic patch in bug 677964.

There seems to be a "timeupdate" event coming in after we have
called SimpleTest.finish()

...
93297 INFO TEST-PASS | /tests/content/media/test/test_playback.html | Name should match test.name #2 - detodos.opus should equal detodos.opus
93298 INFO TEST-PASS | /tests/content/media/test/test_playback.html | detodos.opus checking readyState - 2 should equal 2
93299 INFO TEST-PASS | /tests/content/media/test/test_playback.html | detodos.opus shouldn't report NETWORK_LOADED
93300 INFO TEST-PASS | /tests/content/media/test/test_playback.html | detodos.opus checking playback has ended
93301 INFO TEST-PASS | /tests/content/media/test/test_playback.html | [finished detodos.opus-37] Length of array should match number of running tests - 0 should equal 0
93302 INFO TEST-INFO | /tests/content/media/test/test_playback.html | Finished at Tue May 29 2012 20:24:55 GMT-0700 (Pacific Daylight Time) (1338348295.298s)
93303 INFO TEST-INFO | /tests/content/media/test/test_playback.html | Running time: 38.02s
93304 INFO TEST-PASS | /tests/content/media/test/test_playback.html | Name should match test.name #4 - detodos.opus should equal detodos.opus
93305 INFO TEST-PASS | /tests/content/media/test/test_playback.html | detodos.opus duration (2.9135) should be around 2.9135
93306 INFO TEST-PASS | /tests/content/media/test/test_playback.html | detodos.opus time should run forwards: p=2.9135 c=2.9135
93307 INFO TEST-END | /tests/content/media/test/test_playback.html | finished in 38103ms
93308 INFO TEST-START | /tests/content/media/test/test_playback_errors.html
93309 ERROR TEST-UNEXPECTED-FAIL | /tests/content/media/test/test_playback_errors.html | [onunload] /tests/content/media/test/test_playback.html ran additional tests after finish() was called.
Created attachment 628367 [details] [diff] [review]
Remove the 'timeupdate' event listener when the test is done.
Attachment #628367 - Flags: review?(philringnalda)
https://tbpl.mozilla.org/?usebuildbot=1&tree=Try&rev=b94c04bcd331
Attachment #628367 - Flags: review?(philringnalda) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/326e101b7a6c
Target Milestone: --- → mozilla15
https://hg.mozilla.org/mozilla-central/rev/326e101b7a6c
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.