Skip to content

Allow shouldStartPlayback have the last rebuffer time #2113

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

Merged
merged 3 commits into from
Feb 25, 2025

Conversation

colinkho
Copy link
Contributor

@colinkho colinkho commented Feb 7, 2025

  • Add lastRebufferRealtimeMs to LoadControl.Parameters

@marcbaechinger
Copy link
Contributor

Thanks @colinkho for your PR!

I'm going to send this for internal review. Please ignore the comment around inline comments in the test class. I'm going to fix this myself and will push this to the PR once done.

@colinkho
Copy link
Contributor Author

colinkho commented Feb 16, 2025

Hi, any update on the PR here? @marcbaechinger

@marcbaechinger
Copy link
Contributor

I'm going to send this for internal review now. You may see some more commits being added as I make changes in response to review feedback. Please refrain from pushing any more substantive changes as it will complicate the internal review - thanks!

@copybara-service copybara-service bot merged commit 2c866ce into androidx:main Feb 25, 2025
1 check passed
oceanjules pushed a commit that referenced this pull request Mar 3, 2025
PiperOrigin-RevId: 730919539
(cherry picked from commit 2c866ce)
@androidx androidx locked and limited conversation to collaborators Apr 27, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants