Skip to content

chore: disable RenderDocument feature#37715

Merged
dgozman merged 1 commit intomicrosoft:mainfrom
dgozman:chore-disable-render-document
Oct 6, 2025
Merged

chore: disable RenderDocument feature#37715
dgozman merged 1 commit intomicrosoft:mainfrom
dgozman:chore-disable-render-document

Conversation

@dgozman
Copy link
Copy Markdown
Collaborator

@dgozman dgozman commented Oct 6, 2025

See #37714.

This should fix chrome-beta failures that happen when RenderDocument is enabled via fieldtrials on the particular bot, and unblock chromium-tip-of-tree rolls.

@dgozman dgozman requested a review from Skn0tt October 6, 2025 09:49
@dgozman dgozman added the CQ1 label Oct 6, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Oct 6, 2025

Test results for "tests 1"

1 failed
❌ [playwright-test] › runner.spec.ts:118 › should ignore subprocess creation error because of SIGINT @macos-latest-node18-1

1 flaky ⚠️ [firefox-library] › library/inspector/cli-codegen-1.spec.ts:1079 › cli codegen › should not throw csp directive violation errors `@firefox-ubuntu-22.04-node18`

46917 passed, 811 skipped


Merge workflow run.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Oct 6, 2025

Test results for "tests 2"

3 fatal errors, not part of any test
48 failed
❌ [chromium-library] › library/browsercontext-reuse.spec.ts:195 › reuse launch › should not cache resources @msedge-dev-macos-latest
❌ [webkit-library] › library/inspector/cli-codegen-1.spec.ts:759 › cli codegen › should await popup @webkit-headed-windows-latest
❌ [webkit-library] › library/capabilities.spec.ts:92 › should play audio @smoke @webkit-macos-15-large
❌ [webkit-library] › library/browsercontext-base-url.spec.ts:37 › should construct a new URL when a baseURL in browserType.launchPersistentContext is passed to page.goto @webkit-macos-26-xlarge
❌ [webkit-library] › library/browsercontext-cookies-third-party.spec.ts:526 › should(not) block third party cookies - persistent context @webkit-macos-26-xlarge
❌ [webkit-library] › library/client-certificates.spec.ts:832 › browser › persistentContext › should pass with matching certificates @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-1.spec.ts:28 › context.cookies() should work @smoke @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-1.spec.ts:48 › context.addCookies() should work @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-1.spec.ts:70 › context.clearCookies() should work @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-1.spec.ts:89 › should support viewport option @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-1.spec.ts:96 › should support deviceScaleFactor option @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-1.spec.ts:101 › should support userAgent option @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-1.spec.ts:111 › should support bypassCSP option @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-1.spec.ts:118 › should support javascriptEnabled option @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-1.spec.ts:129 › should support httpCredentials option @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-1.spec.ts:136 › should support offline option @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-1.spec.ts:142 › should support acceptDownloads option @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-2.spec.ts:22 › should support hasTouch option @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-2.spec.ts:28 › should work in persistent context @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-2.spec.ts:36 › should support colorScheme option @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-2.spec.ts:42 › should support reducedMotion option @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-2.spec.ts:48 › should support forcedColors option @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-2.spec.ts:54 › should support contrast option @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-2.spec.ts:60 › should support timezoneId option @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-2.spec.ts:65 › should support locale option @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-2.spec.ts:70 › should support geolocation and permissions options @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-2.spec.ts:79 › should support ignoreHTTPSErrors option @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-2.spec.ts:87 › should support extraHTTPHeaders option @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-2.spec.ts:96 › should accept userDataDir @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-2.spec.ts:104 › should accept relative userDataDir @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-2.spec.ts:111 › should restore state from userDataDir @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-2.spec.ts:138 › should create userDataDir if it does not exist @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-2.spec.ts:145 › should have default URL when launching browser @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-2.spec.ts:159 › should have passed URL when launching with ignoreDefaultArgs: true @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-2.spec.ts:194 › should fire close event for a persistent context @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-2.spec.ts:214 › should respect selectors @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-2.spec.ts:240 › should support har option @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-2.spec.ts:251 › user agent is up to date @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-2.spec.ts:259 › dialog.accept should work @webkit-macos-26-xlarge
❌ [webkit-library] › library/defaultbrowsercontext-2.spec.ts:275 › exposes browser @webkit-macos-26-xlarge
❌ [webkit-library] › library/downloads-path.spec.ts:91 › downloads path › should accept downloads in persistent context @webkit-macos-26-xlarge
❌ [webkit-library] › library/downloads-path.spec.ts:105 › downloads path › should delete downloads when persistent context closes @webkit-macos-26-xlarge
❌ [webkit-library] › library/har.spec.ts:78 › should have pages in persistent context @webkit-macos-26-xlarge
❌ [webkit-library] › library/headful.spec.ts:24 › should have default url when launching browser @smoke @webkit-macos-26-xlarge
❌ [webkit-library] › library/headful.spec.ts:30 › should close browser with beforeunload page @webkit-macos-26-xlarge
❌ [webkit-library] › library/headful.spec.ts:233 › should click in OOPIF @webkit-macos-26-xlarge
❌ [webkit-library] › library/headful.spec.ts:252 › should click bottom row w/ infobar in OOPIF @webkit-macos-26-xlarge
❌ [webkit-library] › library/multiclient.spec.ts:342 › should launch persistent @webkit-macos-26-xlarge

