From d670ca96d7fc3bd19e04c64bc751cc7153ae513e Mon Sep 17 00:00:00 2001
From: Marc Rousavy <me@mrousavy.com>
Date: Wed, 24 Jul 2024 14:41:47 +0200
Subject: [PATCH] fix: Remove `sensorOrientation` hack on iOS, it's always
 `portrait` (#3096)

---
 .../AVCaptureDevice+sensorOrientation.swift   | 61 ++-----------------
 1 file changed, 5 insertions(+), 56 deletions(-)

diff --git a/package/ios/Core/Extensions/AVCaptureDevice+sensorOrientation.swift b/package/ios/Core/Extensions/AVCaptureDevice+sensorOrientation.swift
index 1fca13af07..2bef056262 100644
--- a/package/ios/Core/Extensions/AVCaptureDevice+sensorOrientation.swift
+++ b/package/ios/Core/Extensions/AVCaptureDevice+sensorOrientation.swift
@@ -8,67 +8,16 @@
 import AVFoundation
 import Foundation
 
-// On iOS, all built-in Cameras are landscape-left (90deg rotated)
-let DEFAULT_SENSOR_ORIENTATION = Orientation.landscapeLeft
+// On iOS, a camera device's natural sensor orientation is either
+// always portrait, or .videoOrientation already takes natural sensor
+// orientation into account.
+let DEFAULT_SENSOR_ORIENTATION = Orientation.portrait
 
 extension AVCaptureDevice {
   /**
    Get the natural orientation of the camera sensor of this specific device.
    */
   var sensorOrientation: Orientation {
-    // TODO: There is no iOS API to get native sensor orientation.
-    //       - The new `RotationCoordinator` API is a blackbox, and cannot be used statically.
-    //       - Dynamically creating an AVCaptureSession is very hacky and has a runtime overhead.
-    //       Hopefully iOS adds an API to get sensor orientation soon so we can use that!
-
-    // 0. If we don't have Camera permission yet, we cannot create a temporary AVCaptureSession.
-    //    So we just return the default orientation as a workaround...
-    let cameraPermissionStatus = AVCaptureDevice.authorizationStatus(for: .video)
-    if cameraPermissionStatus != .authorized {
-      return DEFAULT_SENSOR_ORIENTATION
-    }
-
-    // 1. Create a capture session
-    let session = AVCaptureSession()
-    if session.canSetSessionPreset(.low) {
-      session.sessionPreset = .low
-    }
-
-    // 2. Add this device as an input
-    guard let input = try? AVCaptureDeviceInput(device: self) else {
-      VisionLogger.log(level: .error, message: "Cannot dynamically determine \(uniqueID)'s sensorOrientation, " +
-        "because the AVCaptureDeviceInput cannot be created. Falling back to \(DEFAULT_SENSOR_ORIENTATION)...")
-      return DEFAULT_SENSOR_ORIENTATION
-    }
-    guard session.canAddInput(input) else {
-      VisionLogger.log(level: .error, message: "Cannot dynamically determine \(uniqueID)'s sensorOrientation, because " +
-        "it cannot be added to the temporary AVCaptureSession. Falling back to \(DEFAULT_SENSOR_ORIENTATION)...")
-      return DEFAULT_SENSOR_ORIENTATION
-    }
-    session.addInput(input)
-
-    // 3. Add an output (e.g. video data output)
-    let output = AVCaptureVideoDataOutput()
-    output.automaticallyConfiguresOutputBufferDimensions = false
-    output.deliversPreviewSizedOutputBuffers = true
-    guard session.canAddOutput(output) else {
-      VisionLogger.log(level: .error, message: "Cannot dynamically determine \(uniqueID)'s sensorOrientation, because " +
-        "the AVCaptureVideoDataOutput cannot be added to the AVCaptureSession. Falling back to \(DEFAULT_SENSOR_ORIENTATION)...")
-      return DEFAULT_SENSOR_ORIENTATION
-    }
-    session.addOutput(output)
-
-    // 4. Inspect the default orientation of the output
-    let defaultOrientation = output.orientation
-
-    // 5. Rotate the default orientation by the default sensor orientation we know of
-    var sensorOrientation = defaultOrientation.rotatedBy(orientation: DEFAULT_SENSOR_ORIENTATION)
-
-    // 6. If we are on the front Camera, AVCaptureVideoDataOutput.orientation is mirrored.
-    if position == .front {
-      sensorOrientation = sensorOrientation.flipped()
-    }
-
-    return sensorOrientation
+    return .portrait
   }
 }