Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 72dada9c authored by Santiago Seifert's avatar Santiago Seifert
Browse files

Only log first input reader length 

It's not feasible to reliably differentiate a different
segment parsing from a seek.

Bug: 158742256
Test: atest CtsMediaParserTestCases
Test: atest CtsMediaParserHostTestCases
Change-Id: I7d62f1c2b9334bf601f38ce915b0fafdc76fbec1
Merged-In: I7d62f1c2b9334bf601f38ce915b0fafdc76fbec1
(cherry picked from commit 2eee34d2)
parent 3a565830

apex/media/framework/java/android/media/MediaParser.java

+4 −5
Original line number Diff line number Diff line
@@ -1186,11 +1186,10 @@ public final class MediaParser { 
            // TODO: For efficiency, the same implementation should be used, by providing a

            // clearBuffers() method, or similar.

            long resourceLength = seekableInputReader.getLength();

            if (resourceLength == -1) {

                mResourceByteCount = -1;

            }

            if (mResourceByteCount != -1) {

                mResourceByteCount += resourceLength;

            if (mResourceByteCount == 0) {

                // For resource byte count metric collection, we only take into account the length

                // of the first provided input reader.

                mResourceByteCount = resourceLength;

            }

            mExtractorInput =

                    new DefaultExtractorInput(