sjs/media
1
0
Fork 0
mirror of https://github.com/samsonjs/media.git synced 2026-04-10 12:05:47 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

PlayerTrackEmsgHandler: Release sample queue in release()

Browse source 
This change makes it clear the SampleQueue doesn't outlive
the wrapping PlayerTrackEmsgHandler, and releases it from
PlayerTrackEmsgHandler.release().

This change is a no-op because calling release() is the
same as reset() in this case (the behavior only differs if
a non-dummy DrmSessionManager is being used).

PiperOrigin-RevId: 289416622
This commit is contained in:
olly 2020-01-13 12:29:10 +00:00 committed by Oliver Woodman
parent cebad350fa
commit 865e014877
1 changed files with 4 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
library/dash/src/main/java/com/google/android/exoplayer2/source/dash/PlayerEmsgHandler.java
View file

@ -196,8 +196,7 @@ public final class PlayerEmsgHandler implements Handler.Callback {
/** Returns a {@link TrackOutput} that emsg messages could be written to. */
public PlayerTrackEmsgHandler newPlayerTrackEmsgHandler() {
return new PlayerTrackEmsgHandler(
new SampleQueue(allocator, DrmSessionManager.getDummyDrmSessionManager()));
return new PlayerTrackEmsgHandler(allocator);
}
/** Release this emsg handler. It should not be reused after this call. */
@ -284,9 +283,8 @@ public final class PlayerEmsgHandler implements Handler.Callback {
private final FormatHolder formatHolder;
private final MetadataInputBuffer buffer;
/* package */ PlayerTrackEmsgHandler(SampleQueue sampleQueue) {
this.sampleQueue = sampleQueue;
/* package */ PlayerTrackEmsgHandler(Allocator allocator) {
this.sampleQueue = new SampleQueue(allocator, DrmSessionManager.getDummyDrmSessionManager());
formatHolder = new FormatHolder();
buffer = new MetadataInputBuffer();
}
@ -349,7 +347,7 @@ public final class PlayerEmsgHandler implements Handler.Callback {
/** Release this track emsg handler. It should not be reused after this call. */
public void release() {
sampleQueue.reset();
sampleQueue.release();
}
// Internal methods.