commit fab508f32cd8e7ccb6f2458267edb7b601fdcd5c
author Yuan Yao <yuanyaogoog@google.com> Mon Mar 10 04:14:54 2025 +0000
committer Yuan Yao <yuanyaogoog@google.com> Mon Mar 10 04:29:52 2025 +0000
tree 2e8bf92cdb3467edee87b35c292e10e3cbb394f2
parent ee3fcfb9f8acff6e95355e332b05b5682c453cf7

TerminalApp: Remove onDestroy() from ApplicationLifecycleObserver

This CL removes onDestroy() from ApplicationLifecycleObserver, since it
will never be called.

Bug: b/392791968
Test: Verify onDestroy() function is not called when APP exits
Change-Id: I9b42883ed9642d6e1c78dfff6039e710b9bb41e7

android/TerminalApp/java/com/android/virtualization/terminal/Application.kt

diff --git a/android/TerminalApp/java/com/android/virtualization/terminal/Application.kt b/android/TerminalApp/java/com/android/virtualization/terminal/Application.kt
index 9f4909d..c427337 100644
--- a/android/TerminalApp/java/com/android/virtualization/terminal/Application.kt
+++ b/android/TerminalApp/java/com/android/virtualization/terminal/Application.kt
@@ -97,14 +97,6 @@
             super.onStop(owner)
         }
 
-        override fun onDestroy(owner: LifecycleOwner) {
-            if (vmLauncherService != null) {
-                this@Application.unbindService(connection)
-                vmLauncherService = null
-            }
-            super.onDestroy(owner)
-        }
-
         fun bindToVmLauncherService() {
             val intent = Intent(this@Application, VmLauncherService::class.java)
             this@Application.bindService(intent, connection, 0) // No BIND_AUTO_CREATE