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

Merge pull request #1576 from colinkho:main

Browse Source 
PiperOrigin-RevId: 657990422
This commit is contained in:
Copybara-Service 2024-07-31 06:55:45 -07:00
commit 40de898b22
1 changed files with 21 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
libraries/exoplayer/src/main/java/androidx/media3/exoplayer/upstream/Loader.java
View File

@ -36,6 +36,7 @@ import java.lang.annotation.Documented;
import java.lang.annotation.Retention; import java.lang.annotation.Retention;
import java.lang.annotation.RetentionPolicy; import java.lang.annotation.RetentionPolicy;
import java.lang.annotation.Target; import java.lang.annotation.Target;
import java.util.concurrent.Executor;
import java.util.concurrent.ExecutorService; import java.util.concurrent.ExecutorService;
import java.util.concurrent.atomic.AtomicBoolean; import java.util.concurrent.atomic.AtomicBoolean;
@ -204,18 +205,33 @@ public final class Loader implements LoaderErrorThrower {
} }
} }
private final ExecutorService downloadExecutorService; private final Executor downloadExecutor;
private final Runnable downloadExecutorReleaser;
@Nullable private LoadTask<? extends Loadable> currentTask; @Nullable private LoadTask<? extends Loadable> currentTask;
@Nullable private IOException fatalError; @Nullable private IOException fatalError;
/** /**
* Constructs an instance.
*
* @param threadNameSuffix A name suffix for the loader's thread. This should be the name of the * @param threadNameSuffix A name suffix for the loader's thread. This should be the name of the
* component using the loader. * component using the loader.
*/ */
public Loader(String threadNameSuffix) { public Loader(String threadNameSuffix) {
this.downloadExecutorService = ExecutorService executorService =
Util.newSingleThreadExecutor(THREAD_NAME_PREFIX + threadNameSuffix); Util.newSingleThreadExecutor(THREAD_NAME_PREFIX + threadNameSuffix);
this.downloadExecutor = executorService;
this.downloadExecutorReleaser = executorService::shutdown;
}
/**
* Constructs an instance.
*
* @param downloadExecutor An {@link Executor} for supplying the loader's thread.
*/
public Loader(Executor downloadExecutor) {
this.downloadExecutor = downloadExecutor;
this.downloadExecutorReleaser = () -> {};
} }
/** /**
@ -297,9 +313,9 @@ public final class Loader implements LoaderErrorThrower {
currentTask.cancel(true); currentTask.cancel(true);
} }
if (callback != null) { if (callback != null) {
downloadExecutorService.execute(new ReleaseTask(callback)); downloadExecutor.execute(new ReleaseTask(callback));
} }
downloadExecutorService.shutdown(); downloadExecutorReleaser.run();
} }
// LoaderErrorThrower implementation. // LoaderErrorThrower implementation.
@ -516,7 +532,7 @@ public final class Loader implements LoaderErrorThrower {
private void execute() { private void execute() {
currentError = null; currentError = null;
downloadExecutorService.execute(Assertions.checkNotNull(currentTask)); downloadExecutor.execute(Assertions.checkNotNull(currentTask));
} }
private void finish() { private void finish() {