Media Capture Depth Stream Extensions API: focal length and depth range.

content/public/common/media_stream_request.h

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/, so
you can just reuse the CameraCalibration from there,
right?  No need to redefine it here.

[aleksandar.stojiljkovic, 2017/01/09 18:47:01]

Done.
I saw related TODO(ajm) at L.126 but assumed there is a need to keep it this
way...
It is now aliased to content::CameraCalibration (used in content code only) as
it is shorter.

+
+  // 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 want to use
base::Optional (see [1]).

[1] https://chromium.googlesource.com/chromium/src/+/master/docs/optional.md

[aleksandar.stojiljkovic, 2017/01/09 18:47:01]

Done. Thanks.

 };
 
 class CONTENT_EXPORT MediaStreamDevices