fix(chromium): report local frames of existing oopifs when connecting over CDP

[Dmitry Gozman (dgozman)]

Summary

• Handle Page.getFrameTree in OOPIF sessions the same way as in the main session, so that local child frames of an already existing OOPIF are registered and get a utility world.
• When an iframe target has an unknown parentFrameId, attach it as a child of the main frame instead of replacing the main frame.

Fixes #42955

[github-actions]

Test results for "tests 1"

8 flaky

⚠️ [chromium-library] › library/browsertype-connect.spec.ts:990 › run-server › socks proxy › should proxy requests from fetch api over ipv6 `@chromium-ubuntu-22.04-arm-node20`
⚠️ [chromium-library] › library/chromium/chromium.spec.ts:301 › should report intercepted service worker requests in HAR `@realtime-time-library-chromium-linux`
⚠️ [chromium-library] › library/beforeunload.spec.ts:130 › should support dismissing the dialog multiple times `@chromium-ubuntu-22.04-node24`
⚠️ [chromium-library] › library/video.spec.ts:762 › screencast › should work with video+trace `@chromium-ubuntu-22.04-node24`
⚠️ [chromium-library] › library/video.spec.ts:356 › screencast › should work for popups `@chromium-ubuntu-22.04-node20`
⚠️ [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`
⚠️ [playwright-test] › ui-mode-trace.spec.ts:715 › should indicate current test status `@windows-latest-node22`

52410 passed, 1243 skipped

---

Merge workflow run.

[github-actions]

Test results for "MCP"

1 failed
❌ [chromium] › mcp/http.spec.ts:145 › http transport browser sigint @mcp-windows-latest-chromium

8828 passed, 1480 skipped

---

Merge workflow run.

[github-actions]

Hi, I'm the Playwright bot and I took a first look at the CI failure here.

🟢 The one failure is a pre-existing flake

mcp/http.spec.ts:145 › http transport browser sigint failed on Windows only. This test fails on Windows every so often, including on main, and this PR's changes don't reach it.

Details

The merged MCP report (run 36637111129) has one failure and 8828 passes. This PR only changes how Chromium reports OOPIF frames on CDP connect (crPage.ts) and its test (library/chromium/oopif.spec.ts). The MCP HTTP-transport SIGINT shutdown test doesn't use that code path.

Pre-existing flake / infra

• [chromium] › mcp/http.spec.ts:145 › http transport browser sigint on mcp-windows-latest-chromium: pre-existing flake. The test-results DB has 20 other failing runs for this test, on commits unrelated to this PR. Each failed with the same expect(received).toEqual(expected) deep-equality mismatch, apart from one timeout.
  • Windows bots: firefox 9/705 runs, msedge 5/716, chromium 3/720, webkit 3/727.
  • Other PRs: fix(highlight): highlight in the world the selector resolves in #42864, docs(cli): do not suggest generating test scenarios in parallel #42833, fix(clock): make Temporal.Now follow page.clock #42838, feat(mcp): add --timeout-idle to close the browser after inactivity #42663, fix(net): apply socketTimeout from socket creation in httpRequest #42660, devops: restore npm publishing from GitHub Actions #42550, fix(debug-controller): drop recorded actions when recording session ends #42458, fix(recorder): support re-enabling recorder on the same context #42383, fix(fetch): report security details for resumed TLS sessions #42331, fix(dashboard): don't strand a stale tab selection on session switch #42290, test(mcp): fix flaky 'click link' on firefox #42275.
  • main pushes: b8627f95, 52a0e720, 59672632, c874c8a7, 4126cf5a, bd228cb1.
  • Other platforms: one timeout on mcp-ubuntu-latest-firefox (13dbf102), none on macOS.

_{Triaged by the Playwright bot - agent run}