92 flaky ⚠️ [chromium-library] › library/permissions.spec.ts:244 › should be able to use the local-fonts API `@channel-chromium-macos-latest`
⚠️ [chromium-page] › page/page-wait-for-load-state.spec.ts:95 › should wait for load state of about:blank popup with noopener `@channel-chromium-macos-latest`
⚠️ [chromium-page] › page/workers.spec.ts:25 › Page.workers @smoke `@channel-chromium-macos-latest`
⚠️ [chromium-page] › page/workers.spec.ts:271 › should support extra http headers `@channel-chromium-macos-latest`
⚠️ [chromium-page] › page/workers.spec.ts:25 › Page.workers @smoke `@chrome-beta-macos-latest`
⚠️ [chromium-page] › page/workers.spec.ts:150 › should report network activity `@chrome-beta-macos-latest`
⚠️ [chromium-page] › page/workers.spec.ts:271 › should support extra http headers `@chrome-beta-macos-latest`
⚠️ [chromium-page] › page/workers.spec.ts:271 › should support extra http headers `@chrome-macos-latest`
⚠️ [chromium-library] › library/inspector/cli-codegen-1.spec.ts:252 › cli codegen › should work with TrustedTypes `@chrome-windows-latest`
⚠️ [chromium-library] › library/inspector/cli-codegen-2.spec.ts:139 › cli codegen › should upload multiple files `@chrome-windows-latest`
⚠️ [chromium-page] › page/workers.spec.ts:271 › should support extra http headers `@chromium-headed-macos-14-xlarge`
⚠️ [chromium-library] › library/inspector/cli-codegen-3.spec.ts:553 › cli codegen › should generate getByAltText `@chromium-headed-windows-latest`
⚠️ [chromium-library] › library/inspector/cli-codegen-pick-locator.spec.ts:35 › should update locator highlight `@chromium-headed-windows-latest`
⚠️ [chromium-library] › library/video.spec.ts:378 › screencast › should capture navigation `@chromium-headed-windows-latest`
⚠️ [chromium-page] › page/workers.spec.ts:271 › should support extra http headers `@chromium-tip-of-tree-macos-13`
⚠️ [chromium-library] › library/video.spec.ts:206 › screencast › should continue recording main page after popup closes `@chromium-tip-of-tree-macos-13--headed`
⚠️ [chromium-page] › page/workers.spec.ts:25 › Page.workers @smoke `@chromium-tip-of-tree-macos-13--headed`
⚠️ [chromium-library] › library/selector-generator.spec.ts:264 › selector generator › should separate selectors by >> `@chromium-tip-of-tree-windows-latest`
⚠️ [chromium-library] › library/trace-viewer.spec.ts:1727 › should show a popover `@chromium-tip-of-tree-windows-latest`
⚠️ [firefox-library] › library/inspector/cli-codegen-1.spec.ts:1079 › cli codegen › should not throw csp directive violation errors `@firefox-beta-macos-latest`
⚠️ [firefox-library] › library/video.spec.ts:378 › screencast › should capture navigation `@firefox-beta-macos-latest`
⚠️ [firefox-library] › library/inspector/cli-codegen-1.spec.ts:1079 › cli codegen › should not throw csp directive violation errors `@firefox-beta-ubuntu-22.04`
⚠️ [firefox-page] › page/page-emulate-media.spec.ts:144 › should keep reduced motion and color emulation after reload `@firefox-beta-ubuntu-22.04`
⚠️ [firefox-library] › library/inspector/cli-codegen-1.spec.ts:1079 › cli codegen › should not throw csp directive violation errors `@firefox-beta-windows-latest`
⚠️ [firefox-library] › library/capabilities.spec.ts:244 › requestFullscreen `@firefox-headed-macos-14-xlarge`
⚠️ [firefox-library] › library/inspector/cli-codegen-1.spec.ts:1079 › cli codegen › should not throw csp directive violation errors `@firefox-headed-macos-14-xlarge`
⚠️ [firefox-page] › page/page-wait-for-function.spec.ts:104 › should work with strict CSP policy `@firefox-headed-macos-14-xlarge`
⚠️ [firefox-library] › library/inspector/cli-codegen-1.spec.ts:1079 › cli codegen › should not throw csp directive violation errors `@firefox-headed-ubuntu-24.04`
⚠️ [firefox-page] › page/page-goto.spec.ts:83 › should work with Cross-Origin-Opener-Policy `@firefox-headed-ubuntu-24.04`
⚠️ [firefox-library] › library/browsercontext-basic.spec.ts:36 › should be able to click across browser contexts `@firefox-headed-windows-latest`
⚠️ [firefox-library] › library/browsercontext-cookies-third-party.spec.ts:441 › top level 'Partitioned;' cookie and same origin iframe `@firefox-headed-windows-latest`
⚠️ [firefox-library] › library/browsercontext-dsf.spec.ts:31 › should fetch hidpi assets `@firefox-headed-windows-latest`
⚠️ [firefox-library] › library/browsercontext-reuse.spec.ts:94 › reuse launch › should re-add binding after reset `@firefox-headed-windows-latest`
⚠️ [firefox-library] › library/browsercontext-reuse.spec.ts:195 › reuse launch › should not cache resources `@firefox-headed-windows-latest`
⚠️ [firefox-library] › library/browsertype-launch-server.spec.ts:74 › launch server › should fire "close" event during kill `@firefox-headed-windows-latest`
⚠️ [firefox-library] › library/hit-target.spec.ts:398 › should click in iframe with padding `@firefox-headed-windows-latest`
⚠️ [firefox-library] › library/inspector/cli-codegen-1.spec.ts:21 › cli codegen › should click `@firefox-headed-windows-latest`
⚠️ [firefox-library] › library/inspector/cli-codegen-1.spec.ts:120 › cli codegen › should type after clicking twice `@firefox-headed-windows-latest`
⚠️ [firefox-library] › library/tracing.spec.ts:313 › should overwrite existing file `@firefox-headed-windows-latest`
⚠️ [firefox-library] › library/tracing.spec.ts:548 › should export trace concurrently to second navigation `@firefox-headed-windows-latest`
⚠️ [firefox-library] › library/video.spec.ts:378 › screencast › should capture navigation `@firefox-headed-windows-latest`
⚠️ [firefox-page] › page/page-click-scroll.spec.ts:100 › should scroll into view element in iframe `@firefox-headed-windows-latest`
⚠️ [firefox-page] › page/page-click-timeout-4.spec.ts:20 › should timeout waiting for stable position `@firefox-headed-windows-latest`
⚠️ [firefox-page] › page/page-wait-for-function.spec.ts:104 › should work with strict CSP policy `@firefox-headed-windows-latest`
⚠️ [firefox-library] › library/inspector/cli-codegen-1.spec.ts:1079 › cli codegen › should not throw csp directive violation errors `@firefox-macos-14-large`
⚠️ [firefox-page] › page/page-event-request.spec.ts:182 › should return response body when Cross-Origin-Opener-Policy is set `@firefox-macos-14-large`
⚠️ [firefox-library] › library/inspector/cli-codegen-1.spec.ts:1079 › cli codegen › should not throw csp directive violation errors `@firefox-macos-14-xlarge`
⚠️ [firefox-library] › library/inspector/cli-codegen-aria.spec.ts:76 › should update aria snapshot highlight `@firefox-macos-14-xlarge`
⚠️ [firefox-page] › page/page-wait-for-function.spec.ts:104 › should work with strict CSP policy `@firefox-macos-14-xlarge`
⚠️ [firefox-library] › library/inspector/cli-codegen-1.spec.ts:1079 › cli codegen › should not throw csp directive violation errors `@firefox-macos-15-large`
⚠️ [firefox-page] › page/page-wait-for-function.spec.ts:104 › should work with strict CSP policy `@firefox-macos-15-large`
⚠️ [firefox-library] › library/inspector/cli-codegen-1.spec.ts:1079 › cli codegen › should not throw csp directive violation errors `@firefox-macos-15-xlarge`
⚠️ [firefox-page] › page/page-wait-for-function.spec.ts:104 › should work with strict CSP policy `@firefox-macos-15-xlarge`
⚠️ [firefox-library] › library/inspector/cli-codegen-1.spec.ts:1079 › cli codegen › should not throw csp directive violation errors `@firefox-ubuntu-24.04`
⚠️ [firefox-page] › page/page-emulate-media.spec.ts:144 › should keep reduced motion and color emulation after reload `@firefox-ubuntu-24.04`
⚠️ [firefox-library] › library/inspector/cli-codegen-1.spec.ts:1079 › cli codegen › should not throw csp directive violation errors `@firefox-windows-latest`
⚠️ [firefox-page] › page/page-wait-for-function.spec.ts:104 › should work with strict CSP policy `@firefox-windows-latest`
⚠️ [chromium-library] › library/video.spec.ts:378 › screencast › should capture navigation `@msedge-beta-macos-latest`
⚠️ [chromium-library] › library/browsercontext-proxy.spec.ts:103 › should set cookie for top-level domain `@msedge-beta-windows-latest`
⚠️ [chromium-library] › library/video.spec.ts:579 › screencast › should capture static page in persistent context @smoke `@msedge-beta-windows-latest`
⚠️ [chromium-library] › library/headful.spec.ts:114 › should close browser after context menu was triggered `@msedge-dev-macos-latest`
⚠️ [chromium-page] › page/workers.spec.ts:150 › should report network activity `@msedge-dev-macos-latest`
⚠️ [chromium-library] › library/browsercontext-proxy.spec.ts:103 › should set cookie for top-level domain `@msedge-dev-windows-latest`
⚠️ [chromium-library] › library/chromium/connect-over-cdp.spec.ts:299 › should send default User-Agent header with connect request `@msedge-dev-windows-latest`
⚠️ [chromium-library] › library/client-certificates.spec.ts:634 › browser › should pass with matching certificates in pfx format when passing as content `@msedge-dev-windows-latest`
⚠️ [chromium-library] › library/inspector/cli-codegen-pick-locator.spec.ts:35 › should update locator highlight `@msedge-dev-windows-latest`
⚠️ [chromium-library] › library/browsercontext-reuse.spec.ts:114 › reuse launch › should reset serviceworker `@msedge-macos-latest`
⚠️ [chromium-library] › library/proxy.spec.ts:93 › should proxy local network requests › by default › loopback address `@msedge-macos-latest`
⚠️ [chromium-page] › page/workers.spec.ts:271 › should support extra http headers `@msedge-macos-latest`
⚠️ [chromium-library] › library/beforeunload.spec.ts:130 › should support dismissing the dialog multiple times `@msedge-ubuntu-22.04`
⚠️ [chromium-library] › library/browsercontext-proxy.spec.ts:103 › should set cookie for top-level domain `@msedge-ubuntu-22.04`
⚠️ [chromium-library] › library/browsercontext-proxy.spec.ts:103 › should set cookie for top-level domain `@msedge-windows-latest`
⚠️ [chromium-library] › library/browsercontext-reuse.spec.ts:150 › reuse launch › should reset serviceworker that hangs in importScripts `@msedge-windows-latest`
⚠️ [chromium-library] › library/inspector/cli-codegen-pick-locator.spec.ts:35 › should update locator highlight `@msedge-windows-latest`
⚠️ [chromium-library] › library/video.spec.ts:579 › screencast › should capture static page in persistent context @smoke `@msedge-windows-latest`
⚠️ [firefox-library] › library/inspector/cli-codegen-1.spec.ts:1079 › cli codegen › should not throw csp directive violation errors `@tracing-firefox`
⚠️ [firefox-page] › page/page-event-request.spec.ts:182 › should return response body when Cross-Origin-Opener-Policy is set `@tracing-firefox`
⚠️ [firefox-page] › page/page-wait-for-function.spec.ts:104 › should work with strict CSP policy `@tracing-firefox`
⚠️ [webkit-page] › page/wheel.spec.ts:70 › should dispatch wheel events after context menu was opened `@webkit-headed-macos-14-xlarge`
⚠️ [webkit-library] › library/client-certificates.spec.ts:792 › browser › should handle rejected certificate in handshake with HTTP/2 `@webkit-headed-ubuntu-22.04`
⚠️ [webkit-library] › library/inspector/cli-codegen-1.spec.ts:434 › cli codegen › should press `@webkit-headed-ubuntu-22.04`
⚠️ [webkit-library] › library/inspector/cli-codegen-3.spec.ts:553 › cli codegen › should generate getByAltText `@webkit-headed-ubuntu-22.04`
⚠️ [webkit-library] › library/inspector/cli-codegen-3.spec.ts:805 › cli codegen › should assert value on disabled input `@webkit-headed-ubuntu-22.04`
⚠️ [webkit-library] › library/inspector/cli-codegen-1.spec.ts:94 › cli codegen › should click twice `@webkit-headed-ubuntu-24.04`
⚠️ [webkit-library] › library/browsercontext-reuse.spec.ts:270 › reuse launch › should reset mouse position `@webkit-headed-windows-latest`
⚠️ [webkit-library] › library/inspector/cli-codegen-1.spec.ts:434 › cli codegen › should press `@webkit-headed-windows-latest`
⚠️ [webkit-library] › library/inspector/cli-codegen-1.spec.ts:536 › cli codegen › should emit single keyup on ArrowDown `@webkit-headed-windows-latest`
⚠️ [webkit-library] › library/trace-viewer.spec.ts:1727 › should show a popover `@webkit-headed-windows-latest`
⚠️ [webkit-library] › library/video.spec.ts:378 › screencast › should capture navigation `@webkit-headed-windows-latest`
⚠️ [webkit-page] › page/wheel.spec.ts:70 › should dispatch wheel events after context menu was opened `@webkit-macos-14-xlarge`
⚠️ [webkit-page] › page/page-set-input-files.spec.ts:299 › should detect mime type `@webkit-ubuntu-24.04`
⚠️ [webkit-library] › library/browsercontext-reuse.spec.ts:114 › reuse launch › should reset serviceworker `@webkit-windows-latest`

254299 passed, 9904 skipped


Merge workflow run.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Oct 6, 2025

Test results for "tests others"

2 fatal errors, not part of any test

1 interrupted ⚠️ [electron-page] › page/page-history.spec.ts:205 › page.goBack during renderer-initiated navigation `@electron-macos-latest`

21591 passed, 474 skipped, 883 did not run


Merge workflow run.

@dgozman dgozman merged commit 1a20269 into microsoft:main Oct 6, 2025
103 of 112 checks passed
dgozman added a commit to dgozman/playwright that referenced this pull request Oct 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants