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

Side by Side 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: Call VideoTrackList::trackSelected also the SourceBuffer track list 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 unified diff | Download patch
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/media/avtrack/video-track-selected.html » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 <!DOCTYPE html> 1 <!DOCTYPE html>
2 <html> 2 <html>
3 <head> 3 <head>
4 <script src="/w3c/resources/testharness.js"></script> 4 <script src="/w3c/resources/testharness.js"></script>
5 <script src="/w3c/resources/testharnessreport.js"></script> 5 <script src="/w3c/resources/testharnessreport.js"></script>
6 <script src="mediasource-util.js"></script> 6 <script src="mediasource-util.js"></script>
7 </head> 7 </head>
8 <body> 8 <body>
9 <script> 9 <script>
10 function loadMediaAndVerifyAddedTracks(test, mediaElement, segmentIn fo, sourceBuffer, mediaData, successCallback) 10 function loadMediaAndVerifyAddedTracks(test, mediaElement, segmentIn fo, sourceBuffer, mediaData, successCallback)
(...skipping 102 matching lines...) Expand 10 before | Expand all | Expand 10 after
113 { 113 {
114 loadMediaAndVerifyAddedTracks(test, mediaElement, segmentInfo, s ourceBuffer, mediaData, test.step_func(function () 114 loadMediaAndVerifyAddedTracks(test, mediaElement, segmentInfo, s ourceBuffer, mediaData, test.step_func(function ()
115 { 115 {
116 verifyTrackRemoval(test, mediaElement, mediaSource, sourceBu ffer, test.step_func(function () 116 verifyTrackRemoval(test, mediaElement, mediaSource, sourceBu ffer, test.step_func(function ()
117 { 117 {
118 mediaElement.load(); 118 mediaElement.load();
119 }), test.step_func_done()); 119 }), test.step_func_done());
120 })); 120 }));
121 }, "Media tracks must be removed when the HTMLMediaElement.load() is called"); 121 }, "Media tracks must be removed when the HTMLMediaElement.load() is called");
122 122
123 mediasource_testafterdataloaded(function(test, mediaElement, mediaSo urce, segmentInfo, sourceBuffer, mediaData)
124 {
125 loadMediaAndVerifyAddedTracks(test, mediaElement, segmentInfo, s ourceBuffer, mediaData, test.step_func(function ()
126 {
127 assert_equals(mediaSource.activeSourceBuffers.length, 1, "me diaSource.activeSourceBuffers.length == 1");
128 assert_equals(mediaSource.activeSourceBuffers[0], sourceBuff er, "mediaSource.activeSourceBuffers[0]");
129 assert_equals(sourceBuffer.videoTracks.selectedIndex, 0, "so urceBuffer.videoTracks.selectedIndex");
130 test.expectEvent(mediaElement.audioTracks, "change", "mediaE lement.audioTracks changed.");
131 test.expectEvent(mediaElement.videoTracks, "change", "mediaE lement.videoTracks changed.");
132 test.expectEvent(sourceBuffer.audioTracks, "change", "mediaE lement.audioTracks changed.");
133 test.expectEvent(sourceBuffer.videoTracks, "change", "mediaE lement.videoTracks changed.");
134 test.expectEvent(mediaSource.activeSourceBuffers, "removesou rcebuffer", "mediaSource.activeSourceBuffers.onchange");
135 mediaElement.audioTracks[0].enabled = false;
136 mediaElement.videoTracks[0].selected = false;
137 test.waitForExpectedEvents(test.step_func(function()
138 {
139 assert_equals(mediaSource.activeSourceBuffers.length, 0, "mediaSource.activeSourceBuffers.length == 0");
140 assert_equals(sourceBuffer.videoTracks.selectedIndex, -1 , "sourceBuffer.videoTracks.selectedIndex");
141 test.expectEvent(mediaElement.audioTracks, "change", "me diaElement.audioTracks changed.");
142 test.expectEvent(mediaElement.videoTracks, "change", "me diaElement.videoTracks changed.");
143 test.expectEvent(sourceBuffer.audioTracks, "change", "so urceBuffer.audioTracks changed.");
144 test.expectEvent(sourceBuffer.videoTracks, "change", "so urceBuffer.videoTracks changed.");
145 mediaElement.audioTracks[0].enabled = true;
146 mediaElement.videoTracks[0].selected = true;
147 test.expectEvent(mediaSource.activeSourceBuffers, "addso urcebuffer", "mediaSource.activeSourceBuffers.onchange");
148 test.waitForExpectedEvents(test.step_func_done());
149 }));
150 }));
151 }, "Change events must be fired on MediaElement and SourceBuffer tra ck lists when enabling/disabling or selecting/deselecting tracks");
152
123 </script> 153 </script>
124 </body> 154 </body>
125 </html> 155 </html>
126 156
OLDNEW
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/media/avtrack/video-track-selected.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698