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

Unified Diff: third_party/WebKit/LayoutTests/fast/mediacapturefromelement/HTMLMediaElementCapture-capture.html

Issue 2727583007: HTMLMediaElement capture: teach the captured MStream to follow up source events (Closed)
Patch Set: haraken@ comments Created 3 years, 9 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/fast/mediacapturefromelement/HTMLMediaElementCapture-capture.html
diff --git a/third_party/WebKit/LayoutTests/fast/mediacapturefromelement/HTMLMediaElementCapture-capture.html b/third_party/WebKit/LayoutTests/fast/mediacapturefromelement/HTMLMediaElementCapture-capture.html
index 420e3959309daaa64ecc43883bf7c8f59c4d7032..65d9d5cd580ecfa9709bede93f89b70e88c9f137 100644
--- a/third_party/WebKit/LayoutTests/fast/mediacapturefromelement/HTMLMediaElementCapture-capture.html
+++ b/third_party/WebKit/LayoutTests/fast/mediacapturefromelement/HTMLMediaElementCapture-capture.html
@@ -10,21 +10,19 @@ var makeAsyncTest = function(filename) {
var video = document.createElement('video');
video.src = "../../http/tests/media/resources/media-source/webm/" + filename;
video.onerror = this.unreached_func("<video> error");
+ video.play();
- video.onloadedmetadata = this.step_func(function() {
- var stream = video.captureStream();
+ var stream = video.captureStream();
+
+ // onactive event is marked for deprecation (https://crbug.com/649328)
+ stream.onactive = this.step_func_done(function() {
var recorder = new MediaRecorder(stream);
recorder.ondataavailable = test.step_func_done(function(event) {
assert_true(event.data.size > 0, 'Recorded data size should be > 0');
});
-
recorder.start(0);
- video.play();
-
});
-
- video.load();
- }), "<video>.captureStream() and assert data flows.";
+ }), "<video>.captureStream() and assert data flows.";
};
generate_tests(makeAsyncTest, [

Powered by Google App Engine
This is Rietveld 408576698