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

Unified Diff: third_party/WebKit/LayoutTests/http/tests/media/media-source/mediasource-avtracks.html

Issue 2263823002: Deliver change notifications to SourceBuffer track lists. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed video-track-selected test Created 4 years, 4 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: third_party/WebKit/LayoutTests/http/tests/media/media-source/mediasource-avtracks.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/media/media-source/mediasource-avtracks.html b/third_party/WebKit/LayoutTests/http/tests/media/media-source/mediasource-avtracks.html
index 6e3e1be992c1e96dc45dbfa6dd915b06c757894b..f4aef3cf18730ed7e95efa9270224a7d58b48ad0 100644
--- a/third_party/WebKit/LayoutTests/http/tests/media/media-source/mediasource-avtracks.html
+++ b/third_party/WebKit/LayoutTests/http/tests/media/media-source/mediasource-avtracks.html
@@ -120,6 +120,29 @@
}));
}, "Media tracks must be removed when the HTMLMediaElement.load() is called");
+ mediasource_testafterdataloaded(function(test, mediaElement, mediaSource, segmentInfo, sourceBuffer, mediaData)
+ {
+ loadMediaAndVerifyAddedTracks(test, mediaElement, segmentInfo, sourceBuffer, mediaData, test.step_func(function ()
+ {
+ test.expectEvent(mediaElement.audioTracks, "change", "mediaElement.audioTracks changed.");
+ test.expectEvent(mediaElement.videoTracks, "change", "mediaElement.videoTracks changed.");
+ test.expectEvent(sourceBuffer.audioTracks, "change", "mediaElement.audioTracks changed.");
+ test.expectEvent(sourceBuffer.videoTracks, "change", "mediaElement.videoTracks changed.");
+ mediaElement.audioTracks[0].enabled = false;
+ mediaElement.videoTracks[0].selected = false;
+ test.waitForExpectedEvents(test.step_func(function()
+ {
+ test.expectEvent(mediaElement.audioTracks, "change", "mediaElement.audioTracks changed.");
+ test.expectEvent(mediaElement.videoTracks, "change", "mediaElement.videoTracks changed.");
+ test.expectEvent(sourceBuffer.audioTracks, "change", "sourceBuffer.audioTracks changed.");
+ test.expectEvent(sourceBuffer.videoTracks, "change", "sourceBuffer.videoTracks changed.");
+ mediaElement.audioTracks[0].enabled = true;
+ mediaElement.videoTracks[0].selected = true;
+ test.waitForExpectedEvents(test.step_func_done());
+ }));
+ }));
+ }, "Change events must be fired on MediaElement and SourceBuffer track lists when enabling/disabling or selecting/deselecting tracks");
+
</script>
</body>
</html>

Powered by Google App Engine
This is Rietveld 408576698