Skip to content
This repository has been archived by the owner on Jul 30, 2019. It is now read-only.

4.7.10.10 Media resources with multiple media tracks code example #889

Closed
r-romaniuk opened this issue Apr 28, 2017 · 4 comments
Closed

Comments

@r-romaniuk
Copy link
Contributor

There is an inconsistency between 4.7.10.10 Media resources with multiple media tracks code example and loadedmetadata event specification. In HAVE_METADATA state
enough of the resource has been obtained that the duration of the resource is available. In the case of a video element, the dimensions of the video are also available. (According to this video tracsk doesn't have to be ready yet)
According to the specification this example should rely on onaddtrack event.

@chaals
Copy link
Collaborator

chaals commented Apr 29, 2017

@r-romaniuk thank you. Would you like to propose a Pull Request that fixes the issue?

The simplest way may be to edit the relevant page in the Github interface as it will then guide you through the steps. Alternatively, you can clone the repository and create a branch, or fork it...

@r-romaniuk
Copy link
Contributor Author

I've created a pull request #899 but there are some issues regarding this pull request, I'm not sure what is wrong.

@chaals
Copy link
Collaborator

chaals commented May 9, 2017

@r-romaniuk the short answer is that because you are not formally a member of the Working Group, we need to check for IPR commitment if the pull request changes the substantive content of the spec.

It doesn't so it's fairly easy to manage (one of the chairs or W3C staff has to follow a simple procedure). If it happens and there is a substantive (i.e. conformance-affecting) change one of us will be in touch with you asking for your agreement to the terms of the W3C Patent Policy, as per the CONTIBUTING guidelines…

Meanwhile thank you for the PR!

@chaals
Copy link
Collaborator

chaals commented Oct 10, 2017

Fixed by #899

@chaals chaals closed this as completed Oct 10, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants