Skip to content

feat(webkit): launch WebKit in WSL directly over WebSocket transport#41251

Open
yury-s wants to merge 1 commit into
microsoft:mainfrom
yury-s:fix-37036
Open

feat(webkit): launch WebKit in WSL directly over WebSocket transport#41251
yury-s wants to merge 1 commit into
microsoft:mainfrom
yury-s:fix-37036

Conversation

@yury-s

@yury-s yury-s commented Jun 12, 2026

Copy link
Copy Markdown
Member

Summary

  • Launch the Linux WebKit build inside the playwright WSL distribution via wsl.exe --remote-debugging-port=0 and connect to it over WebSocket from the Windows host (replaces the reverted intermediate proxy server).
  • Add tests_webkit_wsl workflow running the WebKit suite headed and headless on the self-hosted Windows runner.

Draft: the last commit temporarily disables other PR workflows so CI only runs tests_webkit_wsl. It will be reverted before merging.

Reference: #37036

@yury-s
feat(webkit): launch WebKit in WSL directly over WebSocket transport
7233e76 
Launch the Linux WebKit build inside the "playwright" WSL distribution
via wsl.exe with --remote-debugging-port=0 and connect to the printed
ws:// endpoint from the Windows host (reachable thanks to WSL localhost
forwarding). This replaces the reverted intermediate proxy server.

Make killForTests wait for the transport disconnect so the kill appears
atomic to clients, add a tests_webkit_wsl workflow that runs the WebKit
suite headed and headless on the self-hosted Windows pool, and update
library/page test expectations for the webkit-wsl channel.

Reference: microsoft#37036
@yury-s yury-s marked this pull request as ready for review June 12, 2026 18:07
@yury-s yury-s requested review from dcrousso and dgozman June 12, 2026 18:11
@github-actions

github-actions Bot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Test results for "MCP"

7328 passed, 1122 skipped

Merge workflow run.

@github-actions

github-actions Bot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Test results for "tests 1"

6 failed
❌ [playwright-test] › ui-mode-test-network-tab.spec.ts:482 › should support keyboard navigation for resource type filters @macos-latest-node22
❌ [playwright-test] › ui-mode-test-network-tab.spec.ts:482 › should support keyboard navigation for resource type filters @ubuntu-latest-node20
❌ [playwright-test] › ui-mode-test-network-tab.spec.ts:482 › should support keyboard navigation for resource type filters @windows-latest-node22
❌ [playwright-test] › ui-mode-test-network-tab.spec.ts:482 › should support keyboard navigation for resource type filters @ubuntu-latest-node24
❌ [playwright-test] › ui-mode-test-network-tab.spec.ts:482 › should support keyboard navigation for resource type filters @ubuntu-latest-node22
❌ [playwright-test] › ui-mode-test-network-tab.spec.ts:482 › should support keyboard navigation for resource type filters @ubuntu-latest-node26

4 flaky ⚠️ [chromium-library] › library/video.spec.ts:645 › screencast › should capture full viewport `@chromium-ubuntu-22.04-node24`
⚠️ [chromium-library] › library/chromium/chromium.spec.ts:177 › serviceWorker(), and fromServiceWorker() work `@chromium-ubuntu-22.04-node20`
⚠️ [chromium-library] › library/video.spec.ts:275 › screencast › should capture navigation `@chromium-ubuntu-22.04-node22`
⚠️ [chromium-library] › library/video.spec.ts:717 › screencast › should work with video+trace `@chromium-ubuntu-22.04-node22`

39551 passed, 743 skipped

Merge workflow run.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dcrousso dcrousso Awaiting requested review from dcrousso

@dgozman dgozman Awaiting requested review from dgozman

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@yury-s