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

Unified Diff: content/public/common/media_stream_request.h

Issue 2606983002: Media Capture Depth Stream Extensions API: focal length and depth range. (Closed)
Patch Set: Created 4 years 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/public/common/media_stream_request.h
diff --git a/content/public/common/media_stream_request.h b/content/public/common/media_stream_request.h
index 07a4e1af114d6adf49e2d65b1982c7ac1c1de92c..d7b77f0cb86069cbe5809345cab361151d119554 100644
--- a/content/public/common/media_stream_request.h
+++ b/content/public/common/media_stream_request.h
@@ -164,6 +164,24 @@ struct CONTENT_EXPORT MediaStreamDevice {
// exists (e.g. webcam w/mic), then the value of this member will be all
// zeros.
AudioDeviceParameters matched_output;
+
+ // Contains camera calibration parameters.
+ // These parameters apply to both RGB and depth video devices. See also
+ // https://w3c.github.io/mediacapture-depth/#mediatracksettings-dictionary
+ // TODO(aleksandar.stojiljkovic): Add principal point and camera distortion
+ // model and coefficients. See also https://crbug.com/616098
+ struct CONTENT_EXPORT CameraCalibration {
+ double focal_length_x = 0.0;
+ double focal_length_y = 0.0;
+ // depth_near and far are used only for depth cameras.
+ double depth_near = 0.0;
+ double depth_far = 0.0;
+ // Specifies if the values here are valid.
+ bool valid = false;
+ };
mcasas 2017/01/05 01:09:32 content/public/ can pull data types out of media/,
aleksandar.stojiljkovic 2017/01/09 18:47:01 Done. I saw related TODO(ajm) at L.126 but assumed
+
+ // This field values are optional and available only for some camera models.
+ CameraCalibration camera_calibration;
mcasas 2017/01/05 01:09:32 If this field is not always present we probably wa
aleksandar.stojiljkovic 2017/01/09 18:47:01 Done. Thanks.
};
class CONTENT_EXPORT MediaStreamDevices

Powered by Google App Engine
This is Rietveld 408576698