sjs/media
1
0
Fork 0
mirror of https://github.com/samsonjs/media.git synced 2026-03-25 09:25:53 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fix duration calculation for AVI files

Browse source 
The duration is now correctly calculated as the maximum of all track durations, instead of being overwritten by the last track's value. This aligns with how other Extractor implementations handle durations for multiple tracks.

PiperOrigin-RevId: 688896743
This commit is contained in:
rohks 2024-10-23 03:31:29 -07:00 committed by Copybara-Service
parent 0038dda3c3
commit e677c8dccd
6 changed files with 17 additions and 16 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
libraries/extractor/src/main/java/androidx/media3/extractor/avi/AviExtractor.java
View file

@ -15,6 +15,7 @@
*/
package androidx.media3.extractor.avi;
import static java.lang.Math.max;
import static java.lang.annotation.ElementType.TYPE_USE;
import androidx.annotation.IntDef;
@ -521,7 +522,7 @@ public final class AviExtractor implements Extractor {
ChunkReader chunkReader =
new ChunkReader(
streamId, trackType, durationUs, aviStreamHeaderChunk.length, trackOutput);
this.durationUs = durationUs;
this.durationUs = max(this.durationUs, durationUs);
return chunkReader;
} else {
// We don't currently support tracks other than video and audio.

6
libraries/test_data/src/test/assets/extractordumps/avi/sample.avi.0.dump
View file

@ -1,10 +1,10 @@
seekMap:
isSeekable = true
duration = 4080000
duration = 4087416
getPosition(0) = [[timeUs=0, position=9996]]
getPosition(1) = [[timeUs=0, position=9996]]
getPosition(2040000) = [[timeUs=1835152, position=160708], [timeUs=2335648, position=198850]]
getPosition(4080000) = [[timeUs=3837136, position=308434]]
getPosition(2043708) = [[timeUs=1835152, position=160708], [timeUs=2335648, position=198850]]
getPosition(4087416) = [[timeUs=3837136, position=308434]]
numberOfTracks = 2
track 0:
total output bytes = 252777

6
libraries/test_data/src/test/assets/extractordumps/avi/sample.avi.1.dump
View file

@ -1,10 +1,10 @@
seekMap:
isSeekable = true
duration = 4080000
duration = 4087416
getPosition(0) = [[timeUs=0, position=9996]]
getPosition(1) = [[timeUs=0, position=9996]]
getPosition(2040000) = [[timeUs=1835152, position=160708], [timeUs=2335648, position=198850]]
getPosition(4080000) = [[timeUs=3837136, position=308434]]
getPosition(2043708) = [[timeUs=1835152, position=160708], [timeUs=2335648, position=198850]]
getPosition(4087416) = [[timeUs=3837136, position=308434]]
numberOfTracks = 2
track 0:
total output bytes = 165531

6
libraries/test_data/src/test/assets/extractordumps/avi/sample.avi.2.dump
View file

@ -1,10 +1,10 @@
seekMap:
isSeekable = true
duration = 4080000
duration = 4087416
getPosition(0) = [[timeUs=0, position=9996]]
getPosition(1) = [[timeUs=0, position=9996]]
getPosition(2040000) = [[timeUs=1835152, position=160708], [timeUs=2335648, position=198850]]
getPosition(4080000) = [[timeUs=3837136, position=308434]]
getPosition(2043708) = [[timeUs=1835152, position=160708], [timeUs=2335648, position=198850]]
getPosition(4087416) = [[timeUs=3837136, position=308434]]
numberOfTracks = 2
track 0:
total output bytes = 102418

6
libraries/test_data/src/test/assets/extractordumps/avi/sample.avi.3.dump
View file

@ -1,10 +1,10 @@
seekMap:
isSeekable = true
duration = 4080000
duration = 4087416
getPosition(0) = [[timeUs=0, position=9996]]
getPosition(1) = [[timeUs=0, position=9996]]
getPosition(2040000) = [[timeUs=1835152, position=160708], [timeUs=2335648, position=198850]]
getPosition(4080000) = [[timeUs=3837136, position=308434]]
getPosition(2043708) = [[timeUs=1835152, position=160708], [timeUs=2335648, position=198850]]
getPosition(4087416) = [[timeUs=3837136, position=308434]]
numberOfTracks = 2
track 0:
total output bytes = 17833

6
libraries/test_data/src/test/assets/extractordumps/avi/sample.avi.unknown_length.dump
View file

@ -1,10 +1,10 @@
seekMap:
isSeekable = true
duration = 4080000
duration = 4087416
getPosition(0) = [[timeUs=0, position=9996]]
getPosition(1) = [[timeUs=0, position=9996]]
getPosition(2040000) = [[timeUs=1835152, position=160708], [timeUs=2335648, position=198850]]
getPosition(4080000) = [[timeUs=3837136, position=308434]]
getPosition(2043708) = [[timeUs=1835152, position=160708], [timeUs=2335648, position=198850]]
getPosition(4087416) = [[timeUs=3837136, position=308434]]
numberOfTracks = 2
track 0:
total output bytes = 252777