Fix LayoutTests that assume canplay, playing, and canplaythrough will only fire once.

Fix LayoutTests that assume canplay, playing, and canplaythrough will only fire once.

Several tests are written under the assumption that the canplay, playing,
and canplaythrough events will only ever fire once. While this is true
for the current implementation, the HTML5 spec allows these events to
fire multiple times during the life of an HTMLMediaElement. This
change just modifies the tests so that they properly reflect their
intent, which is to run some code the first time these events fire.
This frees us to change how we do readyState transitions, which cause
extra events to fire, without breaking these tests.

BUG=144683
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=152683

[acolwell GONE FROM CHROMIUM, 2013-06-18 21:35:07]

https://codereview.chromium.org/17395006/diff/1/LayoutTests/http/tests/media/...
File
LayoutTests/http/tests/media/media-source/webkitmediasource-state-changes-expected.txt
(left):

https://codereview.chromium.org/17395006/diff/1/LayoutTests/http/tests/media/...
LayoutTests/http/tests/media/media-source/webkitmediasource-state-changes-expected.txt:8:
EVENT(playing)
Removed because they weren't providing any extra value since the lines below
each instance are responding to the expected 'playing' event.

https://codereview.chromium.org/17395006/diff/1/LayoutTests/media/video-playe...
File LayoutTests/media/video-played-reset.html (right):

https://codereview.chromium.org/17395006/diff/1/LayoutTests/media/video-playe...
LayoutTests/media/video-played-reset.html:47: waitForEventOnce("canplay",
canplay);
This is needed now because of the waitForEventOnce() change in videoPlayedMain()
in video-played.js

https://codereview.chromium.org/17395006/diff/1/LayoutTests/media/video-seeki...
File LayoutTests/media/video-seeking-expected.txt (left):

https://codereview.chromium.org/17395006/diff/1/LayoutTests/media/video-seeki...
LayoutTests/media/video-seeking-expected.txt:24: RUN(video.currentTime = 2)
This was removed because the original code was allowing an extra seek to occur
right after it signalled the end of the tests. Changes to the test prevent the
extra seek that wasn't really testing anything new.

[adamk, 2013-06-18 21:45:21]

lgtm

[scherkus (not reviewing), 2013-06-18 22:10:35]

lgtm

[commit-bot: I haz the power, 2013-06-18 23:07:42]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/acolwell@chromium.org/17395006/5001

[commit-bot: I haz the power, 2013-06-19 01:56:08]

Change committed as 152683