Remove TODOs for merging RGB and MatrixTransformation processors.

PiperOrigin-RevId: 476362571
DiceTechnology · Oct 19, 2022 · af19f82 · af19f82
1 parent 94a88e9
commit af19f82
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 2 deletions.
diff --git a/.../effect/src/androidTest/java/androidx/media3/effect/GlEffectsFrameProcessorPixelTest.java b/.../effect/src/androidTest/java/androidx/media3/effect/GlEffectsFrameProcessorPixelTest.java
@@ -388,7 +388,6 @@ public void processData_fullRotationIncreaseBrightnessAndCenterCrop_producesExpe
   }
 
   @Test
-  // TODO(b/239757183): Consider moving RgbMatrix composition tests to a new file.
   public void drawFrame_grayscaleAndIncreaseRedChannel_producesGrayscaleAndRedImage()
       throws Exception {
     String testId = "drawFrame_grayscaleAndIncreaseRedChannel";

diff --git a/.../media/libraries/effect/src/main/java/androidx/media3/effect/GlEffectsFrameProcessor.java b/.../media/libraries/effect/src/main/java/androidx/media3/effect/GlEffectsFrameProcessor.java
@@ -167,7 +167,6 @@ private static GlEffectsFrameProcessor createOpenGlObjectsAndFrameProcessor(
    *     The first is an {@link ExternalTextureProcessor} and the last is a {@link
    *     FinalMatrixTextureProcessorWrapper}.
    */
-  // TODO(b/239757183): Squash GlMatrixTransformation and RgbMatrix together.
   private static ImmutableList<GlTextureProcessor> getGlTextureProcessorsForGlEffects(
       Context context,
       List<Effect> effects,