Skip to content

fix: EXPKITS-1441 preserve stream events during WebRTC startup - #187

Draft
Szendvics wants to merge 2 commits into
mainfrom
feature/EXPKITS-1441-public-webrtc-startup-01a162dd
Draft

Szendvics wants to merge 2 commits into
mainfrom
feature/EXPKITS-1441-public-webrtc-startup-01a162dd

Conversation

@Szendvics

@Szendvics Szendvics commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

Goal

Start WebRTC playback when a browser connects before video arrives.

Change

Preserve startup stream events, drop flushes only while the WebRTC pad has neither CAPS nor an input target, and wait for PLAYING before the initial loop seek. The flush handler uses one return.

Keep the delayed RTP fixture separate. The regular smoke source runs until teardown so the startup test cannot consume its remaining frames.

Testing

  • Full Meson CI: 83/83 passed. The startup flush test uses native pad lookup and also passes without Python iteration support for Gst.Iterator. Both startup regressions fail when their corresponding protection is disabled.
  • Chromium smoke: 7/7 passed on x86_64 with the browser and pipelines in the same Linux container.
  • Repository pre-commit checks and the debug build passed.

The separate host-to-Docker Desktop run received no video in either live test. That local network path remains unvalidated.

@Szendvics
Szendvics added this pull request to stack #188 October 22, 2026 13:49
@Szendvics
Szendvics force-pushed the feature/EXPKITS-1441-public-mlwl-recipes-01a162dd branch from 0685963 to d70608c Compare October 22, 2026 13:55
@Szendvics
Szendvics force-pushed the feature/EXPKITS-1441-public-webrtc-startup-01a162dd branch from 27c7142 to aece07b Compare October 22, 2026 13:55
@Szendvics
Szendvics force-pushed the feature/EXPKITS-1441-public-mlwl-recipes-01a162dd branch from d70608c to 501eddc Compare October 22, 2026 14:01
@Szendvics
Szendvics force-pushed the feature/EXPKITS-1441-public-webrtc-startup-01a162dd branch from aece07b to 452d14d Compare October 26, 2026 11:55
@Szendvics
Szendvics removed this pull request from stack #188 October 26, 2026 11:55
@Szendvics
Szendvics changed the base branch from feature/EXPKITS-1441-public-mlwl-recipes-01a162dd to feature/EXPKITS-1441-public-mlwl-models-01a162dd October 26, 2026 11:56
@Szendvics
Szendvics added this pull request to stack #199 October 26, 2026 11:56
@Szendvics
Szendvics force-pushed the feature/EXPKITS-1441-public-webrtc-startup-01a162dd branch from 452d14d to 4d70918 Compare October 26, 2026 11:57
@Szendvics
Szendvics removed this pull request from stack #199 October 26, 2026 11:58
@Szendvics
Szendvics changed the base branch from feature/EXPKITS-1441-public-mlwl-models-01a162dd to main October 26, 2026 11:58
Task: EXPKITS-1441

Wait for PLAYING before the initial loop seek, preserve sticky events,
and handle flushes before RTP startup. Cover delayed video in the browser.

Signed-off-by: Marcell Varga <marcell.varga@arm.com>
@Szendvics
Szendvics force-pushed the feature/EXPKITS-1441-public-webrtc-startup-01a162dd branch from 4d70918 to cc44672 Compare October 28, 2026 12:02
Task: EXPKITS-1441

Avoid relying on the optional Gst.Iterator Python iteration override.

Signed-off-by: Marcell Varga <marcell.varga@arm.com>

This branch has not been deployed

No deployments
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.

1 participant