From 7818e15e185a2b8b07bb3bb24401a334aae33f1c Mon Sep 17 00:00:00 2001 From: Matthew Robertson Date: Fri, 20 Oct 2023 16:24:58 -0400 Subject: [PATCH] Don't wrap sendLifecycleEvents twice --- .../sessions/SessionLifecycleClient.kt | 19 +++++++------------ 1 file changed, 7 insertions(+), 12 deletions(-) diff --git a/firebase-sessions/src/main/kotlin/com/google/firebase/sessions/SessionLifecycleClient.kt b/firebase-sessions/src/main/kotlin/com/google/firebase/sessions/SessionLifecycleClient.kt index 678fec13cde..4211fff61b3 100644 --- a/firebase-sessions/src/main/kotlin/com/google/firebase/sessions/SessionLifecycleClient.kt +++ b/firebase-sessions/src/main/kotlin/com/google/firebase/sessions/SessionLifecycleClient.kt @@ -23,7 +23,6 @@ import android.content.ServiceConnection import android.os.Handler import android.os.HandlerThread import android.os.IBinder -import android.os.Looper import android.os.Message import android.os.Messenger import android.os.RemoteException @@ -60,7 +59,7 @@ internal object SessionLifecycleClient { init { handlerThread.start() } - + /** * The callback class that will be used to receive updated session events from the * [SessionLifecycleService]. @@ -150,16 +149,14 @@ internal object SessionLifecycleClient { sendLifecycleEvent(SessionLifecycleService.BACKGROUNDED) } - /** - * Perform initialization that requires cleanup - */ + /** Perform initialization that requires cleanup */ fun started() { - if (!handlerThread.isAlive) { handlerThread.start() } + if (!handlerThread.isAlive) { + handlerThread.start() + } } - /** - * Cleanup initialization - */ + /** Cleanup initialization */ fun stopped() { handlerThread.quit() } @@ -172,9 +169,7 @@ internal object SessionLifecycleClient { private fun sendLifecycleEvent(messageCode: Int) { val allMessages = drainQueue() allMessages.add(Message.obtain(null, messageCode, 0, 0)) - CoroutineScope(Dispatchers.instance.backgroundDispatcher).launch { - sendLifecycleEvents(allMessages) - } + sendLifecycleEvents(allMessages) } /**