perf(android): Replace RNSentryFrameDelayCollector with sentry-java getFramesDelay API

packages/core/android/src/main/java/io/sentry/react/RNSentryModuleImpl.java

@@ -48,6 +48,7 @@
 import io.sentry.android.core.SentryShakeDetector;
 import io.sentry.android.core.ViewHierarchyEventProcessor;
 import io.sentry.android.core.internal.debugmeta.AssetsDebugMetaLoader;
+import io.sentry.android.core.SentryFramesDelayResult;
 import io.sentry.android.core.internal.util.SentryFrameMetricsCollector;
 import io.sentry.android.core.performance.AppStartMetrics;
 import io.sentry.protocol.Geo;
@@ -98,7 +99,8 @@ public class RNSentryModuleImpl {
   private final ReactApplicationContext reactApplicationContext;
   private final PackageInfo packageInfo;
   private FrameMetricsAggregator frameMetricsAggregator = null;
-  private final RNSentryFrameDelayCollector frameDelayCollector = new RNSentryFrameDelayCollector();
+  private @Nullable SentryFrameMetricsCollector frameMetricsCollector = null;
+  private @Nullable String frameMetricsListenerId = null;
   private boolean androidXAvailable;
 
   @VisibleForTesting static long lastStartTimestampMs = -1;
@@ -413,9 +415,15 @@ public void fetchNativeFramesDelay(
       long startNanos = nowNanos - (long) (startOffsetSeconds * 1e9);
       long endNanos = nowNanos - (long) (endOffsetSeconds * 1e9);
 
-      double delaySeconds = frameDelayCollector.getFramesDelay(startNanos, endNanos);
-      if (delaySeconds >= 0) {
-        promise.resolve(delaySeconds);
+      if (frameMetricsCollector == null) {
+        promise.resolve(null);
+        return;
+      }
+
+      SentryFramesDelayResult result =
+          frameMetricsCollector.getFramesDelay(startNanos, endNanos);
+      if (result.getDelaySeconds() >= 0) {
+        promise.resolve(result.getDelaySeconds());
       } else {
         promise.resolve(null);
       }
@@ -747,12 +755,22 @@ public void enableNativeFramesTracking() {
       if (options instanceof SentryAndroidOptions) {
         final SentryFrameMetricsCollector collector =
             ((SentryAndroidOptions) options).getFrameMetricsCollector();
-        if (frameDelayCollector.start(collector)) {
-          logger.log(SentryLevel.INFO, "RNSentryFrameDelayCollector installed.");
+        if (collector != null) {
+          // Register a no-op listener to ensure frame metrics collection is active.
+          // This is needed so that getFramesDelay() has data to query.
+          stopFrameMetricsCollection();
+          frameMetricsCollector = collector;
+          frameMetricsListenerId =
+              collector.startCollection(
+                  (startNanos, endNanos, durationNanos, delayNanos, isSlow, isFrozen, refreshRate)
+                      -> {});
+          if (frameMetricsListenerId != null) {
+            logger.log(SentryLevel.INFO, "SentryFrameMetricsCollector listener installed.");
+          }
         }
       }
     } catch (Throwable ignored) { // NOPMD - We don't want to crash in any case
-      logger.log(SentryLevel.WARNING, "Error starting RNSentryFrameDelayCollector.");
+      logger.log(SentryLevel.WARNING, "Error starting frame metrics collection.");
     }
   }
 
@@ -761,7 +779,15 @@ public void disableNativeFramesTracking() {
       frameMetricsAggregator.stop();
       frameMetricsAggregator = null;
     }
-    frameDelayCollector.stop();
+    stopFrameMetricsCollection();
+  }
+
+  private void stopFrameMetricsCollection() {
+    if (frameMetricsCollector != null && frameMetricsListenerId != null) {
+      frameMetricsCollector.stopCollection(frameMetricsListenerId);
+    }
+    frameMetricsCollector = null;
+    frameMetricsListenerId = null;
   }
 
   public void getNewScreenTimeToDisplay(Promise promise) {