Disable failing tests of FlacExtractorSeekTest

PiperOrigin-RevId: 485061783
DiceTechnology · Nov 1, 2022 · 55b82ff · 55b82ff
1 parent 8030a49
commit 55b82ff
Show file tree

Hide file tree

Showing 25 changed files with 34 additions and 36 deletions.
diff --git a/...ns/ffmpeg/src/main/java/com/google/android/exoplayer2/ext/ffmpeg/FfmpegAudioRenderer.java b/...ns/ffmpeg/src/main/java/com/google/android/exoplayer2/ext/ffmpeg/FfmpegAudioRenderer.java
@@ -21,9 +21,9 @@
 
 import android.os.Handler;
 import androidx.annotation.Nullable;
-import androidx.media3.common.audio.AudioProcessor;
 import com.google.android.exoplayer2.C;
 import com.google.android.exoplayer2.Format;
+import com.google.android.exoplayer2.audio.AudioProcessor;
 import com.google.android.exoplayer2.audio.AudioRendererEventListener;
 import com.google.android.exoplayer2.audio.AudioSink;
 import com.google.android.exoplayer2.audio.AudioSink.SinkFormatSupport;

diff --git a/...ac/src/androidTest/java/com/google/android/exoplayer2/ext/flac/FlacExtractorSeekTest.java b/...ac/src/androidTest/java/com/google/android/exoplayer2/ext/flac/FlacExtractorSeekTest.java
@@ -29,6 +29,7 @@
 import com.google.android.exoplayer2.util.Util;
 import java.io.IOException;
 import java.util.List;
+import org.junit.Ignore;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 
@@ -75,6 +76,7 @@ public void seeking_seekTable_handlesSeekToZero() throws IOException {
         fileName, trackOutput, targetSeekTimeUs, extractedFrameIndex);
   }
 
+  @Ignore("Fix [internal: b/249505968] before enabling this.")
   @Test
   public void seeking_seekTable_handlesSeekToEoF() throws IOException {
     String fileName = TEST_FILE_SEEK_TABLE;
@@ -92,6 +94,7 @@ public void seeking_seekTable_handlesSeekToEoF() throws IOException {
         fileName, trackOutput, targetSeekTimeUs, extractedFrameIndex);
   }
 
+  @Ignore("Fix [internal: b/249505968] before enabling this.")
   @Test
   public void seeking_seekTable_handlesSeekingBackward() throws IOException {
     String fileName = TEST_FILE_SEEK_TABLE;
@@ -111,6 +114,7 @@ public void seeking_seekTable_handlesSeekingBackward() throws IOException {
         fileName, trackOutput, targetSeekTimeUs, extractedFrameIndex);
   }
 
+  @Ignore("Fix [internal: b/249505968] before enabling this.")
   @Test
   public void seeking_seekTable_handlesSeekingForward() throws IOException {
     String fileName = TEST_FILE_SEEK_TABLE;
@@ -158,6 +162,7 @@ public void seeking_binarySearch_handlesSeekToZero() throws IOException {
         fileName, trackOutput, targetSeekTimeUs, extractedFrameIndex);
   }
 
+  @Ignore("Fix [internal: b/249505968] before enabling this.")
   @Test
   public void seeking_binarySearch_handlesSeekToEoF() throws IOException {
     String fileName = TEST_FILE_BINARY_SEARCH;
@@ -175,6 +180,7 @@ public void seeking_binarySearch_handlesSeekToEoF() throws IOException {
         fileName, trackOutput, targetSeekTimeUs, extractedFrameIndex);
   }
 
+  @Ignore("Fix [internal: b/249505968] before enabling this.")
   @Test
   public void seeking_binarySearch_handlesSeekingBackward() throws IOException {
     String fileName = TEST_FILE_BINARY_SEARCH;
@@ -194,6 +200,7 @@ public void seeking_binarySearch_handlesSeekingBackward() throws IOException {
         fileName, trackOutput, targetSeekTimeUs, extractedFrameIndex);
   }
 
