Remove dependence on MediaSettings mutability

Author: RobertOdrowaz

packages/camera/camera_avfoundation/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 0.9.22+10
+
+* Removes dependence on MediaSettings mutability.
+
 ## 0.9.22+9
 
 * Migrates `FLTSavePhotoDelegate`, `FLTWritableData`, and `FLTImageStreamHandler` classes to Swift.

packages/camera/camera_avfoundation/example/ios/RunnerTests/CameraSettingsTests.swift

@@ -86,8 +86,8 @@ private final class TestMediaSettingsAVWrapper: FLTCamMediaSettingsAVWrapper {
     if let compressionProperties = outputSettings?[AVVideoCompressionPropertiesKey]
       as? [String: Any],
       let bitrate = compressionProperties[AVVideoAverageBitRateKey] as? Int,
-      let frameRate = compressionProperties[AVVideoExpectedSourceFrameRateKey] as? Int,
-      bitrate == testVideoBitrate, frameRate == testFramesPerSecond
+      let frameRate = compressionProperties[AVVideoExpectedSourceFrameRateKey] as? Double,
+      bitrate == testVideoBitrate, frameRate == Double(testFramesPerSecond)
     {
       videoSettingsExpectation.fulfill()
     }

packages/camera/camera_avfoundation/ios/camera_avfoundation/Sources/camera_avfoundation/DefaultCamera.swift

@@ -43,6 +43,7 @@ final class DefaultCamera: NSObject, Camera {
   private let captureSessionQueue: DispatchQueue
 
   private let mediaSettings: FCPPlatformMediaSettings
+  private var framesPerSecond: Double?
   private let mediaSettingsAVWrapper: FLTCamMediaSettingsAVWrapper
 
   private let videoCaptureSession: CaptureSession
@@ -213,14 +214,14 @@ final class DefaultCamera: NSObject, Camera {
 
       try setCaptureSessionPreset(mediaSettings.resolutionPreset)
 
-      FormatUtils.selectBestFormat(
+      (captureDevice.flutterActiveFormat, framesPerSecond) = FormatUtils.findBestFormat(
         for: captureDevice,
         mediaSettings: mediaSettings,
         videoDimensionsConverter: videoDimensionsConverter)
 
-      if let framesPerSecond = mediaSettings.framesPerSecond {
+      if let framesPerSecond = framesPerSecond {
         // Set frame rate with 1/10 precision allowing non-integral values.
-        let fpsNominator = floor(framesPerSecond.doubleValue * 10.0)
+        let fpsNominator = floor(framesPerSecond * 10.0)
         let duration = CMTimeMake(value: 10, timescale: Int32(fpsNominator))
 
         mediaSettingsAVWrapper.setMinFrameDuration(duration, on: captureDevice)
@@ -542,14 +543,14 @@ final class DefaultCamera: NSObject, Camera {
       for: captureVideoOutput
     )
 
-    if mediaSettings.videoBitrate != nil || mediaSettings.framesPerSecond != nil {
+    if mediaSettings.videoBitrate != nil || framesPerSecond != nil {
       var compressionProperties: [String: Any] = [:]
 
       if let videoBitrate = mediaSettings.videoBitrate {
         compressionProperties[AVVideoAverageBitRateKey] = videoBitrate
       }
 
-      if let framesPerSecond = mediaSettings.framesPerSecond {
+      if let framesPerSecond = framesPerSecond {
         compressionProperties[AVVideoExpectedSourceFrameRateKey] = framesPerSecond
       }
 

packages/camera/camera_avfoundation/ios/camera_avfoundation/Sources/camera_avfoundation/FormatUtils.swift

@@ -36,14 +36,13 @@ enum FormatUtils {
 
   /// Finds format with same resolution as current activeFormat in captureDevice for which
   /// bestFrameRate returned frame rate closest to mediaSettings.framesPerSecond.
-  /// Preferred are formats with the same subtype as current activeFormat. Sets this format
-  /// as activeFormat and also updates mediaSettings.framesPerSecond to value which
-  /// bestFrameRate returned for that format.
-  static func selectBestFormat(
+  /// Preferred are formats with the same subtype as current activeFormat. Returns this format
+  /// and frame rate which bestFrameRate returned for that format.
+  static func findBestFormat(
     for captureDevice: CaptureDevice,
     mediaSettings: FCPPlatformMediaSettings,
     videoDimensionsConverter: VideoDimensionsConverter
-  ) {
+  ) -> (CaptureDeviceFormat, Double) {
     let targetResolution = videoDimensionsConverter(captureDevice.flutterActiveFormat)
     let targetFrameRate = mediaSettings.framesPerSecond?.doubleValue ?? 0
     let preferredSubType = CMFormatDescriptionGetMediaSubType(
@@ -76,7 +75,6 @@ enum FormatUtils {
       }
     }
 
-    captureDevice.flutterActiveFormat = bestFormat
-    mediaSettings.framesPerSecond = NSNumber(value: resolvedBastFrameRate)
+    return (bestFormat, resolvedBastFrameRate)
   }
 }

packages/camera/camera_avfoundation/pubspec.yaml

@@ -2,7 +2,7 @@ name: camera_avfoundation
 description: iOS implementation of the camera plugin.
 repository: https://github.com/flutter/packages/tree/main/packages/camera/camera_avfoundation
 issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+camera%22
-version: 0.9.22+9
+version: 0.9.22+10
 
 environment:
   sdk: ^3.9.0