Merge pull request #90 from SDWebImage/bugfix_animatedImage_rare_crash

Fix the issue when dealloc AnimatedImage's native View, the window does not exist and cause internel Crash
SDWebImage · Mar 29, 2020 · ca9922c · ca9922c
2 parents e8dcfc5 + c0f2531
commit ca9922c
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/SDWebImageSwiftUI/Classes/AnimatedImage.swift b/SDWebImageSwiftUI/Classes/AnimatedImage.swift
@@ -218,11 +218,13 @@ public struct AnimatedImage : PlatformViewRepresentable {
             // This is a hack because of Xcode 11.3 bug, the @Published does not trigger another `updateUIView` call
             // Here I have to use UIKit/AppKit API to triger the same effect (the window change implicitly cause re-render)
             if let hostingView = AnimatedImage.findHostingView(from: view) {
-                #if os(macOS)
-                hostingView.viewDidMoveToWindow()
-                #else
-                hostingView.didMoveToWindow()
-                #endif
+                if let _ = hostingView.window {
+                    #if os(macOS)
+                    hostingView.viewDidMoveToWindow()
+                    #else
+                    hostingView.didMoveToWindow()
+                    #endif
+                }
             }
             self.imageLoading.image = image
             self.imageLoading.isLoading = false