[aaltat]

Summary

Tests only, no production code. Follow-up to #5326 and #5356.

utest/test_adopted_context.py
  browser fixture                       replaces 28 hand-written library(tmpdir, monkeypatch)
  + New Context recordVideo={}          -> Browser's video folder, default size
  + adopted options == New Context      recordVideo, recordHar, trace file for the same arguments
  + cached video size, per route        New Context / New Persistent Context / register_adopted_context
                                        x viewport 800x600 (no size) / explicit size 640x480
  ~ embedded <video>                    asserts the HTML, at 800x600
atest/
  test/01_Browser_Management/tracing_groups.py -> library/tracing_groups.py

The next PR unifies how the three routes work out the video size (_video_size at three call sites). The old tests all used 1280x720, which is also the fallback without a viewport or size, so they could not catch a call site that dropped either.

Evidence

• Before: replacing any one _video_size(...) call site with _video_size(None, None) leaves every test green.
  After: it fails that route's ..._caches_its_video_size[viewport] and [size] (and the embed test for register_adopted_context). Bypassing _record_video or _record_har on the New Context route fails the comparison test.
• utest/test_adopted_context.py: 37 passed; inv utest: 850 passed. inv atest --suite "Tracing Groups" and "Adopted Context Recording": passed with the moved helper.

Merge Danger

Door: two-way

Blast Radius: none

Tests and an acceptance-test helper only.

🤖 Generated with Claude Code