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

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

Issue 14188039: MediaStream API: Remove LocalMediaStream (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Fixed tests Created 7 years, 8 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 153f5b575f519773953e3a2be30a0495f44c6d04..4dfaffa9ce444151377d05c16857a45b5935904e 100644
--- a/LayoutTests/fast/mediastream/MediaStream-onended.html
+++ b/LayoutTests/fast/mediastream/MediaStream-onended.html
@@ -26,20 +26,39 @@ function getUserMedia(dictionary, callback) {
var stream;
-function streamEnded() {
+function streamEnded2() {
testPassed('streamEnded was called.');
shouldBeTrue('stream.ended');
finishJSTest();
}
-function gotStream(s) {
+function gotStream2(s) {
stream = new webkitMediaStream(s);
shouldBeFalse('stream.ended');
try {
- stream.onended = streamEnded;
+ stream.onended = streamEnded2;
s.stop();
} catch (e) {
- testFailed('LocalMediaStream threw exception :' + e);
+ testFailed('MediaStream threw exception :' + e);
+ finishJSTest();
+ }
+}
+
+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();
+ } catch (e) {
+ testFailed('MediaStream threw exception :' + e);
finishJSTest();
}
}

Powered by Google App Engine
This is Rietveld 408576698