fix(chromium): stop disabling BoundaryEventDispatchTracksNodeRemoval - #42294
Conversation
Playwright disabled this Chrome 144 feature, so pages under test got pre-144 boundary event semantics and could not observe behavior their users hit in real Chrome. Chromium does not restore a node's hover claim when it is re-attached in the same task, so a node re-appended out from under the pointer never gets its mouseout. Filed upstream as https://crbug.com/548251131. Fixes: microsoft#42270
The boundary events test asserted browser behavior rather than anything playwright-related. "should record slider" is still ~10% flaky on linux chromium with the feature enabled.
49ea228 to
a752288
Compare
This comment has been minimized.
This comment has been minimized.
5f8e7ea
into
microsoft:main
Test results for "MCP"3 failed 8107 passed, 1341 skipped Merge workflow run. |
|
Hi, I'm the Playwright bot and I took a first look at the CI failures here. 🟢 All three failures are pre-existing flakes — this PR looks clearThe MCP report has 3 failures, all in areas this PR doesn't touch. This PR only stops disabling the Chromium DetailsPre-existing flake / infra
The triggering "tests 1" run was also Triaged by the Playwright bot - agent run |
Test results for "tests 1"Warning The triggering workflow run ended with status 12 flaky39790 passed, 857 skipped Merge workflow run. |
Summary
BoundaryEventDispatchTracksNodeRemovalfeature, so pages under test get the same boundary event semantics as real Chrome.cli-codegen-1.spec.ts"should record slider" as fixme on linux chromium, where it stays ~10% flaky with the feature enabled.Chromium does not restore a node's hover claim when it is re-attached in the same task, so a node re-appended out from under the pointer never gets its
mouseout. Filed upstream as https://crbug.com/548251131.The feature was originally disabled in 622ddd8 for #38568, which is the slider recording flake now marked fixme.
Fixes #42270