Kotlin: clean up redundant visibility modifiers (1/2) (facebook#51960)

mateoguzmana · facebook-github-bot · commit f880bfd1abd3 · 2025-06-13T08:42:42.000-07:00
Summary: Static code analysis shows that there are several redundant visibility modifiers across the codebase. These are most likely remnants after making different classes internal. ## Changelog: [INTERNAL] - Kotlin: clean up redundant visibility modifiers Pull Request resolved: facebook#51960 Test Plan: ```sh yarn android yarn test-android ``` Reviewed By: javache Differential Revision: D76503015 Pulled By: cortinico fbshipit-source-id: e60e7aa141fc35ca2fd76335fbee791c86589e4e
diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/animated/NativeAnimatedModule.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/animated/NativeAnimatedModule.kt
@@ -111,7 +111,7 @@ public class NativeAnimatedModule(reactContext: ReactApplicationContext?) :
       private var valueMap: Array<BatchExecutionOpCodes>? = null
 
       @JvmStatic
-      public fun fromId(id: Int): BatchExecutionOpCodes {
+      fun fromId(id: Int): BatchExecutionOpCodes {
         val valueMapNonnull: Array<BatchExecutionOpCodes> =
             valueMap ?: BatchExecutionOpCodes.values()
         if (valueMap == null) {
diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/bridge/JavaScriptModuleRegistry.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/bridge/JavaScriptModuleRegistry.kt
@@ -67,7 +67,7 @@ public class JavaScriptModuleRegistry {
       return name ?: getJSModuleName(moduleInterface).also { name = it }
     }
 
-    public override fun invoke(proxy: Any, method: Method, args: Array<Any?>?): Any? {
+    override fun invoke(proxy: Any, method: Method, args: Array<Any?>?): Any? {
       val jsArgs = if (args != null) Arguments.fromJavaArgs(args) else WritableNativeArray()
       catalystInstance.callFunction(getJSModuleName(), method.name, jsArgs)
       return null
diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/modules/debug/DevMenuModule.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/modules/debug/DevMenuModule.kt
@@ -40,7 +40,7 @@ internal class DevMenuModule(
     devSupportManager.setHotModuleReplacementEnabled(enabled)
   }
 
-  public companion object {
-    public const val NAME: String = NativeDevMenuSpec.NAME
+  companion object {
+    const val NAME: String = NativeDevMenuSpec.NAME
   }
 }
diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/modules/deviceinfo/DeviceInfoModule.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/modules/deviceinfo/DeviceInfoModule.kt
@@ -74,7 +74,7 @@ internal class DeviceInfoModule(reactContext: ReactApplicationContext) :
     reactApplicationContext.removeLifecycleEventListener(this)
   }
 
-  public companion object {
-    public const val NAME: String = NativeDeviceInfoSpec.NAME
+  companion object {
+    const val NAME: String = NativeDeviceInfoSpec.NAME
   }
 }
diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/runtime/BridgelessCatalystInstance.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/runtime/BridgelessCatalystInstance.kt
@@ -84,7 +84,7 @@ internal class BridgelessCatalystInstance(private val reactHost: ReactHostImpl)
     throw UnsupportedOperationException("Unimplemented method 'destroy'")
   }
 
-  public override val isDestroyed: Boolean
+  override val isDestroyed: Boolean
     get() = throw UnsupportedOperationException("Unimplemented method 'isDestroyed'")
 
   @VisibleForTesting
@@ -96,16 +96,16 @@ internal class BridgelessCatalystInstance(private val reactHost: ReactHostImpl)
       reactHost.currentReactContext?.getJSModule(jsInterface)
 
   @get:Deprecated("Deprecated in Java")
-  public override val javaScriptContextHolder: JavaScriptContextHolder
+  override val javaScriptContextHolder: JavaScriptContextHolder
     get() = reactHost.javaScriptContextHolder!!
 
   @Suppress("INAPPLICABLE_JVM_NAME")
   @get:Deprecated("Deprecated in Java")
   @get:JvmName("getJSCallInvokerHolder") // This is needed to keep backward compatibility
-  public override val jsCallInvokerHolder: CallInvokerHolder
+  override val jsCallInvokerHolder: CallInvokerHolder
     get() = reactHost.jsCallInvokerHolder!!
 
-  public override val nativeMethodCallInvokerHolder: NativeMethodCallInvokerHolder
+  override val nativeMethodCallInvokerHolder: NativeMethodCallInvokerHolder
     get() =
         throw UnsupportedOperationException(
             "Unimplemented method 'getNativeMethodCallInvokerHolder'")
@@ -119,23 +119,23 @@ internal class BridgelessCatalystInstance(private val reactHost: ReactHostImpl)
   override fun getNativeModule(moduleName: String): NativeModule? =
       reactHost.getNativeModule(moduleName)
 
-  public override val nativeModules: Collection<NativeModule>
+  override val nativeModules: Collection<NativeModule>
     get() = reactHost.nativeModules
 
-  public override val reactQueueConfiguration: ReactQueueConfiguration
+  override val reactQueueConfiguration: ReactQueueConfiguration
     get() = reactHost.reactQueueConfiguration!!
 
-  public override val runtimeExecutor: RuntimeExecutor?
+  override val runtimeExecutor: RuntimeExecutor?
     get() = reactHost.runtimeExecutor
 
-  public override val runtimeScheduler: RuntimeScheduler
+  override val runtimeScheduler: RuntimeScheduler
     get() = throw UnsupportedOperationException("Unimplemented method 'getRuntimeScheduler'")
 
-  public override fun extendNativeModules(modules: NativeModuleRegistry) {
+  override fun extendNativeModules(modules: NativeModuleRegistry) {
     throw UnsupportedOperationException("Unimplemented method 'extendNativeModules'")
   }
 
-  public override val sourceURL: String
+  override val sourceURL: String
     get() = throw UnsupportedOperationException("Unimplemented method 'getSourceURL'")
 
   override fun addBridgeIdleDebugListener(listener: NotThreadSafeBridgeIdleDebugListener) {
diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/runtime/BridgelessReactContext.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/runtime/BridgelessReactContext.kt
@@ -57,7 +57,7 @@ internal class BridgelessReactContext(context: Context, private val reactHost: R
 
   override fun getSourceURL(): String? = sourceURLRef.get()
 
-  public fun setSourceURL(sourceURL: String?) {
+  fun setSourceURL(sourceURL: String?) {
     sourceURLRef.set(sourceURL)
   }
 
diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/uimanager/events/FabricEventDispatcher.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/uimanager/events/FabricEventDispatcher.kt
@@ -59,7 +59,7 @@ internal class FabricEventDispatcher(
     eventEmitter.registerFabricEventEmitter(fabricEventEmitter)
   }
 
-  public override fun dispatchEvent(event: Event<*>) {
+  override fun dispatchEvent(event: Event<*>) {
     for (listener in listeners) {
       listener.onEventDispatch(event)
     }
@@ -100,7 +100,7 @@ internal class FabricEventDispatcher(
     }
   }
 
-  public override fun dispatchAllEvents() {
+  override fun dispatchAllEvents() {
     scheduleDispatchOfBatchedEvents()
   }
 
@@ -116,46 +116,46 @@ internal class FabricEventDispatcher(
   }
 
   /** Add a listener to this EventDispatcher. */
-  public override fun addListener(listener: EventDispatcherListener) {
+  override fun addListener(listener: EventDispatcherListener) {
     listeners.add(listener)
   }
 
   /** Remove a listener from this EventDispatcher. */
-  public override fun removeListener(listener: EventDispatcherListener) {
+  override fun removeListener(listener: EventDispatcherListener) {
     listeners.remove(listener)
   }
 
-  public override fun addBatchEventDispatchedListener(listener: BatchEventDispatchedListener) {
+  override fun addBatchEventDispatchedListener(listener: BatchEventDispatchedListener) {
     postEventDispatchListeners.add(listener)
   }
 
-  public override fun removeBatchEventDispatchedListener(listener: BatchEventDispatchedListener) {
+  override fun removeBatchEventDispatchedListener(listener: BatchEventDispatchedListener) {
     postEventDispatchListeners.remove(listener)
   }
 
-  public override fun onHostResume() {
+  override fun onHostResume() {
     scheduleDispatchOfBatchedEvents()
     if (!ReactNativeFeatureFlags.useOptimizedEventBatchingOnAndroid()) {
       currentFrameCallback.resume()
     }
   }
 
-  public override fun onHostPause() {
+  override fun onHostPause() {
     cancelDispatchOfBatchedEvents()
   }
 
-  public override fun onHostDestroy() {
+  override fun onHostDestroy() {
     cancelDispatchOfBatchedEvents()
   }
 
-  public fun invalidate() {
+  fun invalidate() {
     eventEmitter.registerFabricEventEmitter(null)
 
     UiThreadUtil.runOnUiThread { cancelDispatchOfBatchedEvents() }
   }
 
   @Deprecated("Private API, should only be used when the concrete implementation is known.")
-  public override fun onCatalystInstanceDestroyed() {
+  override fun onCatalystInstanceDestroyed() {
     invalidate()
   }
 
diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/uimanager/layoutanimation/LayoutUpdateAnimation.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/uimanager/layoutanimation/LayoutUpdateAnimation.kt
@@ -21,9 +21,9 @@ import com.facebook.react.common.annotations.internal.LegacyArchitectureLogger
 @LegacyArchitecture(logLevel = LegacyArchitectureLogLevel.ERROR)
 internal class LayoutUpdateAnimation : AbstractLayoutAnimation() {
 
-  internal override fun isValid(): Boolean = durationMs > 0
+  override fun isValid(): Boolean = durationMs > 0
 
-  internal override fun createAnimationImpl(
+  override fun createAnimationImpl(
       view: View,
       x: Int,
       y: Int,
diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/uimanager/style/ColorStop.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/uimanager/style/ColorStop.kt
@@ -26,7 +26,7 @@ internal class ColorStop(var color: Int? = null, val position: LengthPercentage?
 internal class ProcessedColorStop(var color: Int? = null, val position: Float? = null)
 
 internal object ColorStopUtils {
-  public fun getFixedColorStops(
+  fun getFixedColorStops(
       colorStops: List<ColorStop>,
       gradientLineLength: Float
   ): List<ProcessedColorStop> {
diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/uimanager/style/Gradient.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/uimanager/style/Gradient.kt
@@ -10,5 +10,5 @@ package com.facebook.react.uimanager.style
 import android.graphics.Shader
 
 internal interface Gradient {
-  public fun getShader(width: Float, height: Float): Shader
+  fun getShader(width: Float, height: Float): Shader
 }
diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/scroll/MaintainVisibleScrollPositionHelper.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/scroll/MaintainVisibleScrollPositionHelper.kt
@@ -34,7 +34,7 @@ internal class MaintainVisibleScrollPositionHelper<ScrollViewT>(
     private val horizontal: Boolean
 ) : UIManagerListener where ScrollViewT : HasSmoothScroll?, ScrollViewT : ViewGroup? {
 
-  public var config: Config? = null
+  var config: Config? = null
   private var firstVisibleViewRef: WeakReference<View>? = null
   private var prevFirstVisibleFrame: Rect? = null
   private var isListening = false
diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/text/PreparedLayout.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/text/PreparedLayout.kt
@@ -16,7 +16,7 @@ import com.facebook.proguard.annotations.DoNotStrip
  */
 @DoNotStrip
 internal class PreparedLayout(
-    public val layout: Layout,
-    public val maximumNumberOfLines: Int,
-    public val verticalOffset: Float
+    val layout: Layout,
+    val maximumNumberOfLines: Int,
+    val verticalOffset: Float
 )
diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/text/PreparedLayoutTextView.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/text/PreparedLayoutTextView.kt
@@ -41,7 +41,7 @@ internal class PreparedLayoutTextView(context: Context) : ViewGroup(context), Re
   private var clickableSpans: List<ClickableSpan> = emptyList()
   private var selection: TextSelection? = null
 
-  public var preparedLayout: PreparedLayout? = null
+  var preparedLayout: PreparedLayout? = null
     set(value) {
       if (field != value) {
         val lastSelection = selection
@@ -63,17 +63,17 @@ internal class PreparedLayoutTextView(context: Context) : ViewGroup(context), Re
 
   // T221698007: This is closest to existing behavior, but does not align with web. We may want to
   // change in the future if not too breaking.
-  public var overflow: Overflow = Overflow.HIDDEN
+  var overflow: Overflow = Overflow.HIDDEN
     set(value) {
       if (field != value) {
         field = value
         invalidate()
       }
     }
 
-  public @ColorInt var selectionColor: Int? = null
+  @ColorInt var selectionColor: Int? = null
 
-  public val text: CharSequence?
+  val text: CharSequence?
     get() = preparedLayout?.layout?.text
 
   init {
@@ -88,7 +88,7 @@ internal class PreparedLayoutTextView(context: Context) : ViewGroup(context), Re
     preparedLayout = null
   }
 
-  public fun recycleView(): Unit {
+  fun recycleView(): Unit {
     initView()
     BackgroundStyleApplicator.reset(this)
     overflow = Overflow.HIDDEN
@@ -122,7 +122,7 @@ internal class PreparedLayoutTextView(context: Context) : ViewGroup(context), Re
     // No-op
   }
 
-  public fun setSelection(start: Int, end: Int) {
+  fun setSelection(start: Int, end: Int) {
     val layout = checkNotNull(preparedLayout).layout
     if (start < 0 || end > layout.text.length || start >= end) {
       throw IllegalArgumentException(
@@ -143,7 +143,7 @@ internal class PreparedLayoutTextView(context: Context) : ViewGroup(context), Re
     invalidate()
   }
 
-  public fun clearSelection() {
+  fun clearSelection() {
     selection = null
     invalidate()
   }
diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/text/PreparedLayoutTextViewManager.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/text/PreparedLayoutTextViewManager.kt
@@ -98,23 +98,23 @@ internal class PreparedLayoutTextViewManager :
   }
 
   @ReactProp(name = "overflow")
-  public fun setOverflow(view: PreparedLayoutTextView, overflow: String?): Unit {
+  fun setOverflow(view: PreparedLayoutTextView, overflow: String?): Unit {
     view.overflow = overflow?.let { Overflow.fromString(it) } ?: Overflow.HIDDEN
   }
 
   @ReactProp(name = "accessible")
-  public fun setAccessible(view: PreparedLayoutTextView, accessible: Boolean): Unit {
+  fun setAccessible(view: PreparedLayoutTextView, accessible: Boolean): Unit {
     view.isFocusable = accessible
   }
 
   @ReactProp(name = "selectable", defaultBoolean = false)
-  public fun setSelectable(view: PreparedLayoutTextView, isSelectable: Boolean): Unit {
+  fun setSelectable(view: PreparedLayoutTextView, isSelectable: Boolean): Unit {
     // T222052152: Implement fine-grained text selection for PreparedLayoutTextView
     // view.setTextIsSelectable(isSelectable);
   }
 
   @ReactProp(name = "selectionColor", customType = "Color")
-  public fun setSelectionColor(view: PreparedLayoutTextView, color: Int?): Unit {
+  fun setSelectionColor(view: PreparedLayoutTextView, color: Int?): Unit {
     if (color == null) {
       view.selectionColor = DefaultStyleValuesUtil.getDefaultTextColorHighlight(view.context)
     } else {
@@ -131,15 +131,15 @@ internal class PreparedLayoutTextViewManager :
               ViewProps.BORDER_BOTTOM_RIGHT_RADIUS,
               ViewProps.BORDER_BOTTOM_LEFT_RADIUS],
       defaultFloat = Float.NaN)
-  public fun setBorderRadius(view: PreparedLayoutTextView, index: Int, borderRadius: Float): Unit {
+  fun setBorderRadius(view: PreparedLayoutTextView, index: Int, borderRadius: Float): Unit {
     val radius =
         if (borderRadius.isNaN()) null
         else LengthPercentage(borderRadius, LengthPercentageType.POINT)
     BackgroundStyleApplicator.setBorderRadius(view, BorderRadiusProp.values()[index], radius)
   }
 
   @ReactProp(name = "borderStyle")
-  public fun setBorderStyle(view: PreparedLayoutTextView, borderStyle: String?): Unit {
+  fun setBorderStyle(view: PreparedLayoutTextView, borderStyle: String?): Unit {
     val parsedBorderStyle = if (borderStyle == null) null else BorderStyle.fromString(borderStyle)
     BackgroundStyleApplicator.setBorderStyle(view, parsedBorderStyle)
   }
@@ -155,7 +155,7 @@ internal class PreparedLayoutTextViewManager :
               ViewProps.BORDER_START_WIDTH,
               ViewProps.BORDER_END_WIDTH],
       defaultFloat = Float.NaN)
-  public fun setBorderWidth(view: PreparedLayoutTextView, index: Int, width: Float): Unit {
+  fun setBorderWidth(view: PreparedLayoutTextView, index: Int, width: Float): Unit {
     BackgroundStyleApplicator.setBorderWidth(view, LogicalEdge.values()[index], width)
   }
 
@@ -174,12 +174,12 @@ internal class PreparedLayoutTextViewManager :
               ViewProps.BORDER_BLOCK_START_COLOR,
           ],
       customType = "Color")
-  public fun setBorderColor(view: PreparedLayoutTextView, index: Int, color: Int?): Unit {
+  fun setBorderColor(view: PreparedLayoutTextView, index: Int, color: Int?): Unit {
     BackgroundStyleApplicator.setBorderColor(view, LogicalEdge.values()[index], color)
   }
 
   @ReactProp(name = "disabled", defaultBoolean = false)
-  public fun setDisabled(view: PreparedLayoutTextView, disabled: Boolean): Unit {
+  fun setDisabled(view: PreparedLayoutTextView, disabled: Boolean): Unit {
     view.setEnabled(!disabled)
   }
 
@@ -214,7 +214,7 @@ internal class PreparedLayoutTextViewManager :
     reactTextViewManagerCallback?.onPostProcessSpannable(text)
   }
 
-  public companion object {
-    public const val REACT_CLASS: String = "RCTText"
+  companion object {
+    const val REACT_CLASS: String = "RCTText"
   }
 }
diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/text/frescosupport/FrescoBasedReactTextInlineImageShadowNode.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/text/frescosupport/FrescoBasedReactTextInlineImageShadowNode.kt
diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/text/frescosupport/FrescoBasedReactTextInlineImageSpan.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/text/frescosupport/FrescoBasedReactTextInlineImageSpan.kt
diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/text/internal/span/CustomLetterSpacingSpan.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/text/internal/span/CustomLetterSpacingSpan.kt
diff --git a/packages/react-native/ReactAndroid/src/test/java/com/facebook/react/modules/timing/TimingModuleTest.kt b/packages/react-native/ReactAndroid/src/test/java/com/facebook/react/modules/timing/TimingModuleTest.kt

Original file line number	Diff line number	Diff line change
`@@ -67,7 +67,7 @@ public class JavaScriptModuleRegistry {`
`67`	`67`	`return name ?: getJSModuleName(moduleInterface).also { name = it }`
`68`	`68`	`}`
`69`	`69`
`70`		`- public override fun invoke(proxy: Any, method: Method, args: Array<Any?>?): Any? {`
	`70`	`+ override fun invoke(proxy: Any, method: Method, args: Array<Any?>?): Any? {`
`71`	`71`	`val jsArgs = if (args != null) Arguments.fromJavaArgs(args) else WritableNativeArray()`
`72`	`72`	`catalystInstance.callFunction(getJSModuleName(), method.name, jsArgs)`
`73`	`73`	`return null`
Original file line number	Diff line number	Diff line change
`@@ -40,7 +40,7 @@ internal class DevMenuModule(`
`40`	`40`	`devSupportManager.setHotModuleReplacementEnabled(enabled)`
`41`	`41`	`}`
`42`	`42`
`43`		`- public companion object {`
`44`		`- public const val NAME: String = NativeDevMenuSpec.NAME`
	`43`	`+ companion object {`
	`44`	`+ const val NAME: String = NativeDevMenuSpec.NAME`
`45`	`45`	`}`
`46`	`46`	`}`
Original file line number	Diff line number	Diff line change
`@@ -74,7 +74,7 @@ internal class DeviceInfoModule(reactContext: ReactApplicationContext) :`
`74`	`74`	`reactApplicationContext.removeLifecycleEventListener(this)`
`75`	`75`	`}`
`76`	`76`
`77`		`- public companion object {`
`78`		`- public const val NAME: String = NativeDeviceInfoSpec.NAME`
	`77`	`+ companion object {`
	`78`	`+ const val NAME: String = NativeDeviceInfoSpec.NAME`
`79`	`79`	`}`
`80`	`80`	`}`
Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,7 @@ internal class BridgelessReactContext(context: Context, private val reactHost: R`
`57`	`57`
`58`	`58`	`override fun getSourceURL(): String? = sourceURLRef.get()`
`59`	`59`
`60`		`- public fun setSourceURL(sourceURL: String?) {`
	`60`	`+ fun setSourceURL(sourceURL: String?) {`
`61`	`61`	`sourceURLRef.set(sourceURL)`
`62`	`62`	`}`
`63`	`63`
Original file line number	Diff line number	Diff line change
`@@ -59,7 +59,7 @@ internal class FabricEventDispatcher(`
`59`	`59`	`eventEmitter.registerFabricEventEmitter(fabricEventEmitter)`
`60`	`60`	`}`
`61`	`61`
`62`		`- public override fun dispatchEvent(event: Event<*>) {`
	`62`	`+ override fun dispatchEvent(event: Event<*>) {`
`63`	`63`	`for (listener in listeners) {`
`64`	`64`	`listener.onEventDispatch(event)`
`65`	`65`	`}`
`@@ -100,7 +100,7 @@ internal class FabricEventDispatcher(`
`100`	`100`	`}`
`101`	`101`	`}`
`102`	`102`
`103`		`- public override fun dispatchAllEvents() {`
	`103`	`+ override fun dispatchAllEvents() {`
`104`	`104`	`scheduleDispatchOfBatchedEvents()`
`105`	`105`	`}`
`106`	`106`
`@@ -116,46 +116,46 @@ internal class FabricEventDispatcher(`
`116`	`116`	`}`
`117`	`117`
`118`	`118`	`/** Add a listener to this EventDispatcher. */`
`119`		`- public override fun addListener(listener: EventDispatcherListener) {`
	`119`	`+ override fun addListener(listener: EventDispatcherListener) {`
`120`	`120`	`listeners.add(listener)`
`121`	`121`	`}`
`122`	`122`
`123`	`123`	`/** Remove a listener from this EventDispatcher. */`
`124`		`- public override fun removeListener(listener: EventDispatcherListener) {`
	`124`	`+ override fun removeListener(listener: EventDispatcherListener) {`
`125`	`125`	`listeners.remove(listener)`
`126`	`126`	`}`
`127`	`127`
`128`		`- public override fun addBatchEventDispatchedListener(listener: BatchEventDispatchedListener) {`
	`128`	`+ override fun addBatchEventDispatchedListener(listener: BatchEventDispatchedListener) {`
`129`	`129`	`postEventDispatchListeners.add(listener)`
`130`	`130`	`}`
`131`	`131`
`132`		`- public override fun removeBatchEventDispatchedListener(listener: BatchEventDispatchedListener) {`
	`132`	`+ override fun removeBatchEventDispatchedListener(listener: BatchEventDispatchedListener) {`
`133`	`133`	`postEventDispatchListeners.remove(listener)`
`134`	`134`	`}`
`135`	`135`
`136`		`- public override fun onHostResume() {`
	`136`	`+ override fun onHostResume() {`
`137`	`137`	`scheduleDispatchOfBatchedEvents()`
`138`	`138`	`if (!ReactNativeFeatureFlags.useOptimizedEventBatchingOnAndroid()) {`
`139`	`139`	`currentFrameCallback.resume()`
`140`	`140`	`}`
`141`	`141`	`}`
`142`	`142`
`143`		`- public override fun onHostPause() {`
	`143`	`+ override fun onHostPause() {`
`144`	`144`	`cancelDispatchOfBatchedEvents()`
`145`	`145`	`}`
`146`	`146`
`147`		`- public override fun onHostDestroy() {`
	`147`	`+ override fun onHostDestroy() {`
`148`	`148`	`cancelDispatchOfBatchedEvents()`
`149`	`149`	`}`
`150`	`150`
`151`		`- public fun invalidate() {`
	`151`	`+ fun invalidate() {`
`152`	`152`	`eventEmitter.registerFabricEventEmitter(null)`
`153`	`153`
`154`	`154`	`UiThreadUtil.runOnUiThread { cancelDispatchOfBatchedEvents() }`
`155`	`155`	`}`
`156`	`156`
`157`	`157`	`@Deprecated("Private API, should only be used when the concrete implementation is known.")`
`158`		`- public override fun onCatalystInstanceDestroyed() {`
	`158`	`+ override fun onCatalystInstanceDestroyed() {`
`159`	`159`	`invalidate()`
`160`	`160`	`}`
`161`	`161`
Original file line number	Diff line number	Diff line change
`@@ -10,5 +10,5 @@ package com.facebook.react.uimanager.style`
`10`	`10`	`import android.graphics.Shader`
`11`	`11`
`12`	`12`	`internal interface Gradient {`
`13`		`- public fun getShader(width: Float, height: Float): Shader`
	`13`	`+ fun getShader(width: Float, height: Float): Shader`
`14`	`14`	`}`