Commit 6dcb018d by tofunmi Committed by Rohit Singh

Remove deprecated constructor in DefaultVideoFrameProcessor.Factory

Remove deprecated factory constructor and replace all call instances of it.

PiperOrigin-RevId: 525476062
parent 3bcdac99
...@@ -123,7 +123,7 @@ public class DefaultVideoFrameProcessorImageFrameOutputTest { ...@@ -123,7 +123,7 @@ public class DefaultVideoFrameProcessorImageFrameOutputTest {
String testId) { String testId) {
return new VideoFrameProcessorTestRunner.Builder() return new VideoFrameProcessorTestRunner.Builder()
.setTestId(testId) .setTestId(testId)
.setVideoFrameProcessorFactory(new DefaultVideoFrameProcessor.Factory()) .setVideoFrameProcessorFactory(new DefaultVideoFrameProcessor.Factory.Builder().build())
.setIsInputTextureExternal(false) .setIsInputTextureExternal(false)
.setOnOutputFrameAvailableListener( .setOnOutputFrameAvailableListener(
unused -> checkNotNull(framesProduced).incrementAndGet()); unused -> checkNotNull(framesProduced).incrementAndGet());
......
...@@ -506,7 +506,7 @@ public final class DefaultVideoFrameProcessorPixelTest { ...@@ -506,7 +506,7 @@ public final class DefaultVideoFrameProcessorPixelTest {
String testId) { String testId) {
return new VideoFrameProcessorTestRunner.Builder() return new VideoFrameProcessorTestRunner.Builder()
.setTestId(testId) .setTestId(testId)
.setVideoFrameProcessorFactory(new DefaultVideoFrameProcessor.Factory()) .setVideoFrameProcessorFactory(new DefaultVideoFrameProcessor.Factory.Builder().build())
.setVideoAssetPath(INPUT_SDR_MP4_ASSET_STRING); .setVideoAssetPath(INPUT_SDR_MP4_ASSET_STRING);
} }
......
...@@ -292,7 +292,8 @@ public final class DefaultVideoFrameProcessorVideoFrameReleaseTest { ...@@ -292,7 +292,8 @@ public final class DefaultVideoFrameProcessorVideoFrameReleaseTest {
CountDownLatch videoFrameProcessingEndedCountDownLatch = new CountDownLatch(1); CountDownLatch videoFrameProcessingEndedCountDownLatch = new CountDownLatch(1);
defaultVideoFrameProcessor = defaultVideoFrameProcessor =
checkNotNull( checkNotNull(
new DefaultVideoFrameProcessor.Factory() new DefaultVideoFrameProcessor.Factory.Builder()
.build()
.create( .create(
getApplicationContext(), getApplicationContext(),
ImmutableList.of((GlEffect) (context, useHdr) -> blankFrameProducer), ImmutableList.of((GlEffect) (context, useHdr) -> blankFrameProducer),
......
...@@ -141,18 +141,6 @@ public final class DefaultVideoFrameProcessor implements VideoFrameProcessor { ...@@ -141,18 +141,6 @@ public final class DefaultVideoFrameProcessor implements VideoFrameProcessor {
this.textureOutputListener = textureOutputListener; this.textureOutputListener = textureOutputListener;
} }
// TODO(276913828): Remove and change all calls to a builder.
/**
* @deprecated Use {@link DefaultVideoFrameProcessor.Factory.Builder} instead.
*/
@Deprecated
public Factory() {
this(
/* enableColorTransfers= */ true,
GlObjectsProvider.DEFAULT,
/* textureOutputListener= */ null);
}
/** /**
* {@inheritDoc} * {@inheritDoc}
* *
......
...@@ -213,6 +213,6 @@ public final class ToneMapHdrToSdrUsingOpenGlPixelTest { ...@@ -213,6 +213,6 @@ public final class ToneMapHdrToSdrUsingOpenGlPixelTest {
String testId) { String testId) {
return new VideoFrameProcessorTestRunner.Builder() return new VideoFrameProcessorTestRunner.Builder()
.setTestId(testId) .setTestId(testId)
.setVideoFrameProcessorFactory(new DefaultVideoFrameProcessor.Factory()); .setVideoFrameProcessorFactory(new DefaultVideoFrameProcessor.Factory.Builder().build());
} }
} }
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or sign in to comment