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
load audio playlist on MANIFEST_PARSED #2340
load audio playlist on MANIFEST_PARSED #2340
Conversation
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.
Looks good to me at a high-level approach / goal. Just a few minor details from dev to be fixed.
Changes I requested were made for the unit tests. Not waiting until level playlist is loaded to load alternative audio groups (not in the video track) makes all kinds of sense to me but would like another opinion. |
…d-audio-after-manifest-parsed
34a16e2
to
3c94a82
Compare
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. |
@mad-gooze will you have time to look into the comments @robwalch raised? It seems like a nice performance boost we should add. |
This PR will...
start loading level playlist on MANIFEST_PARSED event so audio will be loaded earlier
Why is this Pull Request needed?
Now hls.js starts fetching audio playlist after loading level playlist, so video start on streams with separate audio track is delayed
With the proposed change audio playlist will be loaded after MANIFEST_PARSED
With the proposed change video will start faster.
This can be tested here.
Are there any points in the code the reviewer needs to double check?
Resolves issues:
Checklist
API or design changes are documented in API.md