From 230f4e5bb5c7e51ebceb645558adb833bd0db1a2 Mon Sep 17 00:00:00 2001
From: krocard <krocard@google.com>
Date: Thu, 15 Oct 2020 08:12:32 +0100
Subject: [PATCH] Do not inherit directly from AudioTrack.StreamEventCallback

This was causing issues old devices where the class
inheriting StreamEventCallback was loaded even though
it was not used.

Instead use an anonymous class that seem to be loaded
more lazily.

PiperOrigin-RevId: 337252687
---
 RELEASENOTES.md                               |  3 ++
 .../exoplayer2/audio/DefaultAudioSink.java    | 52 +++++++++++--------
 2 files changed, 33 insertions(+), 22 deletions(-)

diff --git a/RELEASENOTES.md b/RELEASENOTES.md
index bc44257f74..221c6dca2c 100644
--- a/RELEASENOTES.md
+++ b/RELEASENOTES.md
@@ -48,6 +48,9 @@
     *   Fix the default audio sink position not advancing correctly when using
         `AudioTrack`-based speed adjustment
         ([#7982](https://github.com/google/ExoPlayer/issues/7982)).
+    *   Fix `NoClassDefFoundError` warning for `AudioTrack$StreamEventCallback`
+        even though the class was not used
+        ([#8058](https://github.com/google/ExoPlayer/issues/8058)).
 *   Extractors:
     *   Add support for .mp2 boxes in the `AtomParsers`
         ([#7967](https://github.com/google/ExoPlayer/issues/7967)).
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
index 19fbcb6d6a..fa369ec454 100644
--- 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
@@ -1731,39 +1731,47 @@ public final class DefaultAudioSink implements AudioSink {
   }
 
   @RequiresApi(29)
-  private final class StreamEventCallbackV29 extends AudioTrack.StreamEventCallback {
+  private final class StreamEventCallbackV29 {
     private final Handler handler;
+    private final AudioTrack.StreamEventCallback callback;
 
     public StreamEventCallbackV29() {
       handler = new Handler();
-    }
+      // StreamEventCallbackV29 can NOT inherit directly from AudioTrack.StreamEventCallback as it
+      // would cause a NoClassDefFoundError on the first load of DefaultAudioSink for SDK < 29
+      // fatal on some devices. See: https://github.com/google/ExoPlayer/issues/8058
+      callback =
+          new AudioTrack.StreamEventCallback() {
+            @Override
+            public void onDataRequest(AudioTrack track, int size) {
+              Assertions.checkState(track == audioTrack);
+              if (listener != null && playing) {
+                // Do not signal that the buffer is emptying if not playing as it is a transient
+                // state.
+                listener.onOffloadBufferEmptying();
+              }
+            }
 
-    @Override
-    public void onDataRequest(AudioTrack track, int size) {
-      Assertions.checkState(track == audioTrack);
-      if (listener != null && playing) {
-        // Do not signal that the buffer is emptying if not playing as it is a transient state.
-        listener.onOffloadBufferEmptying();
-      }
-    }
-
-    @Override
-    public void onTearDown(@NonNull AudioTrack track) {
-      Assertions.checkState(track == audioTrack);
-      if (listener != null && playing) {
-        // The audio track was destroyed while in use. Thus a new AudioTrack needs to be created
-        // and its buffer filled, which will be done on the next handleBuffer call.
-        // Request this call explicitly in case ExoPlayer is sleeping waiting for a data request.
-        listener.onOffloadBufferEmptying();
-      }
+            @Override
+            public void onTearDown(@NonNull AudioTrack track) {
+              Assertions.checkState(track == audioTrack);
+              if (listener != null && playing) {
+                // The audio track was destroyed while in use. Thus a new AudioTrack needs to be
+                // created and its buffer filled, which will be done on the next handleBuffer call.
+                // Request this call explicitly in case ExoPlayer is sleeping waiting for a data
+                // request.
+                listener.onOffloadBufferEmptying();
+              }
+            }
+          };
     }
 
     public void register(AudioTrack audioTrack) {
-      audioTrack.registerStreamEventCallback(handler::post, this);
+      audioTrack.registerStreamEventCallback(handler::post, callback);
     }
 
     public void unregister(AudioTrack audioTrack) {
-      audioTrack.unregisterStreamEventCallback(this);
+      audioTrack.unregisterStreamEventCallback(callback);
       handler.removeCallbacksAndMessages(/* token= */ null);
     }
   }