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

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

Issue 2651353002: Get the device ID in getSettings() (Closed)
Patch Set: Created 3 years, 11 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/mediastream/MediaStreamTrack-getSettings.html
diff --git a/third_party/WebKit/LayoutTests/fast/mediastream/MediaStreamTrack-getSettings.html b/third_party/WebKit/LayoutTests/fast/mediastream/MediaStreamTrack-getSettings.html
index 39da4ac828a35b69cd441ef627bc6d70a83ffaae..7401a32193a6ac12e75642875a3acdf94d61babf 100644
--- a/third_party/WebKit/LayoutTests/fast/mediastream/MediaStreamTrack-getSettings.html
+++ b/third_party/WebKit/LayoutTests/fast/mediastream/MediaStreamTrack-getSettings.html
@@ -20,10 +20,22 @@ promise_test(function() {
}, 'A video track returns settings');
promise_test(function() {
+ return navigator.mediaDevices.getUserMedia({audio: true})
+ .then(function(s) {
+ settings = s.getAudioTracks()[0].getSettings();
+ assert_greater_than(Object.keys(settings).length, 0);
+ assert_true('deviceId' in settings,
+ 'Device ID missing: ' + JSON.stringify(settings));
+ });
+}, 'An audio track returns the expected variables');
+
+promise_test(function() {
return navigator.mediaDevices.getUserMedia({video: true})
.then(function(s) {
settings = s.getVideoTracks()[0].getSettings();
assert_greater_than(Object.keys(settings).length, 0);
+ assert_true('deviceId' in settings,
+ 'Device ID missing: ' + JSON.stringify(settings));
assert_true('frameRate' in settings,
'Frame rate missing: ' + JSON.stringify(settings));
assert_true('width' in settings,
@@ -31,7 +43,17 @@ promise_test(function() {
assert_true('height' in settings,
'Height missing: ' + JSON.stringify(settings));
});
-}, 'A video track returns frame rate');
+}, 'A video track returns the expected variables');
+
+promise_test(function() {
+ return navigator.mediaDevices.getUserMedia({audio: true, video: true})
+ .then(function(s) {
+ videoSettings = s.getVideoTracks()[0].getSettings();
+ audioSettings = s.getAudioTracks()[0].getSettings();
+ assert_not_equals(videoSettings.deviceId, audioSettings.deviceId,
+ 'video and audio device IDs should be different')
+ });
+}, 'Video and audio device IDs are different');
promise_test(function() {
return navigator.mediaDevices.getUserMedia(

Powered by Google App Engine
This is Rietveld 408576698