+  @Ignore("Fix [internal: b/249505968] before enabling this.")
   @Test
   public void seeking_binarySearch_handlesSeekingForward() throws IOException {
     String fileName = TEST_FILE_BINARY_SEARCH;

diff --git a/...sions/flac/src/main/java/com/google/android/exoplayer2/ext/flac/LibflacAudioRenderer.java b/...sions/flac/src/main/java/com/google/android/exoplayer2/ext/flac/LibflacAudioRenderer.java
@@ -17,9 +17,9 @@
 
 import android.os.Handler;
 import androidx.annotation.Nullable;
-import androidx.media3.common.audio.AudioProcessor;
 import com.google.android.exoplayer2.C;
 import com.google.android.exoplayer2.Format;
+import com.google.android.exoplayer2.audio.AudioProcessor;
 import com.google.android.exoplayer2.audio.AudioRendererEventListener;
 import com.google.android.exoplayer2.audio.AudioSink;
 import com.google.android.exoplayer2.audio.DecoderAudioRenderer;

diff --git a/...sions/opus/src/main/java/com/google/android/exoplayer2/ext/opus/LibopusAudioRenderer.java b/...sions/opus/src/main/java/com/google/android/exoplayer2/ext/opus/LibopusAudioRenderer.java
@@ -17,9 +17,9 @@
 
 import android.os.Handler;
 import androidx.annotation.Nullable;
-import androidx.media3.common.audio.AudioProcessor;
 import com.google.android.exoplayer2.C;
 import com.google.android.exoplayer2.Format;
+import com.google.android.exoplayer2.audio.AudioProcessor;
 import com.google.android.exoplayer2.audio.AudioRendererEventListener;
 import com.google.android.exoplayer2.audio.AudioSink;
 import com.google.android.exoplayer2.audio.AudioSink.SinkFormatSupport;

diff --git a/...x/media3/common/audio/AudioProcessor.java → ...roid/exoplayer2/audio/AudioProcessor.java b/...x/media3/common/audio/AudioProcessor.java → ...roid/exoplayer2/audio/AudioProcessor.java
@@ -13,7 +13,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-package androidx.media3.common.audio;
+package com.google.android.exoplayer2.audio;
 
 import androidx.annotation.Nullable;
 import com.google.android.exoplayer2.C;

diff --git a/...ia3/common/audio/AudioProcessorChain.java → ...exoplayer2/audio/AudioProcessorChain.java b/...ia3/common/audio/AudioProcessorChain.java → ...exoplayer2/audio/AudioProcessorChain.java
@@ -13,7 +13,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-package androidx.media3.common.audio;
+package com.google.android.exoplayer2.audio;
 
 import com.google.android.exoplayer2.PlaybackParameters;
 

diff --git a/...idx/media3/common/audio/package-info.java → ...ndroid/exoplayer2/audio/package-info.java b/...idx/media3/common/audio/package-info.java → ...ndroid/exoplayer2/audio/package-info.java
@@ -14,6 +14,6 @@
  * limitations under the License.
  */
 @NonNullApi
-package androidx.media3.common.audio;
+package com.google.android.exoplayer2.audio;
 
 import com.google.android.exoplayer2.util.NonNullApi;
diff --git a/library/core/src/main/java/com/google/android/exoplayer2/audio/BaseAudioProcessor.java b/library/core/src/main/java/com/google/android/exoplayer2/audio/BaseAudioProcessor.java
@@ -16,7 +16,6 @@
 package com.google.android.exoplayer2.audio;
 
 import androidx.annotation.CallSuper;
-import androidx.media3.common.audio.AudioProcessor;
 import com.google.errorprone.annotations.CanIgnoreReturnValue;
 import java.nio.ByteBuffer;
 import java.nio.ByteOrder;

diff --git a/.../core/src/main/java/com/google/android/exoplayer2/audio/ChannelMappingAudioProcessor.java b/.../core/src/main/java/com/google/android/exoplayer2/audio/ChannelMappingAudioProcessor.java
@@ -16,7 +16,6 @@
 package com.google.android.exoplayer2.audio;
 
 import androidx.annotation.Nullable;
-import androidx.media3.common.audio.AudioProcessor;
 import com.google.android.exoplayer2.C;
 import com.google.android.exoplayer2.Format;
 import com.google.android.exoplayer2.util.Assertions;

diff --git a/library/core/src/main/java/com/google/android/exoplayer2/audio/DecoderAudioRenderer.java b/library/core/src/main/java/com/google/android/exoplayer2/audio/DecoderAudioRenderer.java
@@ -31,7 +31,6 @@
 import androidx.annotation.IntDef;
 import androidx.annotation.Nullable;
 import androidx.annotation.RequiresApi;
-import androidx.media3.common.audio.AudioProcessor;
 import com.google.android.exoplayer2.BaseRenderer;
 import com.google.android.exoplayer2.C;
 import com.google.android.exoplayer2.ExoPlaybackException;

diff --git a/library/core/src/main/java/com/google/android/exoplayer2/audio/DefaultAudioSink.java b/library/core/src/main/java/com/google/android/exoplayer2/audio/DefaultAudioSink.java
@@ -38,13 +38,12 @@
 import androidx.annotation.IntDef;
 import androidx.annotation.Nullable;
 import androidx.annotation.RequiresApi;
-import androidx.media3.common.audio.AudioProcessor;
-import androidx.media3.common.audio.AudioProcessor.UnhandledAudioFormatException;
 import com.google.android.exoplayer2.C;
 import com.google.android.exoplayer2.ExoPlayer.AudioOffloadListener;
 import com.google.android.exoplayer2.Format;
 import com.google.android.exoplayer2.PlaybackParameters;
 import com.google.android.exoplayer2.analytics.PlayerId;
+import com.google.android.exoplayer2.audio.AudioProcessor.UnhandledAudioFormatException;
 import com.google.android.exoplayer2.util.Assertions;
 import com.google.android.exoplayer2.util.Clock;
 import com.google.android.exoplayer2.util.ConditionVariable;
@@ -99,10 +98,11 @@ private InvalidAudioTrackTimestampException(String message) {
   }
 
   /**
-   * @deprecated Use {@link androidx.media3.common.audio.AudioProcessorChain}.
+   * @deprecated Use {@link com.google.android.exoplayer2.audio.AudioProcessorChain}.
    */
   @Deprecated
-  public interface AudioProcessorChain extends androidx.media3.common.audio.AudioProcessorChain {}
+  public interface AudioProcessorChain
+      extends com.google.android.exoplayer2.audio.AudioProcessorChain {}
 
   /**
    * The default audio processor chain, which applies a (possibly empty) chain of user-defined audio
@@ -212,7 +212,7 @@ int getBufferSizeInBytes(
   public static final class Builder {
 
     private AudioCapabilities audioCapabilities;
-    @Nullable private androidx.media3.common.audio.AudioProcessorChain audioProcessorChain;
+    @Nullable private com.google.android.exoplayer2.audio.AudioProcessorChain audioProcessorChain;
     private boolean enableFloatOutput;
     private boolean enableAudioTrackPlaybackParams;
     private int offloadMode;
@@ -253,15 +253,15 @@ public Builder setAudioProcessors(AudioProcessor[] audioProcessors) {
     }
 
     /**
-     * Sets the {@link androidx.media3.common.audio.AudioProcessorChain} to process audio before
-     * playback. The instance passed in must not be reused in other sinks. Processing chains are
-     * only supported for PCM playback (not passthrough or offload).
+     * Sets the {@link com.google.android.exoplayer2.audio.AudioProcessorChain} to process audio
+     * before playback. The instance passed in must not be reused in other sinks. Processing chains
+     * are only supported for PCM playback (not passthrough or offload).
      *
      * <p>By default, no processing will be applied.
      */
     @CanIgnoreReturnValue
     public Builder setAudioProcessorChain(
-        androidx.media3.common.audio.AudioProcessorChain audioProcessorChain) {
+        com.google.android.exoplayer2.audio.AudioProcessorChain audioProcessorChain) {
       checkNotNull(audioProcessorChain);
       this.audioProcessorChain = audioProcessorChain;
       return this;
@@ -451,7 +451,7 @@ public DefaultAudioSink build() {
   private static int pendingReleaseCount;
 
   private final AudioCapabilities audioCapabilities;
-  private final androidx.media3.common.audio.AudioProcessorChain audioProcessorChain;
+  private final com.google.android.exoplayer2.audio.AudioProcessorChain audioProcessorChain;
   private final boolean enableFloatOutput;
   private final ChannelMappingAudioProcessor channelMappingAudioProcessor;
   private final TrimmingAudioProcessor trimmingAudioProcessor;

diff --git a/...core/src/main/java/com/google/android/exoplayer2/audio/FloatResamplingAudioProcessor.java b/...core/src/main/java/com/google/android/exoplayer2/audio/FloatResamplingAudioProcessor.java
@@ -15,7 +15,6 @@
  */
 package com.google.android.exoplayer2.audio;
 
-import androidx.media3.common.audio.AudioProcessor;
 import com.google.android.exoplayer2.C;
 import com.google.android.exoplayer2.Format;
 import com.google.android.exoplayer2.util.Util;

diff --git a/library/core/src/main/java/com/google/android/exoplayer2/audio/MediaCodecAudioRenderer.java b/library/core/src/main/java/com/google/android/exoplayer2/audio/MediaCodecAudioRenderer.java
@@ -33,7 +33,6 @@
 import androidx.annotation.DoNotInline;
 import androidx.annotation.Nullable;
 import androidx.annotation.RequiresApi;
-import androidx.media3.common.audio.AudioProcessor;
 import com.google.android.exoplayer2.C;
 import com.google.android.exoplayer2.ExoPlaybackException;
 import com.google.android.exoplayer2.ExoPlayer;

diff --git a/library/core/src/main/java/com/google/android/exoplayer2/audio/ResamplingAudioProcessor.java b/library/core/src/main/java/com/google/android/exoplayer2/audio/ResamplingAudioProcessor.java
@@ -15,7 +15,6 @@
  */
 package com.google.android.exoplayer2.audio;
 
-import androidx.media3.common.audio.AudioProcessor;
 import com.google.android.exoplayer2.C;
 import com.google.android.exoplayer2.Format;
 import com.google.android.exoplayer2.util.Util;

diff --git a/...core/src/main/java/com/google/android/exoplayer2/audio/SilenceSkippingAudioProcessor.java b/...core/src/main/java/com/google/android/exoplayer2/audio/SilenceSkippingAudioProcessor.java
@@ -19,7 +19,6 @@
 import static java.lang.annotation.ElementType.TYPE_USE;
 
 import androidx.annotation.IntDef;
-import androidx.media3.common.audio.AudioProcessor;
 import com.google.android.exoplayer2.C;
 import com.google.android.exoplayer2.util.Assertions;
 import com.google.android.exoplayer2.util.Util;

diff --git a/library/core/src/main/java/com/google/android/exoplayer2/audio/SonicAudioProcessor.java b/library/core/src/main/java/com/google/android/exoplayer2/audio/SonicAudioProcessor.java
@@ -18,7 +18,6 @@
 import static com.google.android.exoplayer2.util.Assertions.checkNotNull;
 
 import androidx.annotation.Nullable;
-import androidx.media3.common.audio.AudioProcessor;
 import com.google.android.exoplayer2.C;
 import com.google.android.exoplayer2.Format;
 import com.google.android.exoplayer2.util.Util;

diff --git a/library/core/src/main/java/com/google/android/exoplayer2/audio/TeeAudioProcessor.java b/library/core/src/main/java/com/google/android/exoplayer2/audio/TeeAudioProcessor.java
@@ -18,7 +18,6 @@
 import static java.lang.Math.min;
 
 import androidx.annotation.Nullable;
-import androidx.media3.common.audio.AudioProcessorChain;
 import com.google.android.exoplayer2.C;
 import com.google.android.exoplayer2.util.Assertions;
 import com.google.android.exoplayer2.util.Log;

diff --git a/.../src/test/java/com/google/android/exoplayer2/audio/SilenceSkippingAudioProcessorTest.java b/.../src/test/java/com/google/android/exoplayer2/audio/SilenceSkippingAudioProcessorTest.java
@@ -18,9 +18,9 @@
 import static com.google.common.truth.Truth.assertThat;
 import static java.lang.Math.min;
 
-import androidx.media3.common.audio.AudioProcessor.AudioFormat;
 import androidx.test.ext.junit.runners.AndroidJUnit4;
 import com.google.android.exoplayer2.C;
+import com.google.android.exoplayer2.audio.AudioProcessor.AudioFormat;
 import com.google.android.exoplayer2.util.Assertions;
 import java.nio.ByteBuffer;
 import java.nio.ByteOrder;

diff --git a/library/core/src/test/java/com/google/android/exoplayer2/audio/SonicAudioProcessorTest.java b/library/core/src/test/java/com/google/android/exoplayer2/audio/SonicAudioProcessorTest.java
@@ -18,10 +18,10 @@
 import static com.google.common.truth.Truth.assertThat;
 import static org.junit.Assert.fail;
 
-import androidx.media3.common.audio.AudioProcessor.AudioFormat;
-import androidx.media3.common.audio.AudioProcessor.UnhandledAudioFormatException;
 import androidx.test.ext.junit.runners.AndroidJUnit4;
 import com.google.android.exoplayer2.C;
+import com.google.android.exoplayer2.audio.AudioProcessor.AudioFormat;
+import com.google.android.exoplayer2.audio.AudioProcessor.UnhandledAudioFormatException;
 import org.junit.Before;
 import org.junit.Test;
 import org.junit.runner.RunWith;

diff --git a/library/core/src/test/java/com/google/android/exoplayer2/audio/TeeAudioProcessorTest.java b/library/core/src/test/java/com/google/android/exoplayer2/audio/TeeAudioProcessorTest.java
@@ -17,9 +17,9 @@
 
 import static org.mockito.Mockito.verify;
 
-import androidx.media3.common.audio.AudioProcessor.AudioFormat;
 import androidx.test.ext.junit.runners.AndroidJUnit4;
 import com.google.android.exoplayer2.C;
+import com.google.android.exoplayer2.audio.AudioProcessor.AudioFormat;
 import com.google.android.exoplayer2.audio.TeeAudioProcessor.AudioBufferSink;
 import org.junit.Before;
 import org.junit.Rule;

diff --git a/...ry/core/src/test/java/com/google/android/exoplayer2/audio/TrimmingAudioProcessorTest.java b/...ry/core/src/test/java/com/google/android/exoplayer2/audio/TrimmingAudioProcessorTest.java
@@ -17,9 +17,9 @@
 
 import static com.google.common.truth.Truth.assertThat;
 
-import androidx.media3.common.audio.AudioProcessor.AudioFormat;
 import androidx.test.ext.junit.runners.AndroidJUnit4;
 import com.google.android.exoplayer2.C;
+import com.google.android.exoplayer2.audio.AudioProcessor.AudioFormat;
 import java.nio.ByteBuffer;
 import org.junit.After;
 import org.junit.Before;

diff --git a/...c/main/java/com/google/android/exoplayer2/transformer/AudioTranscodingSamplePipeline.java b/...c/main/java/com/google/android/exoplayer2/transformer/AudioTranscodingSamplePipeline.java
@@ -21,10 +21,10 @@
 import static java.lang.Math.min;
 
 import androidx.annotation.Nullable;
-import androidx.media3.common.audio.AudioProcessor;
-import androidx.media3.common.audio.AudioProcessor.AudioFormat;
 import com.google.android.exoplayer2.C;
 import com.google.android.exoplayer2.Format;
+import com.google.android.exoplayer2.audio.AudioProcessor;
+import com.google.android.exoplayer2.audio.AudioProcessor.AudioFormat;
 import com.google.android.exoplayer2.decoder.DecoderInputBuffer;
 import com.google.android.exoplayer2.util.Util;
 import java.nio.ByteBuffer;

diff --git a/.../src/main/java/com/google/android/exoplayer2/transformer/SpeedChangingAudioProcessor.java b/.../src/main/java/com/google/android/exoplayer2/transformer/SpeedChangingAudioProcessor.java
@@ -18,8 +18,8 @@
 
 import static java.lang.Math.min;
 
-import androidx.media3.common.audio.AudioProcessor;
 import com.google.android.exoplayer2.C;
+import com.google.android.exoplayer2.audio.AudioProcessor;
 import com.google.android.exoplayer2.audio.BaseAudioProcessor;
 import com.google.android.exoplayer2.audio.SonicAudioProcessor;
 import com.google.android.exoplayer2.util.Util;

diff --git a/...rmer/src/main/java/com/google/android/exoplayer2/transformer/TransformationException.java b/...rmer/src/main/java/com/google/android/exoplayer2/transformer/TransformationException.java
@@ -22,10 +22,10 @@
 import android.os.SystemClock;
 import androidx.annotation.IntDef;
 import androidx.annotation.Nullable;
-import androidx.media3.common.audio.AudioProcessor;
-import androidx.media3.common.audio.AudioProcessor.AudioFormat;
 import com.google.android.exoplayer2.Format;
 import com.google.android.exoplayer2.PlaybackException;
+import com.google.android.exoplayer2.audio.AudioProcessor;
+import com.google.android.exoplayer2.audio.AudioProcessor.AudioFormat;
 import com.google.android.exoplayer2.util.Clock;
 import com.google.android.exoplayer2.util.FrameProcessingException;
 import com.google.android.exoplayer2.util.FrameProcessor;

diff --git a/.../test/java/com/google/android/exoplayer2/transformer/SpeedChangingAudioProcessorTest.java b/.../test/java/com/google/android/exoplayer2/transformer/SpeedChangingAudioProcessorTest.java
@@ -18,9 +18,9 @@
 import static com.google.android.exoplayer2.util.Assertions.checkArgument;
 import static com.google.common.truth.Truth.assertThat;
 
-import androidx.media3.common.audio.AudioProcessor;
 import androidx.test.ext.junit.runners.AndroidJUnit4;
 import com.google.android.exoplayer2.C;
+import com.google.android.exoplayer2.audio.AudioProcessor;
 import com.google.android.exoplayer2.util.Util;
 import java.nio.ByteBuffer;
 import java.nio.ByteOrder;