RUM-16664: Replace viewTrackingStrategy type checks with ReplayableViewTrackingStrategy

Author: sbarrio

features/dd-sdk-android-rum/src/main/kotlin/com/datadog/android/rum/internal/tracking/ReplayableViewTrackingStrategy.kt

@@ -0,0 +1,27 @@
+/*
+ * Unless explicitly stated otherwise all files in this repository are licensed under the Apache License Version 2.0.
+ * This product includes software developed at Datadog (https://www.datadoghq.com/).
+ * Copyright 2016-Present Datadog, Inc.
+ */
+
+package com.datadog.android.rum.internal.tracking
+
+import android.app.Activity
+
+/**
+ * Opt-in interface for [com.datadog.android.rum.tracking.ViewTrackingStrategy] implementations
+ * that can handle the late-init scenario: the RUM SDK initialised after the first Activity had
+ * already started (e.g. a cross-platform bridge delay), so the normal lifecycle callbacks were
+ * never delivered for that Activity.
+ *
+ * Implementing this interface lets [com.datadog.android.rum.internal.RumFeature] replay the
+ * relevant lifecycle event without coupling to concrete strategy types.
+ */
+internal interface ReplayableViewTrackingStrategy {
+    /**
+     * Called when the SDK initialises after [activity] has already started/resumed and no RUM
+     * view has been opened for it yet. Implementations should start a view as if the normal
+     * lifecycle callback had fired.
+     */
+    fun onLateActivityReady(activity: Activity)
+}

features/dd-sdk-android-rum/src/main/kotlin/com/datadog/android/rum/tracking/ActivityViewTrackingStrategy.kt

@@ -16,6 +16,7 @@ import com.datadog.android.internal.attributes.ViewScopeInstrumentationType
 import com.datadog.android.internal.attributes.enrichWithConstantAttribute
 import com.datadog.android.rum.GlobalRumMonitor
 import com.datadog.android.rum.RumMonitor
+import com.datadog.android.rum.internal.tracking.ReplayableViewTrackingStrategy
 import com.datadog.android.rum.internal.utils.resolveViewName
 import com.datadog.android.rum.internal.utils.runIfValid
 import java.util.concurrent.ScheduledExecutorService
@@ -37,7 +38,8 @@ constructor(
     internal val componentPredicate: ComponentPredicate<Activity> = AcceptAllActivities()
 ) :
     ActivityLifecycleTrackingStrategy(),
-    ViewTrackingStrategy {
+    ViewTrackingStrategy,
+    ReplayableViewTrackingStrategy {
 
     private val executor: ScheduledExecutorService by lazy {
         sdkCore.createScheduledExecutorService(
@@ -83,6 +85,14 @@ constructor(
 
     // endregion
 
+    // region ReplayableViewTrackingStrategy
+
+    override fun onLateActivityReady(activity: Activity) {
+        onActivityResumed(activity)
+    }
+
+    // endregion
+
     // region Object
 
     override fun equals(other: Any?): Boolean {

features/dd-sdk-android-rum/src/main/kotlin/com/datadog/android/rum/tracking/NavigationViewTrackingStrategy.kt

@@ -23,6 +23,7 @@ import com.datadog.android.rum.GlobalRumMonitor
 import com.datadog.android.rum.NoOpRumMonitor
 import com.datadog.android.rum.internal.RumFeature
 import com.datadog.android.rum.internal.tracking.AndroidXFragmentLifecycleCallbacks
+import com.datadog.android.rum.internal.tracking.ReplayableViewTrackingStrategy
 import com.datadog.android.rum.internal.utils.resolveViewName
 import com.datadog.android.rum.internal.utils.runIfValid
 import java.util.WeakHashMap
@@ -42,7 +43,8 @@ class NavigationViewTrackingStrategy(
 ) :
     ActivityLifecycleTrackingStrategy(),
     ViewTrackingStrategy,
-    NavController.OnDestinationChangedListener {
+    NavController.OnDestinationChangedListener,
+    ReplayableViewTrackingStrategy {
 
     private var startedActivity: Activity? = null
 
@@ -87,6 +89,14 @@ class NavigationViewTrackingStrategy(
 
     // endregion
 
+    // region ReplayableViewTrackingStrategy
+
+    override fun onLateActivityReady(activity: Activity) {
+        onActivityStarted(activity)
+    }
+
+    // endregion
+
     // region OnDestinationChangedListener
 
     override fun onDestinationChanged(