Skip to content

fix(chromium): fail 204 navigation requests - #43291

Merged
Pavel Feldman (pavelfeldman) merged 1 commit into
mainfrom
fix-43279
Oct 12, 2026
Merged

Pavel Feldman (pavelfeldman) merged 1 commit into
mainfrom
fix-43279

Conversation

@pavelfeldman

Copy link
Copy Markdown
Member

Since #42834, Chromium reported every 204 response as finished, including navigation requests. A 204 navigation never commits, and the frame only drops its pending navigation when the navigation request fails, so page.click() on a link answered with 204 waited for the navigation until the action timed out (regressed in 1.64).

This keeps reporting 204 subresources as finished (#42786) and fails 204 navigations, as Firefox and WebKit already do.

Tests: a click on a 204 link no longer waits, and a 204 navigation request emits requestfailed. Both fail on Chromium without the fix and pass on all three browsers with it.

Fixes: #43279

Since #42834, Chromium reported every 204 response as finished, including
navigation requests. A 204 navigation never commits, and the frame only
drops its pending navigation when the navigation request fails, so a click
on a link answered with 204 waited for the navigation until the action
timed out. Keep reporting 204 subresources as finished, and fail 204
navigations as Firefox and WebKit do.

Fixes: #43279
@github-actions

Copy link
Copy Markdown
Contributor

Test results for "tests 1"

7 flaky ⚠️ [installation tests] › playwright-electron-should-work.spec.ts:31 › electron should work `@package-installations-ubuntu-latest`
⚠️ [chromium-library] › library/chromium/chromium.spec.ts:213 › should intercept service worker requests (main and within) `@chromium-ubuntu-22.04-arm-node20`
⚠️ [chromium-library] › library/webmcp.spec.ts:256 › should list and call tools in cross-origin and nested frames `@chromium-ubuntu-22.04-arm-node20`
⚠️ [chromium-library] › library/chromium/chromium.spec.ts:179 › serviceWorker(), and fromServiceWorker() work `@chromium-ubuntu-22.04-node22`
⚠️ [chromium-library] › library/video.spec.ts:762 › screencast › should work with video+trace `@chromium-ubuntu-22.04-node22`
⚠️ [firefox-library] › library/browsercontext-cookies-third-party.spec.ts:257 › third party 'Partitioned;' cookies `@firefox-ubuntu-22.04-node20`
⚠️ [firefox-library] › library/browsercontext-cookies-third-party.spec.ts:470 › top level 'Partitioned;' cookie and same origin iframe `@firefox-ubuntu-22.04-node20`

53306 passed, 1266 skipped


Merge workflow run.

@github-actions

Copy link
Copy Markdown
Contributor

Test results for "MCP"

9165 passed, 1495 skipped


Merge workflow run.

@pavelfeldman
Pavel Feldman (pavelfeldman) marked this pull request as ready for review October 12, 2026 17:29
@pavelfeldman
Pavel Feldman (pavelfeldman) merged commit 5d2cd72 into main Oct 12, 2026
45 checks passed
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.

[Regression]: Chromium click on a link answered with 204 No Content times out (1.64)

2 participants