Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: c8d987888268e34c950756d816d0fdcb96c98119
      
https://github.com/WebKit/WebKit/commit/c8d987888268e34c950756d816d0fdcb96c98119
  Author: Elijah Sawyers <[email protected]>
  Date:   2025-06-11 (Wed, 11 Jun 2025)

  Changed paths:
    M 
Source/WebKit/Shared/Extensions/_WKWebExtensionRegisteredScriptsSQLiteStore.mm
    M Tools/TestWebKitAPI/Tests/WebKitCocoa/WKWebExtensionAPIScripting.mm

  Log Message:
  -----------
  Calling scripting.registerContentScripts() sometimes returns the error: 
"Error: Invalid call to scripting.registerContentScripts(). Failed to add 
content script."
https://bugs.webkit.org/show_bug.cgi?id=294298
rdar://153001967

Reviewed by Brian Weinstein.

This patch fixes a bug where quitting and relaunching Safari would cause
registered content scripts to fail to load. This was happening because we were
missing NSNumber as an allowed class to be passed into +[NSKeyedUnarchiver
unarchivedObjectOfClasses:fromData:error:].

Wrote a new test to validate this fix.

* 
Source/WebKit/Shared/Extensions/_WKWebExtensionRegisteredScriptsSQLiteStore.mm:
(-[_WKWebExtensionRegisteredScriptsSQLiteStore 
_getKeysAndValuesFromRowEnumerator:]):
* Tools/TestWebKitAPI/Tests/WebKitCocoa/WKWebExtensionAPIScripting.mm:
(TestWebKitAPI::TEST(WKWebExtensionAPIScripting, 
RegisteredScriptIsInjectedAfterContextReloads)):
(TestWebKitAPI::TEST(WKWebExtensionAPIScripting, 
PersistentRegisteredScriptIsInjectedAfterContextReloads)):

Canonical link: https://commits.webkit.org/296093@main



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to