fix: inconsistent audio message length [WPB-3334] (#2041)

Co-authored-by: Mateusz <m.pachulski94@gmail.com>
Co-authored-by: yamilmedina <yamilmedina@users.noreply.github.com>

app/src/main/kotlin/com/wire/android/media/audiomessage/AudioState.kt

@@ -9,16 +9,14 @@ data class AudioState(
         val DEFAULT = AudioState(AudioMediaPlayingState.Stopped, 0, TotalTimeInMs.NotKnown)
     }
 
-    // before the user decides to play audio message, we are not able to determine total time ourself using
-    // MediaPlayer API, we are relying on the info retrieved from the other client, until then
-    fun sanitizeTotalTime(otherClientTotalTime: Int): AudioState {
-        if (totalTimeInMs is TotalTimeInMs.NotKnown) {
-            if (otherClientTotalTime != 0) {
-                return copy(totalTimeInMs = TotalTimeInMs.Known(otherClientTotalTime))
-            }
+    // if the back-end returned the total time, we use that, in case it didn't we use what we get from
+    // the [ConversationAudioMessagePlayer.kt] which will emit the time once the users play the audio.
+    fun sanitizeTotalTime(otherClientTotalTime: Int): TotalTimeInMs {
+        if (otherClientTotalTime != 0) {
+           return TotalTimeInMs.Known(otherClientTotalTime)
         }
 
-        return this
+        return totalTimeInMs
     }
 
     sealed class TotalTimeInMs {

app/src/main/kotlin/com/wire/android/media/audiomessage/ConversationAudioMessagePlayer.kt

@@ -2,6 +2,7 @@ package com.wire.android.media.audiomessage
 
 import android.content.Context
 import android.media.MediaPlayer
+import android.media.MediaPlayer.SEEK_CLOSEST_SYNC
 import android.net.Uri
 import com.wire.kalium.logic.data.id.ConversationId
 import com.wire.kalium.logic.feature.asset.GetMessageAssetUseCase
@@ -110,7 +111,9 @@ class ConversationAudioMessagePlayer
                     audioMessageStateHistory = audioMessageStateHistory.toMutableMap().apply {
                         put(
                             audioMessageStateUpdate.messageId,
-                            currentState.copy(totalTimeInMs = AudioState.TotalTimeInMs.Known(audioMessageStateUpdate.totalTimeInMs))
+                            currentState.copy(
+                                totalTimeInMs = AudioState.TotalTimeInMs.Known(audioMessageStateUpdate.totalTimeInMs)
+                            )
                         )
                     }
                 }
@@ -241,7 +244,7 @@ class ConversationAudioMessagePlayer
             val isAudioMessageCurrentlyPlaying = currentAudioMessageId == messageId
 
             if (isAudioMessageCurrentlyPlaying) {
-                audioMediaPlayer.seekTo(position)
+                audioMediaPlayer.seekTo(position.toLong(), SEEK_CLOSEST_SYNC)
             }
         }
 

app/src/main/kotlin/com/wire/android/ui/home/conversations/MessageItem.kt

@@ -547,14 +547,14 @@ private fun MessageContent(
             val audioMessageState: AudioState = audioMessagesState[message.header.messageId]
                 ?: AudioState.DEFAULT
 
-            val adjustedMessageState: AudioState = remember(audioMessagesState) {
+            val totalTimeInMs = remember(audioMessageState.totalTimeInMs) {
                 audioMessageState.sanitizeTotalTime(messageContent.audioMessageDurationInMs.toInt())
             }
 
             AudioMessage(
-                audioMediaPlayingState = adjustedMessageState.audioMediaPlayingState,
-                totalTimeInMs = adjustedMessageState.totalTimeInMs,
-                currentPositionInMs = adjustedMessageState.currentPositionInMs,
+                audioMediaPlayingState = audioMessageState.audioMediaPlayingState,
+                totalTimeInMs = totalTimeInMs,
+                currentPositionInMs = audioMessageState.currentPositionInMs,
                 onPlayButtonClick = { onAudioClick(message.header.messageId) },
                 onSliderPositionChange = { position ->
                     onChangeAudioPosition(message.header.messageId, position.toInt())