Don't call MediaItem.Builder#setKeySetId without setting the DRM UUID

This is known to silently drop the value. This setter is now deprecated
in favour of `MediaItem.Builder#setDrmConfiguration(MediaItem.DrmConfiguration)`,
which requires a UUID in order to construct the `DrmConfiguration`
instance.

Issue: #9378 tracks correctly propagating the DRM info out of
`DownloadRequest#toMediaItem`.
PiperOrigin-RevId: 397291013
This commit is contained in:
ibaker 2021-09-17 12:36:52 +01:00 committed by Christos Tsilopoulos
parent 13827186aa
commit 04943db71c
2 changed files with 0 additions and 2 deletions

View File

@ -98,7 +98,6 @@ public class DefaultDownloaderFactory implements DownloaderFactory {
.setUri(request.uri) .setUri(request.uri)
.setStreamKeys(request.streamKeys) .setStreamKeys(request.streamKeys)
.setCustomCacheKey(request.customCacheKey) .setCustomCacheKey(request.customCacheKey)
.setDrmKeySetId(request.keySetId)
.build(); .build();
try { try {
return constructor.newInstance(mediaItem, cacheDataSourceFactory, executor); return constructor.newInstance(mediaItem, cacheDataSourceFactory, executor);

View File

@ -229,7 +229,6 @@ public final class DownloadRequest implements Parcelable {
.setCustomCacheKey(customCacheKey) .setCustomCacheKey(customCacheKey)
.setMimeType(mimeType) .setMimeType(mimeType)
.setStreamKeys(streamKeys) .setStreamKeys(streamKeys)
.setDrmKeySetId(keySetId)
.build(); .build();
} }