Chromium Code Reviews Chromium Code Reviews
Issue 20114005:
Layout Test for basic MSE seek scenario. (Closed)
Base URL: https://chromium.googlesource.com/chromium/blink.git@master| Index: LayoutTests/http/tests/media/media-source/mediasource-play-then-seek-back.html |
| diff --git a/LayoutTests/http/tests/media/media-source/mediasource-play-then-seek-back.html b/LayoutTests/http/tests/media/media-source/mediasource-play-then-seek-back.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..051f2df13bf40ff217336ea861d000a72861a07d |
| --- /dev/null |
| +++ b/LayoutTests/http/tests/media/media-source/mediasource-play-then-seek-back.html |
| @@ -0,0 +1,80 @@ |
| +<!DOCTYPE html> |
| +<html> |
| + <head> |
| + <script src="/w3c/resources/testharness.js"></script> |
| + <script src="/w3c/resources/testharnessreport.js"></script> |
| + <script src="mediasource-util.js"></script> |
| + <script src="/media/resources/media-source/webm/segment-info.js"></script> |
| + <link rel="stylesheet" href="/w3c/resources/testharness.css"> |
| + </head> |
| + <body> |
| + <div id='log'></div> |
| + |
| + <script> |
| + |
| + function seekBackAfterPlayingTest(testFunction, description, options) |
| + { |
| + mediasource_test(function(test, mediaElement, mediaSource) |
| + { |
| + var segmentInfo = WebMSegmentInfo.testWebM; |
| + assert_equals(segmentInfo.duration, 6.042, 'Expected test media duration'); |
| + test.failOnEvent(mediaElement, 'error'); |
| + |
| + var sourceBuffer = mediaSource.addSourceBuffer(segmentInfo.type); |
| + MediaSourceUtil.loadBinaryData(test, segmentInfo.url, function(mediaData) |
| + { |
| + testFunction(test, mediaElement, mediaSource, segmentInfo, sourceBuffer, mediaData); |
| + }); |
| + }, description, options); |
| + } |
| + |
| + function getSegment(mediaData, info) |
| + { |
| + var start = info.offset; |
| + var end = start + info.size; |
| + return mediaData.subarray(start, end); |
| + } |
| + |
| + seekBackAfterPlayingTest(function(test, mediaElement, mediaSource, segmentInfo, sourceBuffer, mediaData) |
| + { |
| + |
| + mediaElement.play(); |
| + // Append all the segments |
| + test.expectEvent(sourceBuffer, 'updateend', 'sourceBuffer'); |
| + test.expectEvent(mediaElement, 'playing', 'Playing triggered'); |
| + sourceBuffer.appendBuffer(mediaData); |
| + |
| + var playTimeoutID; |
| + |
| + var playTimeoutHandler = test.step_func(function playAndSeek(test, mediaElement) |
|
wolenetz
2013/07/31 01:20:45
nits: remove ' playAndSeek' because the function c
anandc
2013/07/31 19:01:17
Done.
|
| + { |
| + assert_greater_than(mediaElement.currentTime, 0.0, 'Playback has started.'); |
| + test.expectEvent(mediaElement, 'seeking', 'mediaElement'); |
| + mediaElement.currentTime = 0.0; |
| + |
| + test.waitForExpectedEvents(function() |
| + { |
| + assert_true(mediaElement.seeking, 'mediaElement is seeking'); |
| + assert_equals(mediaElement.currentTime, 0.0, 'Current time is 0.0'); |
| + test.expectEvent(mediaElement, 'timeupdate', 'timeupdate fired.'); |
| + test.expectEvent(mediaElement, 'seeked', 'mediaElement finished seek'); |
| + var playAfterSeekHandler = test.step_func(function confirmPlaying(test, mediaElement) |
|
wolenetz
2013/07/31 01:20:45
nits: anonymize the function and remove redundant
anandc
2013/07/31 19:01:17
Done.
|
| + { |
| + assert_greater_than(mediaElement.currentTime, 0.0, 'Playback has started after seek.'); |
| + clearTimeout(playTimeoutID); |
|
wolenetz
2013/07/31 01:20:45
See my comments @ line 67 and 74, below. This clea
anandc
2013/07/31 19:01:17
Done.
|
| + test.done(); |
| + }); |
| + setTimeout(function() {playAfterSeekHandler(test, mediaElement)}, 300); |
|
wolenetz
2013/07/31 01:20:45
Ditto with my playTimeoutID comment @ line 74, bel
anandc
2013/07/31 19:01:17
Done.
|
| + }); |
| + |
| + }); |
| + |
| + test.waitForExpectedEvents(function() |
| + { |
| + playTimeoutID = setTimeout(function() {playTimeoutHandler(test, mediaElement)}, 300); |
|
wolenetz
2013/07/31 01:20:45
If going this route with intent to cancel your not
anandc
2013/07/31 19:01:17
PTAL at updated mediasource-utils.
|
| + }); |
| + |
| + }, 'Test playing then seeking back.'); |
| + </script> |
| + </body> |
| +</html> |
