Update call service

tchapgouv · Nov 16, 2024 · 0e75b7f · 0e75b7f
1 parent 5eac088
commit 0e75b7f
Show file tree

Hide file tree

Showing 3 changed files with 25 additions and 25 deletions.
diff --git a/vector/src/main/AndroidManifest.xml b/vector/src/main/AndroidManifest.xml
@@ -44,7 +44,6 @@
 
     <!-- For CallAndroidService -->
     <uses-permission android:name="android.permission.FOREGROUND_SERVICE_PHONE_CALL" />
-    <uses-permission android:name="android.permission.FOREGROUND_SERVICE_MICROPHONE" />
 
     <!-- For VectorSyncAndroidService -->
     <uses-permission android:name="android.permission.FOREGROUND_SERVICE_DATA_SYNC" />
@@ -58,6 +57,9 @@
     <!-- Jitsi SDK is now API23+ -->
     <uses-sdk tools:overrideLibrary="com.swmansion.gesturehandler,org.jitsi.meet.sdk,com.oney.WebRTCModule,com.learnium.RNDeviceInfo,com.reactnativecommunity.asyncstorage,com.ocetnik.timer,com.calendarevents,com.reactnativecommunity.netinfo,com.kevinresol.react_native_default_preference,com.rnimmersive,com.corbt.keepawake,com.BV.LinearGradient,com.horcrux.svg,com.oblador.performance,com.reactnativecommunity.slider,com.brentvatne.react,com.reactnativecommunity.clipboard,com.swmansion.gesturehandler.react,org.linusu,org.reactnative.maskedview,com.reactnativepagerview,com.swmansion.reanimated,com.th3rdwave.safeareacontext,com.swmansion.rnscreens,org.devio.rn.splashscreen,com.reactnativecommunity.webview,org.wonday.orientation" />
 
+    <!-- For MicrophoneAccessService -->
+    <uses-permission android:name="android.permission.FOREGROUND_SERVICE_MICROPHONE" />
+
     <!-- Tchap: Jitsi libs adds CALENDAR permissions, but we can remove them safely
     according to https://github.com/jitsi/jitsi-meet/issues/4068#issuecomment-480482481 -->
     <uses-permission android:name="android.permission.READ_CALENDAR" tools:node="remove" />

diff --git a/vector/src/withdmvoip/java/im/vector/app/features/call/VectorCallActivity.kt b/vector/src/withdmvoip/java/im/vector/app/features/call/VectorCallActivity.kt
@@ -251,19 +251,17 @@ class VectorCallActivity :
     }
 
     private fun startMicrophoneService() {
-        if (ContextCompat.checkSelfPermission(this, Manifest.permission.RECORD_AUDIO)
-                == PackageManager.PERMISSION_GRANTED) {
-            // Only start the service if the app is in the foreground
-            if (isAppInForeground()) {
-                Timber.tag(loggerTag.value).v("Starting microphone foreground service")
-                val intent = Intent(this, MicrophoneAccessService::class.java)
-                ContextCompat.startForegroundService(this, intent)
-            } else {
-                Timber.tag(loggerTag.value).v("App is not in foreground; cannot start microphone service")
-            }
-        } else {
-            Timber.tag(loggerTag.value).v("Microphone permission not granted; cannot start service")
+        val isConnected = withState(callViewModel) {
+            val callState = it.callState.invoke()
+            callState is CallState.Connected
         }
+        if (!isAppInForeground()) return
+        if (!isConnected) return
+        if (ContextCompat.checkSelfPermission(this, Manifest.permission.RECORD_AUDIO) == PackageManager.PERMISSION_DENIED) return
+
+        Timber.tag(loggerTag.value).d("Starting microphone foreground service")
+        val intent = Intent(this, MicrophoneAccessService::class.java)
+        ContextCompat.startForegroundService(this, intent)
     }
 
     private fun isAppInForeground(): Boolean {
@@ -292,6 +290,7 @@ class VectorCallActivity :
         turnScreenOffAndKeyguardOn()
         removeOnPictureInPictureModeChangedListener(pictureInPictureModeChangedInfoConsumer)
         screenCaptureServiceConnection.unbind()
+        stopMicrophoneService()
         super.onDestroy()
     }
 

diff --git a/vector/src/withvoip/java/im/vector/app/features/call/VectorCallActivity.kt b/vector/src/withvoip/java/im/vector/app/features/call/VectorCallActivity.kt
@@ -251,19 +251,17 @@ class VectorCallActivity :
     }
 
     private fun startMicrophoneService() {
-        if (ContextCompat.checkSelfPermission(this, Manifest.permission.RECORD_AUDIO)
-                == PackageManager.PERMISSION_GRANTED) {
-            // Only start the service if the app is in the foreground
-            if (isAppInForeground()) {
-                Timber.tag(loggerTag.value).v("Starting microphone foreground service")
-                val intent = Intent(this, MicrophoneAccessService::class.java)
-                ContextCompat.startForegroundService(this, intent)
-            } else {
-                Timber.tag(loggerTag.value).v("App is not in foreground; cannot start microphone service")
-            }
-        } else {
-            Timber.tag(loggerTag.value).v("Microphone permission not granted; cannot start service")
+        val isConnected = withState(callViewModel) {
+            val callState = it.callState.invoke()
+            callState is CallState.Connected
         }
+        if (!isAppInForeground()) return
+        if (!isConnected) return
+        if (ContextCompat.checkSelfPermission(this, Manifest.permission.RECORD_AUDIO) == PackageManager.PERMISSION_DENIED) return
+
+        Timber.tag(loggerTag.value).d("Starting microphone foreground service")
+        val intent = Intent(this, MicrophoneAccessService::class.java)
+        ContextCompat.startForegroundService(this, intent)
     }
 
     private fun isAppInForeground(): Boolean {
@@ -292,6 +290,7 @@ class VectorCallActivity :
         turnScreenOffAndKeyguardOn()
         removeOnPictureInPictureModeChangedListener(pictureInPictureModeChangedInfoConsumer)
         screenCaptureServiceConnection.unbind()
+        stopMicrophoneService()
         super.onDestroy()
     }