From f454c9aa7490e40d0ea5fdda6aa5698e7e6f6065 Mon Sep 17 00:00:00 2001
From: leonwind <leonwind@google.com>
Date: Thu, 25 Aug 2022 15:49:13 +0000
Subject: [PATCH] Delete glPrograms in Contrast and RgbMatrix. Remove
 unecessary null checks.

PiperOrigin-RevId: 469999952
---
 .../transformerdemo/BitmapOverlayProcessor.java      | 12 +++++-------
 .../transformerdemo/PeriodicVignetteProcessor.java   | 10 ++++------
 .../androidx/media3/effect/ContrastProcessor.java    | 10 ++++++++++
 .../media3/effect/MatrixTransformationProcessor.java | 10 ++++------
 .../androidx/media3/effect/RgbMatrixProcessor.java   | 10 ++++++++++
 5 files changed, 33 insertions(+), 19 deletions(-)

diff --git a/demos/transformer/src/main/java/com/google/android/exoplayer2/transformerdemo/BitmapOverlayProcessor.java b/demos/transformer/src/main/java/com/google/android/exoplayer2/transformerdemo/BitmapOverlayProcessor.java
index 7220528c90..6b85bb3ade 100644
--- a/demos/transformer/src/main/java/com/google/android/exoplayer2/transformerdemo/BitmapOverlayProcessor.java
+++ b/demos/transformer/src/main/java/com/google/android/exoplayer2/transformerdemo/BitmapOverlayProcessor.java
@@ -129,7 +129,7 @@ import java.util.Locale;
   @Override
   public void drawFrame(int inputTexId, long presentationTimeUs) throws FrameProcessingException {
     try {
-      checkStateNotNull(glProgram).use();
+      glProgram.use();
 
       // Draw to the canvas and store it in a texture.
       String text =
@@ -159,12 +159,10 @@ import java.util.Locale;
   @Override
   public void release() throws FrameProcessingException {
     super.release();
-    if (glProgram != null) {
-      try {
-        glProgram.delete();
-      } catch (GlUtil.GlException e) {
-        throw new FrameProcessingException(e);
-      }
+    try {
+      glProgram.delete();
+    } catch (GlUtil.GlException e) {
+      throw new FrameProcessingException(e);
     }
   }
 
diff --git a/demos/transformer/src/main/java/com/google/android/exoplayer2/transformerdemo/PeriodicVignetteProcessor.java b/demos/transformer/src/main/java/com/google/android/exoplayer2/transformerdemo/PeriodicVignetteProcessor.java
index c44c2cdc38..88ec4a596e 100644
--- a/demos/transformer/src/main/java/com/google/android/exoplayer2/transformerdemo/PeriodicVignetteProcessor.java
+++ b/demos/transformer/src/main/java/com/google/android/exoplayer2/transformerdemo/PeriodicVignetteProcessor.java
@@ -114,12 +114,10 @@ import java.io.IOException;
   @Override
   public void release() throws FrameProcessingException {
     super.release();
-    if (glProgram != null) {
-      try {
-        glProgram.delete();
-      } catch (GlUtil.GlException e) {
-        throw new FrameProcessingException(e);
-      }
+    try {
+      glProgram.delete();
+    } catch (GlUtil.GlException e) {
+      throw new FrameProcessingException(e);
     }
   }
 }
diff --git a/google3/third_party/java_src/android_libs/media/libraries/effect/src/main/java/androidx/media3/effect/ContrastProcessor.java b/google3/third_party/java_src/android_libs/media/libraries/effect/src/main/java/androidx/media3/effect/ContrastProcessor.java
index e267862ffe..206c3c887c 100644
--- a/google3/third_party/java_src/android_libs/media/libraries/effect/src/main/java/androidx/media3/effect/ContrastProcessor.java
+++ b/google3/third_party/java_src/android_libs/media/libraries/effect/src/main/java/androidx/media3/effect/ContrastProcessor.java
@@ -75,4 +75,14 @@ import java.io.IOException;
       throw new FrameProcessingException(e, presentationTimeUs);
     }
   }
+
+  @Override
+  public void release() throws FrameProcessingException {
+    super.release();
+    try {
+      glProgram.delete();
+    } catch (GlUtil.GlException e) {
+      throw new FrameProcessingException(e);
+    }
+  }
 }
diff --git a/google3/third_party/java_src/android_libs/media/libraries/effect/src/main/java/androidx/media3/effect/MatrixTransformationProcessor.java b/google3/third_party/java_src/android_libs/media/libraries/effect/src/main/java/androidx/media3/effect/MatrixTransformationProcessor.java
index c01b919263..67badc6312 100644
--- a/google3/third_party/java_src/android_libs/media/libraries/effect/src/main/java/androidx/media3/effect/MatrixTransformationProcessor.java
+++ b/google3/third_party/java_src/android_libs/media/libraries/effect/src/main/java/androidx/media3/effect/MatrixTransformationProcessor.java
@@ -321,12 +321,10 @@ import java.util.Arrays;
   @Override
   public void release() throws FrameProcessingException {
     super.release();
-    if (glProgram != null) {
-      try {
-        glProgram.delete();
-      } catch (GlUtil.GlException e) {
-        throw new FrameProcessingException(e);
-      }
+    try {
+      glProgram.delete();
+    } catch (GlUtil.GlException e) {
+      throw new FrameProcessingException(e);
     }
   }
 
diff --git a/google3/third_party/java_src/android_libs/media/libraries/effect/src/main/java/androidx/media3/effect/RgbMatrixProcessor.java b/google3/third_party/java_src/android_libs/media/libraries/effect/src/main/java/androidx/media3/effect/RgbMatrixProcessor.java
index e326153908..1518b28816 100644
--- a/google3/third_party/java_src/android_libs/media/libraries/effect/src/main/java/androidx/media3/effect/RgbMatrixProcessor.java
+++ b/google3/third_party/java_src/android_libs/media/libraries/effect/src/main/java/androidx/media3/effect/RgbMatrixProcessor.java
@@ -135,4 +135,14 @@ import java.io.IOException;
       throw new FrameProcessingException(e, presentationTimeUs);
     }
   }
+
+  @Override
+  public void release() throws FrameProcessingException {
+    super.release();
+    try {
+      glProgram.delete();
+    } catch (GlUtil.GlException e) {
+      throw new FrameProcessingException(e);
+    }
+  }
 }