Skip to content

REGRESSION(303293@main): Broke watchOS debug build ('UIContextMenuInteractionDelegate' is unavailable)#60510

Merged
webkit-commit-queue merged 1 commit intoWebKit:mainfrom
aprotyas:eng/309837
Mar 13, 2026
Merged

REGRESSION(303293@main): Broke watchOS debug build ('UIContextMenuInteractionDelegate' is unavailable)#60510
webkit-commit-queue merged 1 commit intoWebKit:mainfrom
aprotyas:eng/309837

Conversation

@aprotyas
Copy link
Member

@aprotyas aprotyas commented Mar 12, 2026

e03b1b9

REGRESSION(303293@main): Broke watchOS debug build ('UIContextMenuInteractionDelegate' is unavailable)
https://bugs.webkit.org/show_bug.cgi?id=309837
rdar://172417044

Unreviewed build fix.

TWKAPI was not building on watchOS because `TestContextMenuDelegate` in
CaptionPreferencesTests.mm adhered to UIContextMenuInteractionDelegate,
which is not available on watchOS.

On a second read, this delegate is not used at all in the test suite, so
let's just remove it.

* Tools/TestWebKitAPI/Tests/WebCore/cocoa/CaptionPreferencesTests.mm:
(-[TestContextMenuDelegate contextMenuInteraction:configurationForMenuAtLocation:]): Deleted.

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

95a7672

Misc iOS, visionOS, tvOS & watchOS macOS Linux Windows Apple Internal
✅ 🧪 style ✅ 🛠 ios ✅ 🛠 mac ✅ 🛠 wpe ✅ 🛠 win ⏳ 🛠 ios-apple
✅ 🧪 bindings ✅ 🛠 ios-sim ✅ 🛠 mac-AS-debug ✅ 🧪 wpe-wk2 🧪 win-tests ⏳ 🛠 mac-apple
✅ 🧪 webkitperl ✅ 🧪 ios-wk2 ✅ 🧪 api-mac ✅ 🧪 api-wpe ✅ 🛠 vision-apple
✅ 🧪 ios-wk2-wpt 🧪 api-mac-debug ✅ 🛠 gtk3-libwebrtc
✅ 🧪 api-ios ✅ 🧪 mac-wk1 ✅ 🛠 gtk
✅ 🧪 mac-wk2 ✅ 🧪 gtk-wk2
✅ 🛠 vision ✅ 🧪 mac-AS-debug-wk2 ✅ 🧪 api-gtk
✅ 🛠 vision-sim ✅ 🧪 mac-wk2-stress ✅ 🛠 playstation
✅ 🛠 🧪 unsafe-merge ✅ 🧪 vision-wk2 ✅ 🧪 mac-intel-wk2
✅ 🛠 tv
✅ 🛠 tv-sim
✅ 🛠 watch
✅ 🛠 watch-sim

@aprotyas aprotyas self-assigned this Mar 12, 2026
@aprotyas aprotyas added the New Bugs Unclassified bugs are placed in this component until the correct component can be determined. label Mar 12, 2026
@aprotyas aprotyas requested review from Phinny01 and jernoble March 12, 2026 22:39
@aprotyas
Copy link
Member Author

Thank you for the review!

@aprotyas aprotyas added the unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing label Mar 13, 2026
…eractionDelegate' is unavailable)

https://bugs.webkit.org/show_bug.cgi?id=309837
rdar://172417044

Unreviewed build fix.

TWKAPI was not building on watchOS because `TestContextMenuDelegate` in
CaptionPreferencesTests.mm adhered to UIContextMenuInteractionDelegate,
which is not available on watchOS.

On a second read, this delegate is not used at all in the test suite, so
let's just remove it.

* Tools/TestWebKitAPI/Tests/WebCore/cocoa/CaptionPreferencesTests.mm:
(-[TestContextMenuDelegate contextMenuInteraction:configurationForMenuAtLocation:]): Deleted.

Canonical link: https://commits.webkit.org/309185@main
@webkit-commit-queue
Copy link
Collaborator

Committed 309185@main (e03b1b9): https://commits.webkit.org/309185@main

Reviewed commits have been landed. Closing PR #60510 and removing active labels.

@webkit-commit-queue webkit-commit-queue merged commit e03b1b9 into WebKit:main Mar 13, 2026
@webkit-commit-queue webkit-commit-queue removed the unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing label Mar 13, 2026
@aprotyas aprotyas deleted the eng/309837 branch March 13, 2026 07:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

New Bugs Unclassified bugs are placed in this component until the correct component can be determined.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants