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

feat: Add preload system to player #5897

Merged
merged 56 commits into from
Feb 2, 2024
Merged

Conversation

theodab
Copy link
Collaborator

@theodab theodab commented Nov 16, 2023

Adds a new player method, preload. This asynchronous method creates a PreloadManager object, which
will preload data for the given manifest, and which can be passed to the load method (in place of an asset URI)
in order to apply that preloaded data. This will allow for lower load latency; if you can predict what asset will
be loaded ahead of time (say, by preloading things the user is hovering their mouse over in a menu),
you can load the manifest before the user presses the load button.
Note that PreloadManagers are only meant to be used by the player instance that created them.

Closes #880

@theodab theodab added type: enhancement New feature or request type: performance A performance issue labels Nov 16, 2023
@theodab theodab marked this pull request as draft November 16, 2023 09:33
@shaka-bot
Copy link
Collaborator

Incremental code coverage: No instrumented code was changed.

@avelad avelad added this to the v5.0 milestone Nov 16, 2023
lib/player.js Outdated Show resolved Hide resolved
@avelad avelad modified the milestones: v4.7, v5.0 Dec 4, 2023
@avelad avelad added the priority: P2 Smaller impact or easy workaround label Jan 30, 2024
@theodab
Copy link
Collaborator Author

theodab commented Feb 1, 2024

I think I successfully merged with master, but the merge with #6204 was messy and I'm worried that I might have re-broken the issue.

@theodab theodab marked this pull request as ready for review February 2, 2024 03:09
@theodab
Copy link
Collaborator Author

theodab commented Feb 2, 2024

After another merge with master, I think this is fully complete and ready to go.

@theodab theodab merged commit 489b11a into shaka-project:main Feb 2, 2024
9 of 15 checks passed
@theodab theodab deleted the preloadBranch branch February 2, 2024 08:27
@shaka-bot shaka-bot added the status: archived Archived and locked; will not be updated label Apr 2, 2024
@shaka-project shaka-project locked as resolved and limited conversation to collaborators Apr 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
priority: P2 Smaller impact or easy workaround status: archived Archived and locked; will not be updated type: enhancement New feature or request type: performance A performance issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Preload API
3 participants