Skip to content

cherry-pick(#41309, #43149): Revert "fix(firefox): treat navigationCommitted events without navigationId as same-document URL updates (#41224)"#41350

Merged
yury-s merged 2 commits into
microsoft:release-1.61from
dcrousso:release-1.61
Jun 18, 2026
Merged

cherry-pick(#41309, #43149): Revert "fix(firefox): treat navigationCommitted events without navigationId as same-document URL updates (#41224)"#41350
yury-s merged 2 commits into
microsoft:release-1.61from
dcrousso:release-1.61

Conversation

@dcrousso

Copy link
Copy Markdown
Contributor

No description provided.

dcrousso added 2 commits June 17, 2026 17:13
…igationId` as same-document URL updates (microsoft#41224)" (microsoft#41349)

this is a partial revert for commits 81881e2 <microsoft#41224> and c2ac911 <microsoft#41309>

leep the test added as that's still a valuable scenario to check, though it's skipped for Firefox due to the known issue

fixes <microsoft#41347>
@dcrousso dcrousso requested review from dgozman and yury-s June 17, 2026 23:16
@github-actions

Copy link
Copy Markdown
Contributor

Test results for "MCP"

7302 passed, 1122 skipped


Merge workflow run.

@github-actions

Copy link
Copy Markdown
Contributor

Test results for "tests 1"

3 flaky ⚠️ [chromium-library] › library/browsertype-connect.spec.ts:699 › run-server › should record trace with sources `@chromium-ubuntu-22.04-arm-node20`
⚠️ [firefox-page] › page/page-emulate-media.spec.ts:144 › should keep reduced motion and color emulation after reload `@firefox-ubuntu-22.04-node20`
⚠️ [playwright-test] › ui-mode-test-run.spec.ts:838 › should run test defined outside of testdir `@windows-latest-node22`

39557 passed, 744 skipped


Merge workflow run.

@yury-s yury-s merged commit b8a0fc3 into microsoft:release-1.61 Jun 18, 2026
46 checks passed
@dcrousso dcrousso deleted the release-1.61 branch June 18, 2026 05:14
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.

2 participants