Reduce Allow first video time

duckduckgo · Oct 31, 2024 · c1f901d · c1f901d
1 parent 757a5b5
commit c1f901d
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/DuckDuckGo/DuckPlayer/DuckPlayerNavigationHandler.swift b/DuckDuckGo/DuckPlayer/DuckPlayerNavigationHandler.swift
@@ -793,7 +793,6 @@ extension DuckPlayerNavigationHandler: DuckPlayerNavigationHandling {
     func handleAttach(webView: WKWebView) {
 
         // Reset referrer and initial settings
-        duckPlayer.settings.allowFirstVideo = false
         referrer = .other
 
         // Ensure feature and mode are enabled
@@ -836,7 +835,7 @@ extension DuckPlayerNavigationHandler: DuckPlayerNavigationHandling {
         // This is a fallback as the WKNavigation Delegate does not
         // Always fires finishLoading (For JS Navigation) which
         // triggers handleDidFinishLoading
-        DispatchQueue.main.asyncAfter(deadline: .now() + 2.0) {
+        DispatchQueue.main.asyncAfter(deadline: .now() + 1.0) {
             self.duckPlayer.settings.allowFirstVideo = false
         }
 

diff --git a/DuckDuckGo/DuckPlayer/DuckPlayerSettings.swift b/DuckDuckGo/DuckPlayer/DuckPlayerSettings.swift
@@ -173,7 +173,11 @@ final class DuckPlayerSettingsDefault: DuckPlayerSettings {
     }
 
     /// Flag to allow the first video to play without redirection.
-    var allowFirstVideo: Bool = false
+    var allowFirstVideo: Bool = false {
+        didSet {
+            print("DP: Allow first video: \(allowFirstVideo)")
+        }
+    }
 
     /// Determines if Duck Player should open videos in a new tab.
     var openInNewTab: Bool {