Bug 1382433 - use fixed fps instead of floating fps to avoid very low fps. r?jib draft
authorMunro Mengjue Chiang <mchiang@mozilla.com>
Wed, 27 Sep 2017 18:50:54 +0800
changeset 671049 e605dee95ac0256a5b12b8293af3d593f3e23846
parent 658358 8e05298328da75f3056a9f1f9609938870d756a0
child 733399 9e4516e4e816e2b72533e13984b3643f62d1c910
push id81808
push userbmo:mchiang@mozilla.com
push dateWed, 27 Sep 2017 10:51:16 +0000
reviewersjib
bugs1382433
milestone57.0a1
Bug 1382433 - use fixed fps instead of floating fps to avoid very low fps. r?jib MozReview-Commit-ID: 2QUWJM2LNkK
media/webrtc/trunk/webrtc/modules/video_capture/mac/avfoundation/video_capture_avfoundation_objc.mm
--- a/media/webrtc/trunk/webrtc/modules/video_capture/mac/avfoundation/video_capture_avfoundation_objc.mm
+++ b/media/webrtc/trunk/webrtc/modules/video_capture/mac/avfoundation/video_capture_avfoundation_objc.mm
@@ -147,17 +147,17 @@ using namespace videocapturemodule;
     AVCaptureConnection* captureConnection =
       [_captureVideoDataOutput connectionWithMediaType:AVMediaTypeVideo];
 
     if ([captureConnection isVideoMinFrameDurationSupported]) {
       [captureConnection setVideoMinFrameDuration:bestFrameRateRange.minFrameDuration];
     }
 
     if ([captureConnection isVideoMaxFrameDurationSupported]) {
-      [captureConnection setVideoMaxFrameDuration:bestFrameRateRange.maxFrameDuration];
+      [captureConnection setVideoMaxFrameDuration:bestFrameRateRange.minFrameDuration];
     }
   }
 }
 
 - (void)startCapture {
   if (_capturing)
     return;