Skip to content

test(webkit): cover IndexedDB after service-worker cross-site navigation - #43755

Merged
Dmitry Gozman (dgozman) merged 1 commit into
mainfrom
triage/42694/40863787946-1
Oct 29, 2026
Merged

Dmitry Gozman (dgozman) merged 1 commit into
mainfrom
triage/42694/40863787946-1

Conversation

@microsoft-playwright-automation

Copy link
Copy Markdown
Contributor

Extend the existing service-worker navigation test to check IndexedDB transaction completion and the saved value after a cross-site round trip. This covers the WebKit process-suspension bug fixed upstream in WebKit/WebKit#74082; no implementation change is needed.

Verified with npm run test -- page-goto.spec.ts page-history.spec.ts --retries=0 --reporter=dot --workers=6 (247 passed, 2 skipped), three repetitions per current browser (9 passed), npm run build, and npm run flint. The same test times out at the post-back IndexedDB read with WebKit r2359 selected through WKPATH.

Fixes: #42694

Prepared by the triage workflow.

@github-actions

Copy link
Copy Markdown
Contributor

Test results for "MCP"

1 failed
❌ [firefox] › mcp/sse.spec.ts:196 › sse transport browser lifecycle (persistent) @mcp-ubuntu-latest-firefox

5193 passed, 616 skipped


Merge workflow run.

@github-actions

Copy link
Copy Markdown
Contributor

🟢 The reported failure is a pre-existing flake

Hi, I'm the Playwright bot. I compared the reported failure with this PR and refreshed CI history.

Details

The single failure in the latest posted report is not caused by this PR.

Pre-existing flake

  • [firefox] › mcp/sse.spec.ts:196 › sse transport browser lifecycle (persistent) on mcp-ubuntu-latest-firefox: localStorage.a returns undefined instead of "42" after reconnecting to the persistent browser. The identical test and assertion failed on main, SHA 4d05531e1483, on October 28 (run), and on unrelated PR devops: run tests through a shared CI environment action #43735 that day (run). Both precede this PR. Refreshed CI history for this same test, project, and bot on October 22–29, excluding this PR, contains 3 failed runs and 176 passed runs.

This PR only extends tests/page/page-goto.spec.ts; it changes no browser implementation, MCP code, or shared fixtures. The changed service-worker/IndexedDB test passed on Chromium, Firefox, and WebKit in this PR's library-test run.

This verdict covers the reported failure. One macOS test-runner job was still running when checked.

Triaged by the Playwright bot - agent run

@github-actions

Copy link
Copy Markdown
Contributor

Test results for "tests 1"

23593 passed, 707 skipped


Merge workflow run.

@dgozman
Dmitry Gozman (dgozman) marked this pull request as ready for review October 29, 2026 14:54
@dgozman
Dmitry Gozman (dgozman) merged commit 67a545e into main Oct 29, 2026
23 of 24 checks passed
@dgozman
Dmitry Gozman (dgozman) deleted the triage/42694/40863787946-1 branch October 29, 2026 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: WebKit 26.6 IndexedDB read stalls after cross-site back navigation when a service worker controls the page

2 participants