In that test we try to play an empty <video> element. A timer is started after pause() has been called. I think it would be better to wait the pause event instead.
Created attachment 47024 [details] Refactored the test to use the pause event instead of a timer.
Comment on attachment 47024 [details] Refactored the test to use the pause event instead of a timer. > +<html> > +<body> This seems like a pointless change to me; including explicit HTML and body elements does not make our tests better. I assume it has no effect on the test. > + waitForEvent("pause", onpause); It seems to me we should call waitForEvent("pause") before calling video.play() and video.pause(). I would not have preserved the blank line at the end of the test created with consoleWrite. I would probably have used waitForEventTestAndEnd. Seems OK as is. r=me
Oh I didn't know about waitForEventTestAndEnd. Perfect fit for this test indeed ;)
Landed as r53550. Thanks for the review Darin!