Only call appBackground if crash was on foreground

Author: mrober

firebase-sessions/src/main/kotlin/com/google/firebase/sessions/SharedSessionRepository.kt

@@ -34,6 +34,8 @@ import kotlinx.coroutines.launch
 
 /** Repository to persist session data to be shared between all app processes. */
 internal interface SharedSessionRepository {
+  val isInForeground: Boolean
+
   fun appBackground()
 
   fun appForeground()
@@ -59,6 +61,9 @@ constructor(
   /** Local copy of the session data. Can get out of sync, must be double-checked in datastore. */
   internal lateinit var localSessionData: SessionData
 
+  override var isInForeground = false
+    private set
+
   /**
    * Either notify the subscribers with general multi-process supported session or fallback local
    * session
@@ -95,6 +100,7 @@ constructor(
   }
 
   override fun appBackground() {
+    isInForeground = false
     if (!::localSessionData.isInitialized) {
       Log.d(TAG, "App backgrounded, but local SessionData not initialized")
       return
@@ -115,6 +121,7 @@ constructor(
   }
 
   override fun appForeground() {
+    isInForeground = true
     if (!::localSessionData.isInitialized) {
       Log.d(TAG, "App foregrounded, but local SessionData not initialized")
       return

firebase-sessions/src/main/kotlin/com/google/firebase/sessions/api/CrashEventReceiver.kt

@@ -38,8 +38,11 @@ object CrashEventReceiver {
   @JvmStatic
   fun notifyCrashOccurred() {
     try {
-      // Treat the crash as if the app went to the background, and update session state.
-      SharedSessionRepository.instance.appBackground()
+      // Treat a foreground crash as if the app went to the background, and update session state.
+      val sharedSessionRepository = SharedSessionRepository.instance
+      if (sharedSessionRepository.isInForeground) {
+        sharedSessionRepository.appBackground()
+      }
     } catch (_: Exception) {
       // Catch and suppress any exception to avoid crashing during crash handling.
       // This can occur if Firebase or the SDK are in an unexpected state (e.g. FirebaseApp deleted)