From 48832cbbc43149bc47bb431f6b10894a30204bb4 Mon Sep 17 00:00:00 2001
From: kimvde <kimvde@google.com>
Date: Tue, 29 Apr 2025 01:55:25 -0700
Subject: [PATCH] Remove unused setter in PlaybackVideoGraphWrapper.Builder

PiperOrigin-RevId: 752639277
---
 .../video/PlaybackVideoGraphWrapper.java      | 31 +++----------------
 1 file changed, 4 insertions(+), 27 deletions(-)

diff --git a/libraries/exoplayer/src/main/java/androidx/media3/exoplayer/video/PlaybackVideoGraphWrapper.java b/libraries/exoplayer/src/main/java/androidx/media3/exoplayer/video/PlaybackVideoGraphWrapper.java
index 02498ff163..4efa8cb38e 100644
--- a/libraries/exoplayer/src/main/java/androidx/media3/exoplayer/video/PlaybackVideoGraphWrapper.java
+++ b/libraries/exoplayer/src/main/java/androidx/media3/exoplayer/video/PlaybackVideoGraphWrapper.java
@@ -110,8 +110,6 @@ public final class PlaybackVideoGraphWrapper implements VideoGraph.Listener {
   public static final class Builder {
     private final Context context;
     private final VideoFrameReleaseControl videoFrameReleaseControl;
-
-    private VideoFrameProcessor.@MonotonicNonNull Factory videoFrameProcessorFactory;
     private VideoGraph.@MonotonicNonNull Factory videoGraphFactory;
     private List<Effect> compositionEffects;
     private VideoCompositorSettings compositorSettings;
@@ -130,23 +128,6 @@ public final class PlaybackVideoGraphWrapper implements VideoGraph.Listener {
       clock = Clock.DEFAULT;
     }
 
-    /**
-     * Sets the {@link VideoFrameProcessor.Factory} that will be used for creating {@link
-     * VideoFrameProcessor} instances.
-     *
-     * <p>By default, the {@code DefaultVideoFrameProcessor.Factory} with its default values will be
-     * used.
-     *
-     * @param videoFrameProcessorFactory The {@link VideoFrameProcessor.Factory}.
-     * @return This builder, for convenience.
-     */
-    @CanIgnoreReturnValue
-    public Builder setVideoFrameProcessorFactory(
-        VideoFrameProcessor.Factory videoFrameProcessorFactory) {
-      this.videoFrameProcessorFactory = videoFrameProcessorFactory;
-      return this;
-    }
-
     /**
      * Sets the {@link VideoGraph.Factory} that will be used for creating {@link VideoGraph}
      * instances.
@@ -269,11 +250,7 @@ public final class PlaybackVideoGraphWrapper implements VideoGraph.Listener {
       checkState(!built);
 
       if (videoGraphFactory == null) {
-        if (videoFrameProcessorFactory == null) {
-          videoFrameProcessorFactory =
-              new ReflectiveDefaultVideoFrameProcessorFactory(enableReplayableCache);
-        }
-        videoGraphFactory = new ReflectiveSingleInputVideoGraphFactory(videoFrameProcessorFactory);
+        videoGraphFactory = new ReflectiveSingleInputVideoGraphFactory(enableReplayableCache);
       }
       PlaybackVideoGraphWrapper playbackVideoGraphWrapper = new PlaybackVideoGraphWrapper(this);
       built = true;
@@ -1182,9 +1159,9 @@ public final class PlaybackVideoGraphWrapper implements VideoGraph.Listener {
 
     private final VideoFrameProcessor.Factory videoFrameProcessorFactory;
 
-    public ReflectiveSingleInputVideoGraphFactory(
-        VideoFrameProcessor.Factory videoFrameProcessorFactory) {
-      this.videoFrameProcessorFactory = videoFrameProcessorFactory;
+    public ReflectiveSingleInputVideoGraphFactory(boolean enableReplayableCache) {
+      this.videoFrameProcessorFactory =
+          new ReflectiveDefaultVideoFrameProcessorFactory(enableReplayableCache);
     }
 
     @Override