From 2e7c84ba00083a5e3f45299e76fa1e0d487e4e14 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Mateo=20Guzm=C3=A1n?= Date: Fri, 7 Feb 2025 04:01:37 -0800 Subject: [PATCH] Make `MultiPostprocessor` internal (#49237) Summary: As part of the initiative to reduce the public API surface, this class can be internalized. I've checked there are [no relevant OSS usages](https://github.com/search?type=code&q=NOT+is%3Afork+NOT+org%3Afacebook+NOT+repo%3Areact-native-tvos%2Freact-native-tvos+NOT+repo%3Anuagoz%2Freact-native+NOT+repo%3A2lambda123%2Freact-native+NOT+repo%3Abeanchips%2Ffacebookreactnative+NOT+repo%3AfabOnReact%2Freact-native-notes+NOT+user%3Ahuntie+com.facebook.react.views.image.MultiPostprocessor). ## Changelog: [INTERNAL] - Make com.facebook.react.views.image.MultiPostprocessor internal Pull Request resolved: https://github.com/facebook/react-native/pull/49237 Test Plan: ```bash yarn test-android yarn android ``` Reviewed By: cipolleschi Differential Revision: D69293117 Pulled By: cortinico fbshipit-source-id: 7e2713520a5a4fe492fb9d25cdd92c46746ded40 --- .../react-native/ReactAndroid/api/ReactAndroid.api | 13 ------------- .../react/views/image/MultiPostprocessor.kt | 8 ++++---- 2 files changed, 4 insertions(+), 17 deletions(-) diff --git a/packages/react-native/ReactAndroid/api/ReactAndroid.api b/packages/react-native/ReactAndroid/api/ReactAndroid.api index cc0ac29c152..e3fe4c6cf1f 100644 --- a/packages/react-native/ReactAndroid/api/ReactAndroid.api +++ b/packages/react-native/ReactAndroid/api/ReactAndroid.api @@ -6239,19 +6239,6 @@ public final class com/facebook/react/views/image/ImageResizeMode { public static final fun toTileMode (Ljava/lang/String;)Landroid/graphics/Shader$TileMode; } -public final class com/facebook/react/views/image/MultiPostprocessor : com/facebook/imagepipeline/request/Postprocessor { - public static final field Companion Lcom/facebook/react/views/image/MultiPostprocessor$Companion; - public synthetic fun (Ljava/util/List;Lkotlin/jvm/internal/DefaultConstructorMarker;)V - public static final fun from (Ljava/util/List;)Lcom/facebook/imagepipeline/request/Postprocessor; - public fun getName ()Ljava/lang/String; - public fun getPostprocessorCacheKey ()Lcom/facebook/cache/common/CacheKey; - public fun process (Landroid/graphics/Bitmap;Lcom/facebook/imagepipeline/bitmaps/PlatformBitmapFactory;)Lcom/facebook/common/references/CloseableReference; -} - -public final class com/facebook/react/views/image/MultiPostprocessor$Companion { - public final fun from (Ljava/util/List;)Lcom/facebook/imagepipeline/request/Postprocessor; -} - public abstract interface class com/facebook/react/views/image/ReactCallerContextFactory { public abstract fun getOrCreateCallerContext (Ljava/lang/String;Ljava/lang/String;)Ljava/lang/Object; } diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/image/MultiPostprocessor.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/image/MultiPostprocessor.kt index d0b7d48d5dc..4e80b0a4c77 100644 --- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/image/MultiPostprocessor.kt +++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/image/MultiPostprocessor.kt @@ -15,7 +15,7 @@ import com.facebook.imagepipeline.bitmaps.PlatformBitmapFactory import com.facebook.imagepipeline.request.Postprocessor import java.util.LinkedList -public class MultiPostprocessor private constructor(postprocessors: List) : +internal class MultiPostprocessor private constructor(postprocessors: List) : Postprocessor { private val postprocessors: List = LinkedList(postprocessors) @@ -25,7 +25,7 @@ public class MultiPostprocessor private constructor(postprocessors: List { @@ -48,9 +48,9 @@ public class MultiPostprocessor private constructor(postprocessors: List): Postprocessor? { + fun from(postprocessors: List): Postprocessor? { return when (postprocessors.size) { 0 -> null 1 -> postprocessors[0]