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

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

Issue 1522463003: Refactor resource load and resource selection algorithms as per spec (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added tests using internals.idl Created 4 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-creation.html
diff --git a/third_party/WebKit/LayoutTests/fast/mediacapturefromelement/HTMLMediaElementCapture-creation.html b/third_party/WebKit/LayoutTests/fast/mediacapturefromelement/HTMLMediaElementCapture-creation.html
index 47718e2be63e30ecb18e0432af6d0b587b716c25..860fda090db675649e22a6a41753b5960aa33fd0 100644
--- a/third_party/WebKit/LayoutTests/fast/mediacapturefromelement/HTMLMediaElementCapture-creation.html
+++ b/third_party/WebKit/LayoutTests/fast/mediacapturefromelement/HTMLMediaElementCapture-creation.html
@@ -20,21 +20,25 @@ test(function() {
var video = document.createElement('video');
video.src = "file:///super_duper_videos/amazing_video.webm";
video.load();
philipj_slow 2016/03/14 13:03:15 The extra load() after setting src should now be r
Srirama 2016/03/14 15:12:54 Done.
- var stream = video.captureStream();
+ video.onloadstart = function() {
+ var stream = video.captureStream();
- assert_not_equals(stream, null);
- assert_equals(1, stream.getVideoTracks().length);
- assert_equals(0, stream.getAudioTracks().length);
+ assert_not_equals(stream, null);
+ assert_equals(1, stream.getVideoTracks().length);
+ assert_equals(0, stream.getAudioTracks().length);
+ }
}, 'check <video> captureStream().');
test(function() {
var audio = document.createElement('audio');
audio.src = "file:///super_duper_videos/amazing_audio_file.webm";
audio.load();
- var stream = audio.captureStream();
+ audio.onloadstart = function() {
+ var stream = audio.captureStream();
- // TODO(mcasas): http://crbug.com/575492, implement <audio>.captureStream().
- assert_equals(stream, null);
+ // TODO(mcasas): http://crbug.com/575492, implement <audio>.captureStream().
+ assert_equals(stream, null);
+ };
}, 'check <audio> captureStream().');
test(function() {

Powered by Google App Engine
This is Rietveld 408576698