Skip to content

Add implementation for getPlaylistMetadata() and setPlaylistMetadata() in CastPlayer #2235

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

Conversation

MGaetan89
Copy link
Contributor

This commit implements the getPlaylistMetadata() and setPlaylistMetadata() methods in the CastPlayer class.
The new implementations are based on what is done in ExoPlayerImpl.

MGaetan89 and others added 2 commits March 19, 2025 15:46
…ta()` in `CastPlayer`

This commit implements the `getPlaylistMetadata()` and `setPlaylistMetadata()` methods in the `CastPlayer` class.
The new implementations are based on what is done in `ExoPlayerImpl`.
@marcbaechinger marcbaechinger force-pushed the add_CastPlayer_playlistMetadata branch from 0d53f5c to 1db0fe2 Compare March 19, 2025 14:49
@marcbaechinger
Copy link
Contributor

Thanks for your pull request!

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!

@marcbaechinger marcbaechinger self-requested a review March 19, 2025 14:52
@copybara-service copybara-service bot merged commit 2d1bcc7 into androidx:main Mar 19, 2025
1 check passed
@MGaetan89 MGaetan89 deleted the add_CastPlayer_playlistMetadata branch March 19, 2025 18:13
tonihei pushed a commit that referenced this pull request Apr 14, 2025
PiperOrigin-RevId: 738455260
(cherry picked from commit 2d1bcc7)
l1068 pushed a commit to l1068org/media that referenced this pull request Apr 14, 2025
…stMetadata

PiperOrigin-RevId: 738455260
(cherry picked from commit 2d1bcc7)
@androidx androidx locked and limited conversation to collaborators May 19, 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.

None yet

2 participants