Visitar URL original
Move Cocoa-only IPC tests to ipc/cocoa and http/tests/ipc/cocoa by csaavedra · Pull Request #76314 · WebKit/WebKit · GitHub
Skip to content

Move Cocoa-only IPC tests to ipc/cocoa and http/tests/ipc/cocoa - #76314

Merged
webkit-commit-queue merged 1 commit into
WebKit:mainfrom
csaavedra:eng/move-cocoa-only-ipc-tests
Oct 9, 2026
Merged

webkit-commit-queue merged 1 commit into
WebKit:mainfrom
csaavedra:eng/move-cocoa-only-ipc-tests

Conversation

@csaavedra

@csaavedra csaavedra commented Oct 8, 2026 •

Copy link
Copy Markdown
Member

a92f48e

Move Cocoa-only IPC tests to ipc/cocoa and http/tests/ipc/cocoa
https://bugs.webkit.org/show_bug.cgi?id=326811

Reviewed by Alex Christensen.

These tests exercise Cocoa-only receivers or payloads (PrepareImageBufferSetsForDisplay,
AVFoundation media engines, AVContentKeySession, CoreText font palettes, NSError), so
non-Cocoa ports had to skip them one by one. Move them under the existing ipc/cocoa
directory gating, and add the same gating for a new http/tests/ipc/cocoa directory.

remotedisplaylistrecorder-drawcontrolpart-slidertrackpart-crash.html moves along with
executor.js, its only user, which needs PrepareImageBufferSetsForDisplay to create the
graphics context. nested-display-list-draw-control-part-crash.html already covers
DrawControlPart decoding on the other ports.

The [ Debug ] entries for gpu-load-error-empty-user-info-crssh.html and
media-containment-bypass-create-player.html move from TestExpectations and
platform/wk2 to the Cocoa platform files, since the directory [ Pass ] lines there
would otherwise override them.

* LayoutTests/TestExpectations:
* LayoutTests/http/tests/ipc/cocoa/gpu-load-error-empty-user-info-crssh-expected.txt: Renamed from LayoutTests/http/tests/ipc/gpu-load-error-empty-user-info-crssh-expected.txt.
* LayoutTests/http/tests/ipc/cocoa/gpu-load-error-empty-user-info-crssh.html: Renamed from LayoutTests/http/tests/ipc/gpu-load-error-empty-user-info-crssh.html.
* LayoutTests/http/tests/ipc/cocoa/webpageproxy-didfailload-failingurl-message-check-expected.txt: Renamed from LayoutTests/http/tests/ipc/webpageproxy-didfailload-failingurl-message-check-expected.txt.
* LayoutTests/http/tests/ipc/cocoa/webpageproxy-didfailload-failingurl-message-check.html: Renamed from LayoutTests/http/tests/ipc/webpageproxy-didfailload-failingurl-message-check.html.
* LayoutTests/ipc/cocoa/cache-font-null-palette-colors-crash-expected.txt: Renamed from LayoutTests/ipc/cache-font-null-palette-colors-crash-expected.txt.
* LayoutTests/ipc/cocoa/cache-font-null-palette-colors-crash.html: Renamed from LayoutTests/ipc/cache-font-null-palette-colors-crash.html.
* LayoutTests/ipc/cocoa/executor.js: Renamed from LayoutTests/ipc/executor.js.
* LayoutTests/ipc/cocoa/legacy-cdm-session-avcontentkeysession-destruction-race-expected.txt: Renamed from LayoutTests/ipc/legacy-cdm-session-avcontentkeysession-destruction-race-expected.txt.
* LayoutTests/ipc/cocoa/legacy-cdm-session-avcontentkeysession-destruction-race.html: Renamed from LayoutTests/ipc/legacy-cdm-session-avcontentkeysession-destruction-race.html.
* LayoutTests/ipc/cocoa/mark-surfaces-volatile-during-prepare-for-display-expected.txt: Renamed from LayoutTests/ipc/mark-surfaces-volatile-during-prepare-for-display-expected.txt.
* LayoutTests/ipc/cocoa/mark-surfaces-volatile-during-prepare-for-display.html: Renamed from LayoutTests/ipc/mark-surfaces-volatile-during-prepare-for-display.html.
* LayoutTests/ipc/cocoa/media-containment-bypass-create-player-expected.txt: Renamed from LayoutTests/ipc/media-containment-bypass-create-player-expected.txt.
* LayoutTests/ipc/cocoa/media-containment-bypass-create-player.html: Renamed from LayoutTests/ipc/media-containment-bypass-create-player.html.
* LayoutTests/ipc/cocoa/remotedisplaylistrecorder-drawcontrolpart-slidertrackpart-crash-expected.txt: Renamed from LayoutTests/ipc/remotedisplaylistrecorder-drawcontrolpart-slidertrackpart-crash-expected.txt.
* LayoutTests/ipc/cocoa/remotedisplaylistrecorder-drawcontrolpart-slidertrackpart-crash.html: Renamed from LayoutTests/ipc/remotedisplaylistrecorder-drawcontrolpart-slidertrackpart-crash.html.
* LayoutTests/platform/glib/TestExpectations:
* LayoutTests/platform/ios-simulator/TestExpectations:
* LayoutTests/platform/ios/TestExpectations:
* LayoutTests/platform/mac-wk2/TestExpectations:
* LayoutTests/platform/visionos/TestExpectations:
* LayoutTests/platform/wk2/TestExpectations:

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

ab1ed60

Misc iOS, visionOS, tvOS & watchOS macOS Linux Windows
✅ 🧪 style ✅ 🛠 ios ✅ 🛠 mac ✅ 🛠 wpe   🛠 win
✅ 🛠 ios-sim ✅ 🛠 mac-AS-debug ✅ 🧪 wpe-wk2   🧪 win-tests
✅ 🧪 webkitperl ✅ 🧪 ios-wk2 ✅ 🧪 api-mac ✅ 🧪 api-wpe
✅ 🧪 ios-wk2-wpt   🧪 api-mac-debug
  🧪 api-ios ✅ 🧪 mac-wk2 ✅ 🛠 gtk3-gcc
  🧪 mac-AS-debug-wk2 ✅ 🛠 gtk
✅ 🛠 vision ✅ 🧪 mac-wk2-stress ✅ 🧪 gtk-wk2
✅ 🛠 vision-sim   🧪 mac-intel-wk2 ✅ 🧪 api-gtk
✅ 🛠 🧪 unsafe-merge ✅ 🧪 vision-wk2 ✅ 🛠 playstation
✅ 🛠 tv ✅ 🧪 mac-site-isolation
✅ 🛠 tv-sim
✅ 🛠 watch
✅ 🛠 watch-sim

@csaavedra csaavedra self-assigned this Oct 8, 2026
@csaavedra csaavedra added the Tools / Tests Tools in the Tools directory, build issues, test infrastructure, and bugs in test cases label Oct 8, 2026
@webkit-early-warning-system

webkit-early-warning-system commented Oct 8, 2026 •

Copy link
Copy Markdown
Collaborator

@csaavedra
csaavedra force-pushed the eng/move-cocoa-only-ipc-tests branch from fa9bc9b to ab1ed60 Compare October 9, 2026 03:53
@webkit-early-warning-system

webkit-early-warning-system commented Oct 9, 2026 •

Copy link
Copy Markdown
Collaborator

@csaavedra csaavedra added the unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing label Oct 9, 2026
@webkit-commit-queue
webkit-commit-queue force-pushed the eng/move-cocoa-only-ipc-tests branch from ab1ed60 to 02d59a3 Compare October 9, 2026 09:37
https://bugs.webkit.org/show_bug.cgi?id=326811

Reviewed by Alex Christensen.

These tests exercise Cocoa-only receivers or payloads (PrepareImageBufferSetsForDisplay,
AVFoundation media engines, AVContentKeySession, CoreText font palettes, NSError), so
non-Cocoa ports had to skip them one by one. Move them under the existing ipc/cocoa
directory gating, and add the same gating for a new http/tests/ipc/cocoa directory.

remotedisplaylistrecorder-drawcontrolpart-slidertrackpart-crash.html moves along with
executor.js, its only user, which needs PrepareImageBufferSetsForDisplay to create the
graphics context. nested-display-list-draw-control-part-crash.html already covers
DrawControlPart decoding on the other ports.

The [ Debug ] entries for gpu-load-error-empty-user-info-crssh.html and
media-containment-bypass-create-player.html move from TestExpectations and
platform/wk2 to the Cocoa platform files, since the directory [ Pass ] lines there
would otherwise override them.

