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

Unified Diff: content/renderer/media/media_stream_video_track.cc

Issue 2590193002: Add FacingMode to MediaStreamTrack.getSettings() (Closed)
Patch Set: Layout test, with TODO 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
« no previous file with comments | « no previous file | content/renderer/media/media_stream_video_track_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/media/media_stream_video_track.cc
diff --git a/content/renderer/media/media_stream_video_track.cc b/content/renderer/media/media_stream_video_track.cc
index dede54fecb104f1e46e97f43eb0ba1d2ed1dea20..e9eece0b225735ff86db11ac0a2239d4f6903012 100644
--- a/content/renderer/media/media_stream_video_track.cc
+++ b/content/renderer/media/media_stream_video_track.cc
@@ -308,6 +308,21 @@ void MediaStreamVideoTrack::getSettings(
settings.width = format->frame_size.width();
settings.height = format->frame_size.height();
}
+ switch (source_->device_info().device.video_facing) {
+ case MEDIA_VIDEO_FACING_NONE:
+ settings.facingMode = blink::WebMediaStreamTrack::FacingMode::None;
+ break;
+ case MEDIA_VIDEO_FACING_USER:
+ settings.facingMode = blink::WebMediaStreamTrack::FacingMode::User;
+ break;
+ case MEDIA_VIDEO_FACING_ENVIRONMENT:
+ settings.facingMode =
+ blink::WebMediaStreamTrack::FacingMode::Environment;
+ break;
+ default:
+ settings.facingMode = blink::WebMediaStreamTrack::FacingMode::None;
+ break;
+ }
}
// TODO(hta): Extract the real value.
settings.deviceId = blink::WebString("video device ID");
« no previous file with comments | « no previous file | content/renderer/media/media_stream_video_track_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698