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: content/renderer/media/media_stream_video_track.cc

Issue 2606983002: Media Capture Depth Stream Extensions API: focal length and depth range. (Closed)
Patch Set: Added to MediaTrackSupportedConstraints and MediaTrackConstraintSet. Thanks mcasas@. 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: 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..7128effe008c80a15dffa66468d11607e21240f7 100644
--- a/content/renderer/media/media_stream_video_track.cc
+++ b/content/renderer/media/media_stream_video_track.cc
@@ -301,16 +301,25 @@ void MediaStreamVideoTrack::Stop() {
void MediaStreamVideoTrack::getSettings(
blink::WebMediaStreamTrack::Settings& settings) {
- if (source_) {
- const media::VideoCaptureFormat* format = source_->GetCurrentFormat();
- if (format) {
- settings.frameRate = format->frame_rate;
- settings.width = format->frame_size.width();
- settings.height = format->frame_size.height();
- }
- }
// TODO(hta): Extract the real value.
settings.deviceId = blink::WebString("video device ID");
+ if (!source_)
+ return;
+
+ const media::VideoCaptureFormat* format = source_->GetCurrentFormat();
+ if (format) {
+ settings.frameRate = format->frame_rate;
+ settings.width = format->frame_size.width();
+ settings.height = format->frame_size.height();
+ }
+ const base::Optional<CameraCalibration> calibration =
+ source_->device_info().device.camera_calibration;
+ if (calibration) {
+ settings.depthNear = calibration->depth_near;
+ settings.depthFar = calibration->depth_far;
+ settings.focalLengthX = calibration->focal_length_x;
+ settings.focalLengthY = calibration->focal_length_y;
+ }
}
void MediaStreamVideoTrack::OnReadyStateChanged(

Powered by Google App Engine
This is Rietveld 408576698