* LayoutTests/TestExpectations:
* LayoutTests/http/tests/ipc/cocoa/gpu-load-error-empty-user-info-crssh-expected.txt: Renamed from LayoutTests/http/tests/ipc/gpu-load-error-empty-user-info-crssh-expected.txt.
* LayoutTests/http/tests/ipc/cocoa/gpu-load-error-empty-user-info-crssh.html: Renamed from LayoutTests/http/tests/ipc/gpu-load-error-empty-user-info-crssh.html.
* LayoutTests/http/tests/ipc/cocoa/webpageproxy-didfailload-failingurl-message-check-expected.txt: Renamed from LayoutTests/http/tests/ipc/webpageproxy-didfailload-failingurl-message-check-expected.txt.
* LayoutTests/http/tests/ipc/cocoa/webpageproxy-didfailload-failingurl-message-check.html: Renamed from LayoutTests/http/tests/ipc/webpageproxy-didfailload-failingurl-message-check.html.
* LayoutTests/ipc/cocoa/cache-font-null-palette-colors-crash-expected.txt: Renamed from LayoutTests/ipc/cache-font-null-palette-colors-crash-expected.txt.
* LayoutTests/ipc/cocoa/cache-font-null-palette-colors-crash.html: Renamed from LayoutTests/ipc/cache-font-null-palette-colors-crash.html.
* LayoutTests/ipc/cocoa/executor.js: Renamed from LayoutTests/ipc/executor.js.
* LayoutTests/ipc/cocoa/legacy-cdm-session-avcontentkeysession-destruction-race-expected.txt: Renamed from LayoutTests/ipc/legacy-cdm-session-avcontentkeysession-destruction-race-expected.txt.
* LayoutTests/ipc/cocoa/legacy-cdm-session-avcontentkeysession-destruction-race.html: Renamed from LayoutTests/ipc/legacy-cdm-session-avcontentkeysession-destruction-race.html.
* LayoutTests/ipc/cocoa/mark-surfaces-volatile-during-prepare-for-display-expected.txt: Renamed from LayoutTests/ipc/mark-surfaces-volatile-during-prepare-for-display-expected.txt.
* LayoutTests/ipc/cocoa/mark-surfaces-volatile-during-prepare-for-display.html: Renamed from LayoutTests/ipc/mark-surfaces-volatile-during-prepare-for-display.html.
* LayoutTests/ipc/cocoa/media-containment-bypass-create-player-expected.txt: Renamed from LayoutTests/ipc/media-containment-bypass-create-player-expected.txt.
* LayoutTests/ipc/cocoa/media-containment-bypass-create-player.html: Renamed from LayoutTests/ipc/media-containment-bypass-create-player.html.
* LayoutTests/ipc/cocoa/remotedisplaylistrecorder-drawcontrolpart-slidertrackpart-crash-expected.txt: Renamed from LayoutTests/ipc/remotedisplaylistrecorder-drawcontrolpart-slidertrackpart-crash-expected.txt.
* LayoutTests/ipc/cocoa/remotedisplaylistrecorder-drawcontrolpart-slidertrackpart-crash.html: Renamed from LayoutTests/ipc/remotedisplaylistrecorder-drawcontrolpart-slidertrackpart-crash.html.
* LayoutTests/platform/glib/TestExpectations:
* LayoutTests/platform/ios-simulator/TestExpectations:
* LayoutTests/platform/ios/TestExpectations:
* LayoutTests/platform/mac-wk2/TestExpectations:
* LayoutTests/platform/visionos/TestExpectations:
* LayoutTests/platform/wk2/TestExpectations:

Canonical link: https://commits.webkit.org/323208@main
@webkit-commit-queue
webkit-commit-queue force-pushed the eng/move-cocoa-only-ipc-tests branch from 02d59a3 to a92f48e Compare October 9, 2026 09:38
@webkit-commit-queue

Copy link
Copy Markdown
Collaborator

Committed 323208@main (a92f48e): https://commits.webkit.org/323208@main

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

@webkit-commit-queue
webkit-commit-queue merged commit a92f48e into WebKit:main Oct 9, 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 Oct 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Tools / Tests Tools in the Tools directory, build issues, test infrastructure, and bugs in test cases

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants