From 33912576753638ff41e2a3c5af56a1eb7bcf5f98 Mon Sep 17 00:00:00 2001
From: Brandon-T <bthomas@brave.com>
Date: Thu, 4 Jan 2024 13:18:31 -0500
Subject: [PATCH] Fix brave/brave-ios#8585 - Fix Javascript Disabling
 <NoScript> (brave/brave-ios#8586)

- Fix Javascript Disabling not working
---
 .../BrowserViewController+WKNavigationDelegate.swift  | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/Sources/Brave/Frontend/Browser/BrowserViewController/BrowserViewController+WKNavigationDelegate.swift b/Sources/Brave/Frontend/Browser/BrowserViewController/BrowserViewController+WKNavigationDelegate.swift
index 8d6d4b4f9e8b..a1b5b28cb4fb 100644
--- a/Sources/Brave/Frontend/Browser/BrowserViewController/BrowserViewController+WKNavigationDelegate.swift
+++ b/Sources/Brave/Frontend/Browser/BrowserViewController/BrowserViewController+WKNavigationDelegate.swift
@@ -380,7 +380,18 @@ extension BrowserViewController: WKNavigationDelegate {
       if let documentTargetURL = documentTargetURL {
         let domainForShields = Domain.getOrCreate(forUrl: documentTargetURL, persistent: !isPrivateBrowsing)
         let isScriptsEnabled = !domainForShields.isShieldExpected(.NoScript, considerAllShieldsOption: true)
+        
+        // Due to a bug in iOS WKWebpagePreferences.allowsContentJavaScript does NOT work!
+        // https://github.com/brave/brave-ios/issues/8585
+        //
+        // However, the deprecated API WKWebViewConfiguration.preferences.javaScriptEnabled DOES work!
+        // Even though `configuration` is @NSCopying, somehow this actually updates the preferences LIVE!!
+        // This follows the same behaviour as Safari
+        //
+        // - Brandon T.
+        //
         preferences.allowsContentJavaScript = isScriptsEnabled
+        webView.configuration.preferences.javaScriptEnabled = isScriptsEnabled
       }
 
       // Cookie Blocking code below