Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(173)

Unified Diff: LayoutTests/http/tests/media/media-source/mediasource-removesourcebuffer.html

Issue 419673007: Adding more MSE tests. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Fixing CR comments Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: LayoutTests/http/tests/media/media-source/mediasource-removesourcebuffer.html
diff --git a/LayoutTests/http/tests/media/media-source/mediasource-removesourcebuffer.html b/LayoutTests/http/tests/media/media-source/mediasource-removesourcebuffer.html
index b7cbd42077252efd93438115b6311ce746772079..9d0b66e4daf7bc902ca3afa043dbb93ee7d38a49 100644
--- a/LayoutTests/http/tests/media/media-source/mediasource-removesourcebuffer.html
+++ b/LayoutTests/http/tests/media/media-source/mediasource-removesourcebuffer.html
@@ -61,7 +61,33 @@
test.done();
}, "Test calling removeSourceBuffer() in ended state.");
+
+ mediasource_testafterdataloaded(function(test, mediaElement, mediaSource, segmentInfo, sourceBuffer, mediaData)
+ {
+ var initSegment = MediaSourceUtil.extractSegmentData(mediaData, segmentInfo.init);
+
+ test.expectEvent(sourceBuffer, 'updateend', 'initSegment append ended.');
+ test.expectEvent(mediaElement, 'loadedmetadata', 'loadedmetadata done.');
+ sourceBuffer.appendBuffer(initSegment);
+ test.waitForExpectedEvents(function()
+ {
+ assert_true(mediaSource.sourceBuffers.length == 1, "MediaSource.sourceBuffers is not empty");
+ assert_true(mediaSource.activeSourceBuffers.length == 1, "MediaSource.activesourceBuffers is not empty");
+ assert_equals(mediaElement.readyState, mediaElement.HAVE_METADATA);
+ assert_equals(mediaSource.duration, segmentInfo.durationInInitSegment);
+ test.expectEvent(mediaSource.activeSourceBuffers, 'removesourcebuffer', 'SourceBuffer removed from activeSourceBuffers.');
+ test.expectEvent(mediaSource.sourceBuffers, 'removesourcebuffer', 'SourceBuffer removed.');
+ mediaSource.removeSourceBuffer(sourceBuffer);
+ });
+
+ test.waitForExpectedEvents(function()
+ {
+ assert_true(mediaSource.sourceBuffers.length == 0, "MediaSource.sourceBuffers is empty");
+ assert_true(mediaSource.activeSourceBuffers.length == 0, "MediaSource.activesourceBuffers is empty");
+ test.done();
+ });
+ }, 'Test removesourcebuffer event on activeSourceBuffers.');
</script>
</body>
</html>

Powered by Google App Engine
This is Rietveld 408576698