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

constructing new VideoFrame(HTMLVideoElement,...) shouldn't rely on <video>.networkState #487

Open
chcunningham opened this issue Apr 28, 2022 · 1 comment
Labels
need-definition An issues where something needs to be specified normatively

Comments

@chcunningham
Copy link
Collaborator

Right now the constructor's steps say

If image’s networkState attribute is NETWORK_EMPTY, then throw an InvalidStateError DOMException.

This was a copy paste from createImageBitmap() steps.

But networkState really doesn't have any correspondence to "is your video element ready to render a frame".

There's a related discussion in Chrome's bug tracker where play() promise resolution (and thereby readyState > HAVE_CURRENT_DATA) is proposed as the signal that a frame is ready. Some unanswered questions there, and maybe some risk to making that change.

@chcunningham
Copy link
Collaborator Author

This test demonstrates the issue https://chromium-review.googlesource.com/c/chromium/src/+/3612133

@dalecurtis dalecurtis added the need-definition An issues where something needs to be specified normatively label Mar 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
need-definition An issues where something needs to be specified normatively
Projects
None yet
Development

No branches or pull requests

2 participants