loliball/media3
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Rollback of c5e894e2d6

Browse Source 
PiperOrigin-RevId: 622292092
This commit is contained in:
Googler 2024-04-05 15:08:06 -07:00 committed by Copybara-Service
parent 2a5b9afb88
commit c2356e3989
5 changed files with 17 additions and 106 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
RELEASENOTES.md
View File

@ -82,10 +82,6 @@
if playback is ongoing or stops the service otherwise.
* UI:
* Downloads:
* Ensure that `DownloadHelper` doesn't leak unreleased `Renderer`
instances, which can eventually result in an app crashing with
`IllegalStateException: Too many receivers, total of 1000, registered
for pid` ([#1224](https://github.com/androidx/media/issues/1224)).
* OkHttp Extension:
* Cronet Extension:
* RTMP Extension:

5
libraries/exoplayer/src/main/java/androidx/media3/exoplayer/DefaultRendererCapabilitiesList.java
View File

@ -84,11 +84,6 @@ public final class DefaultRendererCapabilitiesList implements RendererCapabiliti
return rendererCapabilities;
}
@Override
public int size() {
return renderers.length;
}
@Override
public void release() {
for (Renderer renderer : renderers) {

3
libraries/exoplayer/src/main/java/androidx/media3/exoplayer/RendererCapabilitiesList.java
View File

@ -31,9 +31,6 @@ public interface RendererCapabilitiesList {
/** Returns an array of {@link RendererCapabilities}. */
RendererCapabilities[] getRendererCapabilities();
/** Returns the number of {@link RendererCapabilities}. */
int size();
/** Releases any resources associated with this {@link RendererCapabilitiesList}. */
void release();
}

79
libraries/exoplayer/src/main/java/androidx/media3/exoplayer/offline/DownloadHelper.java
View File

@ -38,12 +38,10 @@ import androidx.media3.common.util.UnstableApi;
import androidx.media3.common.util.Util;
import androidx.media3.datasource.DataSource;
import androidx.media3.datasource.TransferListener;
import androidx.media3.exoplayer.DefaultRendererCapabilitiesList;
import androidx.media3.exoplayer.ExoPlaybackException;
import androidx.media3.exoplayer.LoadingInfo;
import androidx.media3.exoplayer.Renderer;
import androidx.media3.exoplayer.RendererCapabilities;
import androidx.media3.exoplayer.RendererCapabilitiesList;
import androidx.media3.exoplayer.RenderersFactory;
import androidx.media3.exoplayer.analytics.PlayerId;
import androidx.media3.exoplayer.audio.AudioRendererEventListener;
@ -146,12 +144,12 @@ public final class DownloadHelper {
public static class LiveContentUnsupportedException extends IOException {}
/**
* @deprecated This method leaks un-released {@link Renderer} instances. There is no direct
* replacement. Equivalent functionality can be implemented by constructing the renderer
* instances, calling {@link Renderer#getCapabilities()} on each one, then releasing the
* renderers when the capabilities are no longer required.
* Extracts renderer capabilities for the renderers created by the provided renderers factory.
*
* @param renderersFactory A {@link RenderersFactory}.
* @return The {@link RendererCapabilities} for each renderer created by the {@code
* renderersFactory}.
*/
@Deprecated
public static RendererCapabilities[] getRendererCapabilities(RenderersFactory renderersFactory) {
Renderer[] renderers =
renderersFactory.createRenderers(
@ -276,9 +274,8 @@ public final class DownloadHelper {
mediaItem, castNonNull(dataSourceFactory), drmSessionManager),
trackSelectionParameters,
renderersFactory != null
? new DefaultRendererCapabilitiesList.Factory(renderersFactory)
.createRendererCapabilitiesList()
: new UnreleaseableRendererCapabilitiesList(new RendererCapabilities[0]));
? getRendererCapabilities(renderersFactory)
: new RendererCapabilities[0]);
}
/**
@ -311,7 +308,7 @@ public final class DownloadHelper {
private final MediaItem.LocalConfiguration localConfiguration;
@Nullable private final MediaSource mediaSource;
private final DefaultTrackSelector trackSelector;
private final RendererCapabilitiesList rendererCapabilities;
private final RendererCapabilities[] rendererCapabilities;
private final SparseIntArray scratchSet;
private final Handler callbackHandler;
private final Timeline.Window window;
@ -325,26 +322,6 @@ public final class DownloadHelper {
private List<ExoTrackSelection> @MonotonicNonNull [][]
immutableTrackSelectionsByPeriodAndRenderer;
/**
* @deprecated The {@link Renderer} instances used to produce {@code rendererCapabilities} must be
* kept alive for the lifetime of this {@code DownloadHelper} instance and then released (to
* avoid a resource leak). Use {@link DownloadHelper#DownloadHelper(MediaItem, MediaSource,
* TrackSelectionParameters, RendererCapabilitiesList)} instead to avoid needing to manually
* manage this bookkeeping.
*/
@Deprecated
public DownloadHelper(
MediaItem mediaItem,
@Nullable MediaSource mediaSource,
TrackSelectionParameters trackSelectionParameters,
RendererCapabilities[] rendererCapabilities) {
this(
mediaItem,
mediaSource,
trackSelectionParameters,
new UnreleaseableRendererCapabilitiesList(rendererCapabilities));
}
/**
* Creates download helper.
*
@ -353,14 +330,14 @@ public final class DownloadHelper {
* selection needs to be made.
* @param trackSelectionParameters {@link TrackSelectionParameters} for selecting tracks for
* downloading.
* @param rendererCapabilities The {@link RendererCapabilitiesList} of the renderers for which
* tracks are selected.
* @param rendererCapabilities The {@link RendererCapabilities} of the renderers for which tracks
* are selected.
*/
public DownloadHelper(
MediaItem mediaItem,
@Nullable MediaSource mediaSource,
TrackSelectionParameters trackSelectionParameters,
RendererCapabilitiesList rendererCapabilities) {
RendererCapabilities[] rendererCapabilities) {
this.localConfiguration = checkNotNull(mediaItem.localConfiguration);
this.mediaSource = mediaSource;
this.trackSelector =
@ -394,7 +371,6 @@ public final class DownloadHelper {
mediaPreparer.release();
}
trackSelector.release();
rendererCapabilities.release();
}
/**
@ -486,7 +462,7 @@ public final class DownloadHelper {
*/
public void clearTrackSelections(int periodIndex) {
assertPreparedWithMedia();
for (int i = 0; i < rendererCapabilities.size(); i++) {
for (int i = 0; i < rendererCapabilities.length; i++) {
trackSelectionsByPeriodAndRenderer[periodIndex][i].clear();
}
}
@ -545,7 +521,7 @@ public final class DownloadHelper {
// Prefer highest supported bitrate for downloads.
parametersBuilder.setForceHighestSupportedBitrate(true);
// Disable all non-audio track types supported by the renderers.
for (RendererCapabilities capabilities : rendererCapabilities.getRendererCapabilities()) {
for (RendererCapabilities capabilities : rendererCapabilities) {
@C.TrackType int trackType = capabilities.getTrackType();
parametersBuilder.setTrackTypeDisabled(
trackType, /* disabled= */ trackType != C.TRACK_TYPE_AUDIO);
@ -586,7 +562,7 @@ public final class DownloadHelper {
// Prefer highest supported bitrate for downloads.
parametersBuilder.setForceHighestSupportedBitrate(true);
// Disable all non-text track types supported by the renderers.
for (RendererCapabilities capabilities : rendererCapabilities.getRendererCapabilities()) {
for (RendererCapabilities capabilities : rendererCapabilities) {
@C.TrackType int trackType = capabilities.getTrackType();
parametersBuilder.setTrackTypeDisabled(
trackType, /* disabled= */ trackType != C.TRACK_TYPE_TEXT);
@ -718,7 +694,7 @@ public final class DownloadHelper {
checkNotNull(mediaPreparer.mediaPeriods);
checkNotNull(mediaPreparer.timeline);
int periodCount = mediaPreparer.mediaPeriods.length;
int rendererCount = rendererCapabilities.size();
int rendererCount = rendererCapabilities.length;
trackSelectionsByPeriodAndRenderer =
(List<ExoTrackSelection>[][]) new List<?>[periodCount][rendererCount];
immutableTrackSelectionsByPeriodAndRenderer =
@ -786,7 +762,7 @@ public final class DownloadHelper {
private TrackSelectorResult runTrackSelection(int periodIndex) throws ExoPlaybackException {
TrackSelectorResult trackSelectorResult =
trackSelector.selectTracks(
rendererCapabilities.getRendererCapabilities(),
rendererCapabilities,
trackGroupArrays[periodIndex],
new MediaPeriodId(mediaPreparer.timeline.getUidOfPeriod(periodIndex)),
mediaPreparer.timeline);
@ -1090,27 +1066,4 @@ public final class DownloadHelper {
// Do nothing.
}
}
private static final class UnreleaseableRendererCapabilitiesList
implements RendererCapabilitiesList {
private final RendererCapabilities[] rendererCapabilities;
private UnreleaseableRendererCapabilitiesList(RendererCapabilities[] rendererCapabilities) {
this.rendererCapabilities = rendererCapabilities;
}
@Override
public RendererCapabilities[] getRendererCapabilities() {
return rendererCapabilities;
}
@Override
public int size() {
return rendererCapabilities.length;
}
@Override
public void release() {}
}
}

32
libraries/exoplayer/src/test/java/androidx/media3/exoplayer/offline/DownloadHelperTest.java
View File

@ -29,7 +29,6 @@ import androidx.media3.common.Timeline;
import androidx.media3.common.TrackGroup;
import androidx.media3.common.TrackSelectionOverride;
import androidx.media3.common.TrackSelectionParameters;
import androidx.media3.exoplayer.DefaultRendererCapabilitiesList;
import androidx.media3.exoplayer.Renderer;
import androidx.media3.exoplayer.RenderersFactory;
import androidx.media3.exoplayer.offline.DownloadHelper.Callback;
@ -40,7 +39,6 @@ import androidx.media3.exoplayer.trackselection.DefaultTrackSelector;
import androidx.media3.exoplayer.trackselection.ExoTrackSelection;
import androidx.media3.exoplayer.trackselection.MappingTrackSelector.MappedTrackInfo;
import androidx.media3.exoplayer.upstream.Allocator;
import androidx.media3.test.utils.FakeDataSource;
import androidx.media3.test.utils.FakeMediaPeriod;
import androidx.media3.test.utils.FakeMediaSource;
import androidx.media3.test.utils.FakeRenderer;
@ -125,8 +123,7 @@ public class DownloadHelperTest {
testMediaItem,
new TestMediaSource(),
DownloadHelper.DEFAULT_TRACK_SELECTOR_PARAMETERS_WITHOUT_CONTEXT,
new DefaultRendererCapabilitiesList.Factory(renderersFactory)
.createRendererCapabilitiesList());
DownloadHelper.getRendererCapabilities(renderersFactory));
}
@Test
@ -442,33 +439,6 @@ public class DownloadHelperTest {
new StreamKey(/* periodIndex= */ 0, /* groupIndex= */ 2, /* streamIndex= */ 0));
}
// https://github.com/androidx/media/issues/1224
@Test
public void prepareThenRelease_renderersReleased() throws Exception {
// We can't use this.downloadHelper because we need access to the FakeRenderer instances for
// later assertions, so we recreate a local DownloadHelper.
FakeRenderer videoRenderer = new FakeRenderer(C.TRACK_TYPE_VIDEO);
FakeRenderer audioRenderer = new FakeRenderer(C.TRACK_TYPE_AUDIO);
FakeRenderer textRenderer = new FakeRenderer(C.TRACK_TYPE_TEXT);
RenderersFactory renderersFactory =
(handler, videoListener, audioListener, metadata, text) ->
new Renderer[] {textRenderer, audioRenderer, videoRenderer};
DownloadHelper downloadHelper =
DownloadHelper.forMediaItem(
testMediaItem,
DownloadHelper.DEFAULT_TRACK_SELECTOR_PARAMETERS_WITHOUT_CONTEXT,
renderersFactory,
new FakeDataSource.Factory());
prepareDownloadHelper(downloadHelper);
downloadHelper.release();
assertThat(videoRenderer.isReleased).isTrue();
assertThat(audioRenderer.isReleased).isTrue();
assertThat(textRenderer.isReleased).isTrue();
}
private static void prepareDownloadHelper(DownloadHelper downloadHelper) throws Exception {
AtomicReference<Exception> prepareException = new AtomicReference<>(null);
CountDownLatch preparedLatch = new CountDownLatch(1);