Do not reuse frame extractor if HDR output is enabled

The media3 effects pipeline doesn't allow changing the output
color space. If we create an HDR pipeline, we cannot output SDR afterwards.

PiperOrigin-RevId: 827510777

Author: ychaparov

libraries/inspector/src/androidTest/java/androidx/media3/inspector/mh/FrameExtractorHdrTest.java

@@ -36,6 +36,7 @@
 import android.graphics.BitmapFactory;
 import android.graphics.ColorSpace;
 import androidx.media3.common.MediaItem;
+import androidx.media3.exoplayer.mediacodec.MediaCodecSelector;
 import androidx.media3.inspector.FrameExtractor;
 import androidx.test.core.app.ApplicationProvider;
 import androidx.test.ext.junit.runners.AndroidJUnit4;
@@ -218,4 +219,37 @@ public void extractFrame_oneFrameHlgWithHdrDisplayUnsupported_returnsSdrFrame()
     assertThat(frameSecondItem.presentationTimeMs).isEqualTo(0);
     assertBitmapsAreSimilar(expectedBitmapSecondItem, actualBitmapSecondItem, PSNR_THRESHOLD);
   }
+
+  @Test
+  @SdkSuppress(minSdkVersion = 34) // HLG Bitmaps are only supported on API 34+.
+  public void extractFrame_changeMediaItemFromHdrToSdrWithHdrOutput_succeeds() throws Exception {
+    // TODO: b/438478509 - rename assumeDeviceSupportsOpenGlToneMapping. This check verifies
+    // that HDR input can be sampled by the GL pipeline.
+    assumeDeviceSupportsOpenGlToneMapping(testId, MP4_ASSET_COLOR_TEST_1080P_HLG10.videoFormat);
+
+    FrameExtractor.Frame frameFirstItem;
+    try (FrameExtractor frameExtractor =
+        new FrameExtractor.Builder(context, MediaItem.fromUri(MP4_ASSET_COLOR_TEST_1080P_HLG10.uri))
+            .setMediaCodecSelector(MediaCodecSelector.DEFAULT)
+            .setExtractHdrFrames(true)
+            .build()) {
+      ListenableFuture<FrameExtractor.Frame> frameFutureFirstItem =
+          frameExtractor.getFrame(/* positionMs= */ 0);
+      frameFirstItem = frameFutureFirstItem.get(TIMEOUT_SECONDS, SECONDS);
+    }
+
+    FrameExtractor.Frame frameSecondItem;
+    try (FrameExtractor frameExtractor =
+        new FrameExtractor.Builder(context, MediaItem.fromUri(MP4_TRIM_OPTIMIZATION_270.uri))
+            .setMediaCodecSelector(MediaCodecSelector.DEFAULT)
+            .setExtractHdrFrames(true)
+            .build()) {
+      ListenableFuture<FrameExtractor.Frame> frameFutureSecondItem =
+          frameExtractor.getFrame(/* positionMs= */ 0);
+      frameSecondItem = frameFutureSecondItem.get(TIMEOUT_SECONDS, SECONDS);
+    }
+
+    assertThat(frameFirstItem.presentationTimeMs).isEqualTo(0);
+    assertThat(frameSecondItem.presentationTimeMs).isEqualTo(0);
+  }
 }

libraries/inspector/src/main/java/androidx/media3/inspector/FrameExtractorInternal.java

@@ -233,8 +233,11 @@ public ListenableFuture<FrameExtractor.Frame> submitTask(FrameExtractionRequest
         () -> {
           boolean needsNewPlayer =
               player == null
+                  // TODO: b/457376636 - reuse player when switching between HDR and SDR, after the
+                  // video processing pipeline is updated.
+                  || currentExtractHdrFrames
+                  || request.extractHdrFrames
                   || request.mediaCodecSelector != currentMediaCodecSelector
-                  || request.extractHdrFrames != currentExtractHdrFrames
                   || request.glObjectsProvider != currentGlObjectsProvider;
 
           boolean needsPrepare =