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

Unified Diff: LayoutTests/fast/mediastream/MediaStream-onended.html

Issue 1312213006: Remove MediaStream label, ended attributes and stop() method (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 5 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: LayoutTests/fast/mediastream/MediaStream-onended.html
diff --git a/LayoutTests/fast/mediastream/MediaStream-onended.html b/LayoutTests/fast/mediastream/MediaStream-onended.html
index 3cd40cb337b1c4798f8e9a8e9186e14426b66f23..44804b3de924d083fdc09a60a52b57404df79583 100644
--- a/LayoutTests/fast/mediastream/MediaStream-onended.html
+++ b/LayoutTests/fast/mediastream/MediaStream-onended.html
@@ -27,16 +27,17 @@ var stream;
function streamEnded2() {
testPassed('streamEnded was called.');
- shouldBeTrue('stream.ended');
finishJSTest();
}
function gotStream2(s) {
stream = new webkitMediaStream(s);
- shouldBeFalse('stream.ended');
try {
stream.onended = streamEnded2;
- s.stop();
+ for (var i = 0; i < s.getAudioTracks().length; i++)
philipj_slow 2015/08/27 14:34:54 If you know how many stream there are in this test
shiva.jm 2015/08/28 04:40:22 Done.
+ s.getAudioTracks()[i].stop();
+ for (var i = 0; i < s.getVideoTracks().length; i++)
+ s.getVideoTracks()[i].stop();
} catch (e) {
testFailed('MediaStream threw exception :' + e);
finishJSTest();
@@ -45,17 +46,17 @@ function gotStream2(s) {
function streamEnded() {
testPassed('streamEnded was called.');
- shouldBeTrue('stream.ended');
-
getUserMedia({audio:true, video:true}, gotStream2);
}
function gotStream(s) {
stream = s;
- shouldBeFalse('stream.ended');
try {
stream.onended = streamEnded;
- stream.stop();
+ for (var i = 0; i < s.getAudioTracks().length; i++)
+ s.getAudioTracks()[i].stop();
+ for (var i = 0; i < s.getVideoTracks().length; i++)
+ s.getVideoTracks()[i].stop();
} catch (e) {
testFailed('MediaStream threw exception :' + e);
finishJSTest();

Powered by Google App Engine
This is Rietveld 408576698