summaryrefslogtreecommitdiff
path: root/service
diff options
context:
space:
mode:
authorBodam Nam <bodamnam@google.com>2023-03-22 05:17:18 +0000
committerBodam Nam <bodamnam@google.com>2023-03-22 05:21:27 +0000
commitb1a4f02e6d4d60c36ea5d32e902abd744aeb56a3 (patch)
treec9c0f29a7c509465f4e668e32b9a019bde19b7dd /service
parentd48560af47c40dd0259a3b5e233a3d8d1bc211ab (diff)
downloadImsMedia-b1a4f02e6d4d60c36ea5d32e902abd744aeb56a3.tar.gz
Revert "Fix the SID playout delay issue"
This reverts commit d48560af47c40dd0259a3b5e233a3d8d1bc211ab. Reason for revert: atest failure Bug: 274715723 Change-Id: I681fc792f219467557355e5fe73d6fdb032e37f6
Diffstat (limited to 'service')
-rw-r--r--service/src/com/android/telephony/imsmedia/lib/libimsmedia/core/audio/AudioJitterBuffer.cpp16
1 files changed, 3 insertions, 13 deletions
diff --git a/service/src/com/android/telephony/imsmedia/lib/libimsmedia/core/audio/AudioJitterBuffer.cpp b/service/src/com/android/telephony/imsmedia/lib/libimsmedia/core/audio/AudioJitterBuffer.cpp
index ac152b2b..5b948204 100644
--- a/service/src/com/android/telephony/imsmedia/lib/libimsmedia/core/audio/AudioJitterBuffer.cpp
+++ b/service/src/com/android/telephony/imsmedia/lib/libimsmedia/core/audio/AudioJitterBuffer.cpp
@@ -284,20 +284,14 @@ bool AudioJitterBuffer::Get(ImsMediaSubType* psubtype, uint8_t** ppData, uint32_
if (mDataQueue.GetCount() == 0)
{
- IMLOGD_PACKET0(IM_PACKET_LOG_JITTER, "[Get] fail - empty");
-
- if (!mWaiting)
- {
- mCurrPlayingTS += FRAME_INTERVAL;
- }
-
+ IMLOGD_PACKET0(IM_PACKET_LOG_JITTER, "[Get] wait - empty");
return false;
}
else if (mDataQueue.Get(&pEntry) && mWaiting)
{
uint32_t jitterDelay = currentTime - pEntry->arrivalTime;
- if (jitterDelay < (mCurrJitterBufferSize - 1) * FRAME_INTERVAL)
+ if (jitterDelay < (mCurrJitterBufferSize - 1) * FRAME_INTERVAL + ALLOWABLE_ERROR)
{
if (psubtype)
*psubtype = MEDIASUBTYPE_UNDEFINED;
@@ -327,10 +321,6 @@ bool AudioJitterBuffer::Get(ImsMediaSubType* psubtype, uint8_t** ppData, uint32_
}
else
{
- IMLOGD_PACKET4(IM_PACKET_LOG_JITTER,
- "[Get] Wait - seq[%u], CurrJBSize[%u], delay[%u], QueueCount[%u]",
- pEntry->nSeqNum, mCurrJitterBufferSize, jitterDelay, mDataQueue.GetCount());
- mCannotGetCount++;
return false;
}
}
@@ -603,7 +593,7 @@ bool AudioJitterBuffer::Resync(uint32_t currentTime)
{
uint32_t timeDiff = currentTime - entry->arrivalTime;
- if (timeDiff > mCurrJitterBufferSize * FRAME_INTERVAL + ALLOWABLE_ERROR)
+ if (timeDiff > mCurrJitterBufferSize * FRAME_INTERVAL)
{
CollectRxRtpStatus(entry->nSeqNum, kRtpStatusDiscarded);
mDataQueue.Delete();