refactor(ios): remove type assertion within setupPipController (#3493)

* refactor(ios): remove type assertion within setupPipController * lint: apply swift lint
TheWidlarzGroup · Jan 24, 2024 · e63c7cb · e63c7cb
1 parent 7eeb1c5
commit e63c7cb
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/ios/Video/Features/RCTPictureInPicture.swift b/ios/Video/Features/RCTPictureInPicture.swift
@@ -47,8 +47,10 @@ import React
         }
 
         func setupPipController(_ playerLayer: AVPlayerLayer?) {
+            guard let playerLayer else { return }
+            if !AVPictureInPictureController.isPictureInPictureSupported() { return }
             // Create new controller passing reference to the AVPlayerLayer
-            _pipController = AVPictureInPictureController(playerLayer: playerLayer!)
+            _pipController = AVPictureInPictureController(playerLayer: playerLayer)
             if #available(iOS 14.2, *) {
                 _pipController?.canStartPictureInPictureAutomaticallyFromInline = true
             }

diff --git a/ios/Video/Features/RCTVideoTVUtils.swift b/ios/Video/Features/RCTVideoTVUtils.swift
@@ -12,7 +12,7 @@ import Foundation
             var metadataGroups = [AVTimedMetadataGroup]()
 
             // Iterate over the defined chapters and build a timed metadata group object for each.
-            chapters.forEach { chapter in
+            for chapter in chapters {
                 metadataGroups.append(makeTimedMetadataGroup(for: chapter))
             }