Skip to content

BADGER-293 Remove video frames not advancing#417

Merged
ryanmccartney merged 5 commits intomasterfrom
badger-293-remove-video-frames-not-advancing
Feb 5, 2026
Merged

BADGER-293 Remove video frames not advancing#417
ryanmccartney merged 5 commits intomasterfrom
badger-293-remove-video-frames-not-advancing

Conversation

@ryanmccartney
Copy link
Copy Markdown
Contributor

@ryanmccartney ryanmccartney commented Jan 19, 2026

📺 What

Removes videoFramesNotAdvancing feature from Dash.js. This was added during phase 2 of the low latency trial and uses the uncertified Video Quality API. The varying performance across devices of this interface led to a number of false positives.

This combined with the addition of a lowLatencyStallThreshold now in mainline Dash.js and the resolution of the underlying Chrome bug that led to this workaround mean we can now safely remove this feature.

🛠 How

Adds a temporary testing branch of a version of Dash.js without the videoFramesNotAdvancing feature.

Removes plugin interface and reporting on the PLAYBACK_FROZEN event that videoFramesNotAdvancing exposes.

@ryanmccartney ryanmccartney requested a review from a team as a code owner January 19, 2026 12:46
@ryanmccartney ryanmccartney added the semver minor This PR is a semver minor release label Jan 19, 2026
@ryanmccartney ryanmccartney merged commit 859967d into master Feb 5, 2026
4 checks passed
@ryanmccartney ryanmccartney deleted the badger-293-remove-video-frames-not-advancing branch February 5, 2026 11:49
@ryanmccartney ryanmccartney restored the badger-293-remove-video-frames-not-advancing branch February 5, 2026 16:46
@ryanmccartney ryanmccartney deleted the badger-293-remove-video-frames-not-advancing branch February 5, 2026 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver minor This PR is a semver minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants