android: fix Quick Settings tailscale

-Get rid of unused stopVPN() function
-Get rid of unused ACTION_STOP_VPN intent handling; this is redundant with DISCONNECT_VPN intent
-Tile active state should only depend on ipn state, and not the results of editing the prefs with wantRunning set. It should be active iff ipn.State > Stopped

Fixes tailscale/tailscale#11920

Signed-off-by: kari-ts <kari@tailscale.com>

android/src/main/java/com/tailscale/ipn/App.kt

@@ -87,8 +87,6 @@ class App : Application(), libtailscale.AppContext {
   }
 
   val dns = DnsConfig()
-  var autoConnect = false
-  var vpnReady = false
   private lateinit var connectivityManager: ConnectivityManager
   private lateinit var app: libtailscale.Application
 
@@ -137,7 +135,7 @@ class App : Application(), libtailscale.AppContext {
   fun setWantRunning(wantRunning: Boolean) {
     val callback: (Result<Ipn.Prefs>) -> Unit = { result ->
       result.fold(
-          onSuccess = { _ -> setTileStatus(wantRunning) },
+          onSuccess = { },
           onFailure = { error ->
             Log.d("TAG", "Set want running: failed to update preferences: ${error.message}")
           })
@@ -187,11 +185,6 @@ class App : Application(), libtailscale.AppContext {
     startService(intent)
   }
 
-  fun stopVPN() {
-    val intent = Intent(this, IPNService::class.java)
-    intent.setAction(IPNService.ACTION_STOP_VPN)
-    startService(intent)
-  }
 
   // encryptToPref a byte array of data using the Jetpack Security
   // library and writes it to a global encrypted preference store.
@@ -224,20 +217,12 @@ class App : Application(), libtailscale.AppContext {
       return
     }
     QuickToggleService.setReady(this, ready)
-    Log.d("App", "Set Tile Ready: $ready $autoConnect")
-    vpnReady = ready
-    if (ready && autoConnect) {
+    Log.d("App", "Set Tile Ready: $ready")
+    if (ready){
       startVPN()
     }
   }
 
-  fun setTileStatus(status: Boolean) {
-    if (Build.VERSION.SDK_INT < Build.VERSION_CODES.N) {
-      return
-    }
-    QuickToggleService.setStatus(this, status)
-  }
-
   fun getHostname(): String {
     val userConfiguredDeviceName = getUserConfiguredDeviceName()
     if (!userConfiguredDeviceName.isNullOrEmpty()) return userConfiguredDeviceName

android/src/main/java/com/tailscale/ipn/IPNService.kt

@@ -21,11 +21,6 @@ open class IPNService : VpnService(), libtailscale.IPNService {
   }
 
   override fun onStartCommand(intent: Intent?, flags: Int, startId: Int): Int {
-    if (intent != null && ACTION_STOP_VPN == intent.action) {
-      (applicationContext as App).autoConnect = false
-      close()
-      return START_NOT_STICKY
-    }
     val app = applicationContext as App
     if (intent != null && "android.net.VpnService" == intent.action) {
       // Start VPN and connect to it due to Always-on VPN
@@ -38,9 +33,9 @@ open class IPNService : VpnService(), libtailscale.IPNService {
       return START_STICKY
     }
     Libtailscale.requestVPN(this)
-    if (app.vpnReady && app.autoConnect) {
+    //if (app.autoConnect) {
       app.setWantRunning(true)
-    }
+    //}
     return START_STICKY
   }
 
@@ -134,6 +129,5 @@ open class IPNService : VpnService(), libtailscale.IPNService {
 
   companion object {
     const val ACTION_REQUEST_VPN = "com.tailscale.ipn.REQUEST_VPN"
-    const val ACTION_STOP_VPN = "com.tailscale.ipn.STOP_VPN"
   }
 }

android/src/main/java/com/tailscale/ipn/QuickToggleService.java

@@ -13,8 +13,6 @@
 public class QuickToggleService extends TileService {
     // lock protects the static fields below it.
     private static final Object lock = new Object();
-    // Active tracks whether the VPN is active.
-    private static boolean active;
     // Ready tracks whether the tailscale backend is
     // ready to switch on/off.
     private static boolean ready;
@@ -28,7 +26,7 @@ private static void updateTile() {
         boolean act;
         synchronized (lock) {
             t = currentTile;
-            act = active && ready;
+            act = ready;
         }
         if (t == null) {
             return;
@@ -44,13 +42,6 @@ static void setReady(Context ctx, boolean rdy) {
         updateTile();
     }
 
-    static void setStatus(Context ctx, boolean act) {
-        synchronized (lock) {
-            active = act;
-        }
-        updateTile();
-    }
-
     @Override
     public void onStartListening() {
         synchronized (lock) {
@@ -88,7 +79,7 @@ public void onClick() {
     private void onTileClick() {
         boolean act;
         synchronized (lock) {
-            act = active && ready;
+            act = ready;
         }
         Intent i = new Intent(act ? IPNReceiver.INTENT_DISCONNECT_VPN : IPNReceiver.INTENT_CONNECT_VPN);
         i.setPackage(getPackageName());

android/src/main/java/com/tailscale/ipn/StartVPNWorker.java

@@ -27,7 +27,7 @@ public Result doWork() {
         App app = ((App) getApplicationContext());
 
         // We will start the VPN from the background
-        app.setAutoConnect(true);
+       // app.setAutoConnect(true);
         // We need to make sure we prepare the VPN Service, just in case it isn't prepared.
 
         Intent intent = VpnService.prepare(app);

android/src/main/java/com/tailscale/ipn/ui/notifier/Notifier.kt

@@ -82,7 +82,7 @@ object Notifier {
           }
       state.collect { currstate ->
         readyToPrepareVPN.set(currstate > Ipn.State.Stopped)
-        tileReady.set(currstate >= Ipn.State.Stopped)
+        tileReady.set(currstate > Ipn.State.Stopped)
       }
     }
   }