blallo
/
fenix
1
0
Fork 0
Code Releases Activity

For #8950: Unregister observers in the main scope coroutine

master
Jonathan Almeida 2020-03-03 15:02:40 -05:00 committed by Jonathan Almeida
parent 2603683d42
commit 0d48b65a25
1 changed files with 8 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
app/src/main/java/org/mozilla/fenix/push/PushFxaIntegration.kt
View File

@ -6,6 +6,7 @@ package org.mozilla.fenix.push
import kotlinx.coroutines.CoroutineScope
import kotlinx.coroutines.Dispatchers
import kotlinx.coroutines.MainScope
import kotlinx.coroutines.launch
import mozilla.components.concept.sync.AccountObserver
import mozilla.components.concept.sync.AuthType
@ -100,8 +101,10 @@ internal class OneTimePushMessageObserver(
}
}
// Remove ourselves when we're done.
pushFeature.unregister(this)
MainScope().launch {
// Remove ourselves when we're done.
pushFeature.unregister(this@OneTimePushMessageObserver)
}
}
}
}
@ -122,6 +125,8 @@ internal class OneTimeMessageDeliveryObserver(
it.processRawEventAsync(String(message))
}
lazyAccount.value.unregister(this)
MainScope().launch {
lazyAccount.value.unregister(this@OneTimeMessageDeliveryObserver)
}
}
}