MainActivity: remove redundant Notifier start/stop (#293)

We're starting Notifier in App so that states are available outside of MA lifecycle. Updates tailscale/corp#18202 Signed-off-by: kari-ts <[email protected]>
tailscale · Apr 25, 2024 · b0fd028 · b0fd028
1 parent b3b299e
commit b0fd028
Showing 1 changed file with 0 additions and 5 deletions.
diff --git a/android/src/main/java/com/tailscale/ipn/MainActivity.kt b/android/src/main/java/com/tailscale/ipn/MainActivity.kt
@@ -63,7 +63,6 @@ import kotlinx.coroutines.cancel
 import kotlinx.coroutines.launch
 
 class MainActivity : ComponentActivity() {
-  private var notifierScope: CoroutineScope? = null
   private lateinit var requestVpnPermission: ActivityResultLauncher<Unit>
 
   companion object {
@@ -254,17 +253,13 @@ class MainActivity : ComponentActivity() {
 
   override fun onStart() {
     super.onStart()
-    val scope = CoroutineScope(Dispatchers.IO)
-    notifierScope = scope
-    Notifier.start(lifecycleScope)
 
     // (jonathan) TODO: Requesting VPN permissions onStart is a bit aggressive.  This should
     // be done when the user initiall starts the VPN
     requestVpnPermission()
   }
 
   override fun onStop() {
-    Notifier.stop()
     super.onStop()
     val restrictionsManager =
         this.getSystemService(Context.RESTRICTIONS_SERVICE) as RestrictionsManager