From 06ada01730b6544ad672d4a7afbda51496ba01f2 Mon Sep 17 00:00:00 2001
From: ibaker <ibaker@google.com>
Date: Tue, 22 Feb 2022 14:50:15 +0000
Subject: [PATCH] Suppress @VisibleForTesting violation in Transformer

#minor-release

PiperOrigin-RevId: 430200331
---
 .../exoplayer2/transformer/Transformer.java        | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

diff --git a/library/transformer/src/main/java/com/google/android/exoplayer2/transformer/Transformer.java b/library/transformer/src/main/java/com/google/android/exoplayer2/transformer/Transformer.java
index 34343b6720..5d3eeca028 100644
--- a/library/transformer/src/main/java/com/google/android/exoplayer2/transformer/Transformer.java
+++ b/library/transformer/src/main/java/com/google/android/exoplayer2/transformer/Transformer.java
@@ -677,7 +677,7 @@ public final class Transformer {
                 DEFAULT_BUFFER_FOR_PLAYBACK_MS / 10,
                 DEFAULT_BUFFER_FOR_PLAYBACK_AFTER_REBUFFER_MS / 10)
             .build();
-    player =
+    ExoPlayer.Builder playerBuilder =
         new ExoPlayer.Builder(
                 context,
                 new TransformerRenderersFactory(
@@ -693,9 +693,15 @@ public final class Transformer {
             .setMediaSourceFactory(mediaSourceFactory)
             .setTrackSelector(trackSelector)
             .setLoadControl(loadControl)
-            .setLooper(looper)
-            .setClock(clock)
-            .build();
+            .setLooper(looper);
+    if (clock != Clock.DEFAULT) {
+      // Transformer.Builder#setClock is also @VisibleForTesting, so if we're using a non-default
+      // clock we must be in a test context.
+      @SuppressWarnings("VisibleForTests")
+      ExoPlayer.Builder unusedForAnnotation = playerBuilder.setClock(clock);
+    }
+
+    player = playerBuilder.build();
     player.setMediaItem(mediaItem);
     player.addListener(new TransformerPlayerListener(mediaItem, muxerWrapper));
     player.prepare();