Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

improve buffered/seeked detection #392

Merged
merged 24 commits into from
Sep 17, 2024

Conversation

martinstark
Copy link
Collaborator

native "playing" triggers when readyState is 3 and, in some browsers, whenever it seems to feel like. It should be more correct to use "canplaythrough" to detect when a buffer or seek is completed.

@martinstark martinstark force-pushed the feat/improve-bufferd-seeked-detection branch from 0e0e2da to ec88eb0 Compare September 16, 2024 10:13
@martinstark martinstark force-pushed the feat/improve-bufferd-seeked-detection branch from ec88eb0 to bef473a Compare September 16, 2024 10:15
@martinstark
Copy link
Collaborator Author

@bwallberg added wip multi-engine (currently only Shaka) demo app to help testing this change. I'll add the last bits once other things are out of the way.

@martinstark
Copy link
Collaborator Author

(build works locally, need to look into what's problematic in actions)

@martinstark martinstark force-pushed the feat/improve-bufferd-seeked-detection branch from b61f1b2 to e4884c4 Compare September 17, 2024 09:14
@martinstark martinstark force-pushed the feat/improve-bufferd-seeked-detection branch from 422d77f to 78cc38e Compare September 17, 2024 11:16
Copy link
Collaborator

@bwallberg bwallberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@martinstark martinstark merged commit ef77416 into main Sep 17, 2024
2 checks passed
@martinstark martinstark deleted the feat/improve-bufferd-seeked-detection branch September 17, 2024 12:38
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.

2 participants