Moved the max duration to the trimmer view parent class.

Changed the generate images step to use a composition with a render size derived from the smallest video in the asset's video tracks.

Author: RemyLivewall

Diff for: .gitignore

@@ -25,3 +25,6 @@ DerivedData
 Carthage
 
 Pods/
+
+#Jetbrains IDE
+.idea/

Diff for: PryntTrimmerView/Classes/Parents/AVAssetTimeSelector.swift

@@ -15,6 +15,13 @@ public class AVAssetTimeSelector: UIView, UIScrollViewDelegate {
 
     let assetPreview = AssetVideoScrollView()
 
+    /// The maximum duration allowed for the trimming. Change it before setting the asset, as the asset preview
+    public var maxDuration: Double = 15 {
+        didSet {
+            assetPreview.maxDuration = maxDuration
+        }
+    }
+
     /// The asset to be displayed in the underlying scroll view. Setting a new asset will automatically refresh the thumbnails.
     public var asset: AVAsset? {
         didSet {

Diff for: PryntTrimmerView/Classes/Parents/AssetVideoScrollView.swift

@@ -14,7 +14,7 @@ class AssetVideoScrollView: UIScrollView {
     private var widthConstraint: NSLayoutConstraint?
 
     let contentView = UIView()
-    var maxDuration: Double = 15
+    public var maxDuration: Double = 15
     private var generator: AVAssetImageGenerator?
 
     override init(frame: CGRect) {
@@ -116,7 +116,6 @@ class AssetVideoScrollView: UIScrollView {
     }
 
     private func getThumbnailTimes(for asset: AVAsset, numberOfThumbnails: Int) -> [NSValue] {
-
         let timeIncrement = (asset.duration.seconds * 1000) / Double(numberOfThumbnails)
         var timesForThumbnails = [NSValue]()
         for index in 0..<numberOfThumbnails {
@@ -128,11 +127,19 @@ class AssetVideoScrollView: UIScrollView {
     }
 
     private func generateImages(for asset: AVAsset, at times: [NSValue], with maximumSize: CGSize, visibleThumnails: Int) {
-
         generator = AVAssetImageGenerator(asset: asset)
-        generator?.appliesPreferredTrackTransform = true
-        let scaledSize = CGSize(width: maximumSize.width * UIScreen.main.scale, height: maximumSize.height *  UIScreen.main.scale)
+        let scaledSize = CGSize(width: maximumSize.width * UIScreen.main.scale, height: maximumSize.height * UIScreen.main.scale)
+        let composition = AVMutableVideoComposition(propertiesOf: asset)
+        var renderSize = CGSize(width: CGFloat.greatestFiniteMagnitude, height: CGFloat.greatestFiniteMagnitude)
+        for track in asset.tracks where track.mediaType == .video {
+            let size = track.naturalSize
+            if size.width <= renderSize.width && size.height <= renderSize.height {
+                renderSize = size
+            }
+        }
+        composition.renderSize = renderSize
         generator?.maximumSize = scaledSize
+        generator?.videoComposition = composition
         var count = 0
 
         let handler: AVAssetImageGeneratorCompletionHandler = { [weak self] (_, cgimage, _, result, error) in

Diff for: PryntTrimmerView/Classes/Trimmer/PryntTrimmerView.swift

@@ -71,13 +71,6 @@ public protocol TrimmerViewDelegate: class {
 
     private let handleWidth: CGFloat = 15
 
-    /// The maximum duration allowed for the trimming. Change it before setting the asset, as the asset preview
-    public var maxDuration: Double = 15 {
-        didSet {
-            assetPreview.maxDuration = maxDuration
-        }
-    }
-
     /// The minimum duration allowed for the trimming. The handles won't pan further if the minimum duration is attained.
     public var minDuration: Double = 3