Support cameras that report invalid frame rates

Desktop / Chromium - Guido Urdaneta [chromium.org] - 5 August 2017 06:03 EDT

They should be supported if no constraints are placed on the frame rate. This CL moves some logic from the algorithm for content capture to a generic class that supports numeric ranges for both device and content capture. No behavior change is intended for content capture.

Bug: 739953 Change-Id: I2ae6f19f75b4c8a404d8bd997bb8d5085eb22b0e Reviewed-on: https://chromium-review.googlesource.com/596311

18e768a Support cameras that report invalid frame rates.
.../media/media_stream_constraints_util.cc | 18 +-
.../media/media_stream_constraints_util_sets.h | 64 ++++---
.../media_stream_constraints_util_sets_unittest.cc | 18 +-
.../media_stream_constraints_util_video_content.cc | 56 ++----
...ream_constraints_util_video_content_unittest.cc | 80 ++++++---
.../media_stream_constraints_util_video_device.cc | 61 ++++---
...tream_constraints_util_video_device_unittest.cc | 198 +++++++++++++++++----
.../media/user_media_client_impl_unittest.cc | 2 +-
8 files changed, 338 insertions(+), 159 deletions(-)

Upstream: git.chromium.org


  • Share