Apply natural size transform for 90/270 degrees orientated videos

Author: AlexV525

packages/video_player/video_player_avfoundation/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 2.3.2
+
+* Apply natural size transform for 90/270 degrees orientated videos.
+
 ## 2.3.1
 
 * Renames internal method channels to avoid potential confusion with the

packages/video_player/video_player_avfoundation/ios/Classes/FLTVideoPlayerPlugin.m

@@ -189,23 +189,18 @@ - (instancetype)initWithURL:(NSURL *)url
 
 - (CGAffineTransform)fixTransform:(AVAssetTrack *)videoTrack {
   CGAffineTransform transform = videoTrack.preferredTransform;
-  // TODO(@recastrodiaz): why do we need to do this? Why is the preferredTransform incorrect?
-  // At least 2 user videos show a black screen when in portrait mode if we directly use the
-  // videoTrack.preferredTransform Setting tx to the height of the video instead of 0, properly
-  // displays the video https://github.com/flutter/flutter/issues/17606#issuecomment-413473181
-  if (transform.tx == 0 && transform.ty == 0) {
-    NSInteger rotationDegrees = (NSInteger)round(radiansToDegrees(atan2(transform.b, transform.a)));
-    NSLog(@"TX and TY are 0. Rotation: %ld. Natural width,height: %f, %f", (long)rotationDegrees,
-          videoTrack.naturalSize.width, videoTrack.naturalSize.height);
-    if (rotationDegrees == 90) {
-      NSLog(@"Setting transform tx");
-      transform.tx = videoTrack.naturalSize.height;
-      transform.ty = 0;
-    } else if (rotationDegrees == 270) {
-      NSLog(@"Setting transform ty");
-      transform.tx = 0;
-      transform.ty = videoTrack.naturalSize.width;
-    }
+  NSInteger rotationDegrees = (NSInteger)round(radiansToDegrees(atan2(transform.b, transform.a)));
+  CGSize size = videoTrack.naturalSize;
+  NSLog(@"TX and TY: %f, %f. Rotation: %ld. Natural width and height: %f, %f", transform.tx,
+        transform.ty, (long)rotationDegrees, size.width, size.height);
+  if (rotationDegrees == 90) {
+    NSLog(@"Setting transform tx");
+    transform.tx = size.height;
+    transform.ty = 0;
+  } else if (rotationDegrees == 270) {
+    NSLog(@"Setting transform ty");
+    transform.tx = 0;
+    transform.ty = size.width;
   }
   return transform;
 }

packages/video_player/video_player_avfoundation/pubspec.yaml

@@ -2,7 +2,7 @@ name: video_player_avfoundation
 description: iOS implementation of the video_player plugin.
 repository: https://github.com/flutter/plugins/tree/master/packages/video_player/video_player_avfoundation
 issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+video_player%22
-version: 2.3.1
+version: 2.3.2
 
 environment:
   sdk: ">=2.14.0 <3.0.0"