-
Notifications
You must be signed in to change notification settings - Fork 426
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
feat: setup EME key systems for HLS as well as DASH (#629) #657
Conversation
* @return {Boolean} if this is an audio codec | ||
* @private | ||
*/ | ||
export const isAudioCodec = function(codec) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
brought these back from 1.x
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not as part of this PR, but may make sense to move this into vhs-utils.
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I may be missing something in the chain here, but:
setupEmeOptions
is called on selectedinitialmedia
from masterPlaylistController
but
sourceUpdater's createSourceBuffers
(which sets the codecs
property) isn't called until trackinfo
or handleSourceOpen
, both of which require startingMedia_
to be set on the mainSegmentLoader, which is set on handleTrackInfo_
, which only comes on handling of segment data.
Given that, would sourceUpdater's codecs
property be set at this point?
You're right, |
7e825dc
to
5ce23a7
Compare
fabb1ef
to
65b637c
Compare
65b637c
to
8bcd942
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested
Description
#629 but for master!