From 8bdb34732bed0dc45de4cad57ee859d1a58595bd Mon Sep 17 00:00:00 2001
From: Nicola Corti <ncor@meta.com>
Date: Thu, 21 Aug 2025 05:31:55 -0700
Subject: [PATCH] Delete internal `ReactPackageLogger` as no longer necessary
 (#53387)

Summary:
Pull Request resolved: https://github.com/facebook/react-native/pull/53387

This interface was internal and legacy arch only, so it can safely be removed.
I've also removed the logic inside `ReactInstanceManager` that was using it as no longer necessary.

Changelog:
[Internal] [Changed] -

Reviewed By: mdvacca

Differential Revision: D80626639

fbshipit-source-id: b173e71b92e29cebbfc5ed589e01ac295eda2bf0
---
 .../com/facebook/react/CoreModulesPackage.kt  | 10 +---------
 .../facebook/react/ReactInstanceManager.java  |  7 -------
 .../com/facebook/react/ReactPackageLogger.kt  | 20 -------------------
 3 files changed, 1 insertion(+), 36 deletions(-)
 delete mode 100644 packages/react-native/ReactAndroid/src/main/java/com/facebook/react/ReactPackageLogger.kt

diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/CoreModulesPackage.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/CoreModulesPackage.kt
index 9693326d710..34536799751 100644
--- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/CoreModulesPackage.kt
+++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/CoreModulesPackage.kt
@@ -66,7 +66,7 @@ internal class CoreModulesPackage(
     private val hardwareBackBtnHandler: DefaultHardwareBackBtnHandler,
     private val lazyViewManagersEnabled: Boolean,
     private val minTimeLeftInFrameForNonBatchedOperationMs: Int,
-) : BaseReactPackage(), ReactPackageLogger {
+) : BaseReactPackage() {
   /**
    * This method is overridden, since OSS does not run the annotation processor to generate
    * [CoreModulesPackage.ReactModuleInfoProvider] class. Here we check if it exists with the method
@@ -196,14 +196,6 @@ internal class CoreModulesPackage(
     }
   }
 
-  override fun startProcessPackage() {
-    ReactMarker.logMarker(ReactMarkerConstants.PROCESS_CORE_REACT_PACKAGE_START)
-  }
-
-  override fun endProcessPackage() {
-    ReactMarker.logMarker(ReactMarkerConstants.PROCESS_CORE_REACT_PACKAGE_END)
-  }
-
   private companion object {
     init {
       LegacyArchitectureLogger.assertLegacyArchitecture(
diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/ReactInstanceManager.java b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/ReactInstanceManager.java
index 18c93e63d86..04affe8015d 100644
--- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/ReactInstanceManager.java
+++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/ReactInstanceManager.java
@@ -1570,14 +1570,7 @@ public class ReactInstanceManager {
     SystraceMessage.beginSection(TRACE_TAG_REACT, "processPackage")
         .arg("className", reactPackage.getClass().getSimpleName())
         .flush();
-    if (reactPackage instanceof ReactPackageLogger) {
-      ((ReactPackageLogger) reactPackage).startProcessPackage();
-    }
     nativeModuleRegistryBuilder.processPackage(reactPackage);
-
-    if (reactPackage instanceof ReactPackageLogger) {
-      ((ReactPackageLogger) reactPackage).endProcessPackage();
-    }
     SystraceMessage.endSection(TRACE_TAG_REACT).flush();
   }
 
diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/ReactPackageLogger.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/ReactPackageLogger.kt
deleted file mode 100644
index 2ceffbf1366..00000000000
--- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/ReactPackageLogger.kt
+++ /dev/null
@@ -1,20 +0,0 @@
-/*
- * Copyright (c) Meta Platforms, Inc. and affiliates.
- *
- * This source code is licensed under the MIT license found in the
- * LICENSE file in the root directory of this source tree.
- */
-
-package com.facebook.react
-
-import com.facebook.react.common.annotations.internal.LegacyArchitecture
-import com.facebook.react.common.annotations.internal.LegacyArchitectureLogLevel
-
-/** Interface for the bridge to call for TTI start and end markers. */
-@Deprecated("This class is deprecated and will be removed in the next major release.")
-@LegacyArchitecture(logLevel = LegacyArchitectureLogLevel.ERROR)
-internal interface ReactPackageLogger {
-  fun startProcessPackage()
-
-  fun endProcessPackage()
-}