triform/chromeless
1
0
Fork 0
Code Issues Pull requests 3 Projects Releases Packages Wiki Activity Actions

fix(framesink): retarget running capturer on repeated start #34

Open
triform-admin wants to merge 1 commit from codex/cv2-framesink-retarget-20260609 into codex/cv2-wave2-verification-20260522
pull from: codex/cv2-framesink-retarget-20260609
merge into: triform:codex/cv2-wave2-verification-20260522
Conversation 0 Commits 1 Files changed 3 +18 -8
triform-admin commented 2026-06-09 11:31:44 +00:00
Owner
Copy link

Implements the FrameSink retarget handoff for CV2 native WebRTC.\n\nScope:\n- CloudBrowserFrameSinkCapturer::Start now calls ChangeTarget on an already-running producer instead of returning.\n- Capturer API comment updated from idempotent/no-op to retarget semantics.\n- Existing repeated-Start unit test now asserts one producer Start and two ChangeTarget calls.\n\nValidation run here:\n- git diff --check\n- branch pushed: codex/cv2-framesink-retarget-20260609\n\nNote: I did not run the Chromium/Chromeless binary test suite locally; this checkout does not expose a lightweight local build harness and the fleet bake remains the next validation step.

Implements the FrameSink retarget handoff for CV2 native WebRTC.\n\nScope:\n- CloudBrowserFrameSinkCapturer::Start now calls ChangeTarget on an already-running producer instead of returning.\n- Capturer API comment updated from idempotent/no-op to retarget semantics.\n- Existing repeated-Start unit test now asserts one producer Start and two ChangeTarget calls.\n\nValidation run here:\n- git diff --check\n- branch pushed: codex/cv2-framesink-retarget-20260609\n\nNote: I did not run the Chromium/Chromeless binary test suite locally; this checkout does not expose a lightweight local build harness and the fleet bake remains the next validation step.
fix(framesink): retarget running capturer on repeated start
Some checks failed
CI / Lint (pull_request) Failing after 23s
Details
CI / Docs link check (pull_request) Failing after 2s
Details
CodeQL / Analyze go (pull_request) Failing after 2s
Details
CodeQL / Analyze javascript-typescript (pull_request) Failing after 3s
Details
CI / Build container image (pull_request) Failing after 42s
Details
CI / Container smoke test (pull_request) Has been skipped
Details
E2E / docker-compose + Playwright (pull_request) Failing after 30s
Details
native-peer-gate / Build chromeless:ci for gate (pull_request) Failing after 1m24s
Details
native-peer-gate / native-peer-gate-scaffold (permissive) (pull_request) Has been skipped
Details
native-peer-gate / native-peer-gate-strict (M7 gate) (pull_request) Has been skipped
Details
a1fc09634a
Some checks failed
CI / Lint (pull_request) Failing after 23s
Details
CI / Docs link check (pull_request) Failing after 2s
Details
CodeQL / Analyze go (pull_request) Failing after 2s
Details
CodeQL / Analyze javascript-typescript (pull_request) Failing after 3s
Details
CI / Build container image (pull_request) Failing after 42s
Details
CI / Container smoke test (pull_request) Has been skipped
Details
E2E / docker-compose + Playwright (pull_request) Failing after 30s
Details
native-peer-gate / Build chromeless:ci for gate (pull_request) Failing after 1m24s
Details
native-peer-gate / native-peer-gate-scaffold (permissive) (pull_request) Has been skipped
Details
native-peer-gate / native-peer-gate-strict (M7 gate) (pull_request) Has been skipped
Details
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin codex/cv2-framesink-retarget-20260609:codex/cv2-framesink-retarget-20260609
git switch codex/cv2-framesink-retarget-20260609

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch codex/cv2-wave2-verification-20260522
git merge --no-ff codex/cv2-framesink-retarget-20260609
git switch codex/cv2-framesink-retarget-20260609
git rebase codex/cv2-wave2-verification-20260522
git switch codex/cv2-wave2-verification-20260522
git merge --ff-only codex/cv2-framesink-retarget-20260609
git switch codex/cv2-framesink-retarget-20260609
git rebase codex/cv2-wave2-verification-20260522
git switch codex/cv2-wave2-verification-20260522
git merge --no-ff codex/cv2-framesink-retarget-20260609
git switch codex/cv2-wave2-verification-20260522
git merge --squash codex/cv2-framesink-retarget-20260609
git switch codex/cv2-wave2-verification-20260522
git merge --ff-only codex/cv2-framesink-retarget-20260609
git switch codex/cv2-wave2-verification-20260522
git merge codex/cv2-framesink-retarget-20260609
git push origin codex/cv2-wave2-verification-20260522
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
triform/chromeless!34
No description provided.