Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: b5ff6a931f2d3000b5b6fc1b4178ac5c77ce950a
      
https://github.com/WebKit/WebKit/commit/b5ff6a931f2d3000b5b6fc1b4178ac5c77ce950a
  Author: Kiara Rose <[email protected]>
  Date:   2024-02-22 (Thu, 22 Feb 2024)

  Changed paths:
    M Source/WebKit/UIProcess/Extensions/Cocoa/WebExtensionCommandCocoa.mm

  Log Message:
  -----------
  Crash with Turn Off the Lights extension enabled
https://bugs.webkit.org/show_bug.cgi?id=269924
rdar://123323719

Reviewed by Timothy Hatcher.

This assertion is being hit:
*** Assertion failure in -[_UIMenuBuilder 
_insertMenu:intoParentMenu:newParentMenu:], UIMenuBuilder.m:443

This is because "duplicate" commands are being inserting into the menu. Per 
UIKit,
commands cannot have the same selector if their propertyList is the same. To 
fix this,
pass in a propertyList to each command so they can be distinguished from one 
other.

* Source/WebKit/UIProcess/Extensions/Cocoa/WebExtensionCommandCocoa.mm:
(+[_WKWebExtensionKeyCommand 
commandWithTitle:image:input:modifierFlags:handler:]):

Canonical link: https://commits.webkit.org/275193@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