Fix closing full-screen videos with the X button

duckduckgo · May 9, 2024 · fd13d64 · fd13d64
1 parent 88a7504
commit fd13d64
Showing 1 changed file with 0 additions and 19 deletions.
diff --git a/DuckDuckGo/Tab/View/WebViewContainerView.swift b/DuckDuckGo/Tab/View/WebViewContainerView.swift
@@ -20,7 +20,6 @@ import AppKit
 import Combine
 
 final class WebViewContainerView: NSView {
-    @objc
     let webView: WebView
     weak var tab: Tab?
 
@@ -51,7 +50,6 @@ final class WebViewContainerView: NSView {
     }
 
     private var blurViewIsHiddenCancellable: AnyCancellable?
-    private var fullScreenWindowWillCloseCancellable: AnyCancellable?
     private var cancellables = Set<AnyCancellable>()
 
     override func didAddSubview(_ subview: NSView) {
@@ -140,23 +138,6 @@ final class WebViewContainerView: NSView {
                 }
             }
             .store(in: &cancellables)
-
-        fullScreenWindowWillCloseCancellable = NotificationCenter.default.publisher(for: NSWindow.willCloseNotification, object: fullScreenWindow)
-            .sink { [weak self] notification in
-                self?.fullScreenWindowWillCloseCancellable = nil
-                let fullScreenWindowController = (notification.object as? NSWindow)?.windowController
-                DispatchQueue.main.async { [weak fullScreenWindowController] in
-                    guard let fullScreenWindowController else { return }
-                    // just in case.
-                    // if WKFullScreenWindowController receives `close()` the next time it‘s open it will crash because its _webView is nil
-                    // https://errors.duckduckgo.com/organizations/ddg/issues/3411/?project=6&referrer=release-issue-stream
-                    NSException.try {
-                        fullScreenWindowController.setValue(NSView(), forKeyPath: #keyPath(webView))
-                    }
-
-                }
-            }
-
     }
 
     override func removeFromSuperview() {