[webkit-changes] [WebKit/WebKit] b30ec2: "Turn off the lights" extension service worker fai...

kiaraarose noreply at github.com
Wed Jan 17 10:15:15 PST 2024


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: b30ec2ac54474e2b9e82e0cc5bbf1b6abd7821f9
      https://github.com/WebKit/WebKit/commit/b30ec2ac54474e2b9e82e0cc5bbf1b6abd7821f9
  Author: Kiara Rose <kiara_rose at apple.com>
  Date:   2024-01-17 (Wed, 17 Jan 2024)

  Changed paths:
    M Source/WebKit/WebProcess/Extensions/API/Cocoa/WebExtensionAPIMenusCocoa.mm
    M Tools/TestWebKitAPI/Tests/WebKitCocoa/WKWebExtensionAPIMenus.mm

  Log Message:
  -----------
  "Turn off the lights" extension service worker fails to load.
https://bugs.webkit.org/show_bug.cgi?id=267590
rdar://120864698

Reviewed by Timothy Hatcher.

Turn off the light's service worker fails to load because we incorrectly throw an error for empty
titles when context menus of type "separator" are created. We should update the check to allow for
empty titles for type 'separator'. We should also remove the checks that prevent 'command', 'onclick',
'icons', 'checked', and 'enabled' from being specified if the type is 'separator' since that isn't
specified in the Chrome or MDN documentation.

Testing:
TOTL extension works as expected.

* Source/WebKit/WebProcess/Extensions/API/Cocoa/WebExtensionAPIMenusCocoa.mm:
(WebKit::WebExtensionAPIMenus::parseCreateAndUpdateProperties):
* Tools/TestWebKitAPI/Tests/WebKitCocoa/WKWebExtensionAPIMenus.mm:
(TestWebKitAPI::TEST):

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




More information about the webkit-changes mailing list