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

Side by Side Diff: third_party/WebKit/LayoutTests/fast/mediastream/MediaStreamTrack-getSettings.html

Issue 2037573004: Add video width and height to getSettings. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 6 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
OLDNEW
1 <!DOCTYPE HTML> 1 <!DOCTYPE HTML>
2 <script src="../../resources/testharness.js"></script> 2 <script src="../../resources/testharness.js"></script>
3 <script src="../../resources/testharnessreport.js"></script> 3 <script src="../../resources/testharnessreport.js"></script>
4 <script> 4 <script>
5 5
6 promise_test(function() { 6 promise_test(function() {
7 return navigator.mediaDevices.getUserMedia({audio: true}) 7 return navigator.mediaDevices.getUserMedia({audio: true})
8 .then(function(s) { 8 .then(function(s) {
9 settings = s.getAudioTracks()[0].getSettings(); 9 settings = s.getAudioTracks()[0].getSettings();
10 assert_greater_than(Object.keys(settings).length, 0); 10 assert_greater_than(Object.keys(settings).length, 0);
11 }); 11 });
12 }, 'An audio track returns settings'); 12 }, 'An audio track returns settings');
13 13
14 promise_test(function() { 14 promise_test(function() {
15 return navigator.mediaDevices.getUserMedia({video: true}) 15 return navigator.mediaDevices.getUserMedia({video: true})
16 .then(function(s) { 16 .then(function(s) {
17 settings = s.getVideoTracks()[0].getSettings(); 17 settings = s.getVideoTracks()[0].getSettings();
18 assert_greater_than(Object.keys(settings).length, 0); 18 assert_greater_than(Object.keys(settings).length, 0);
19 }); 19 });
20 }, 'A video track returns settings'); 20 }, 'A video track returns settings');
21 21
22 promise_test(function() { 22 promise_test(function() {
23 return navigator.mediaDevices.getUserMedia({video: true}) 23 return navigator.mediaDevices.getUserMedia({video: true})
24 .then(function(s) { 24 .then(function(s) {
25 settings = s.getVideoTracks()[0].getSettings(); 25 settings = s.getVideoTracks()[0].getSettings();
26 assert_greater_than(Object.keys(settings).length, 0); 26 assert_greater_than(Object.keys(settings).length, 0);
27 assert_true('frameRate' in settings, 27 assert_true('frameRate' in settings,
28 'Frame rate missing: ' + JSON.stringify(settings)); 28 'Frame rate missing: ' + JSON.stringify(settings));
29 assert_true('width' in settings,
30 'Width missing: ' + JSON.stringify(settings));
31 assert_true('height' in settings,
32 'Height missing: ' + JSON.stringify(settings));
29 }); 33 });
30 }, 'A video track returns frame rate'); 34 }, 'A video track returns frame rate');
31 35
32 </script> 36 </script>
OLDNEW
« no previous file with comments | « content/renderer/media/media_stream_video_track.cc ('k') | third_party/WebKit/Source/modules/mediastream/MediaStreamTrack.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698