From 552c83171bc63ae6eef82317e45022433653c978 Mon Sep 17 00:00:00 2001 From: Jakob Ivarsson Date: Thu, 9 Sep 2021 10:09:28 +0200 Subject: [PATCH] Update nack tracker when receiving empty packets. Bug: webrtc:7611 Change-Id: Ied50829f245ea99f8d737cffdf9a3d646de3fbaf Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/231641 Reviewed-by: Ivo Creusen Commit-Queue: Jakob Ivarsson Cr-Commit-Position: refs/heads/main@{#34958} --- modules/audio_coding/neteq/neteq_impl.cc | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/modules/audio_coding/neteq/neteq_impl.cc b/modules/audio_coding/neteq/neteq_impl.cc index 624d8d3a48..c7b98a5450 100644 --- a/modules/audio_coding/neteq/neteq_impl.cc +++ b/modules/audio_coding/neteq/neteq_impl.cc @@ -195,11 +195,12 @@ int NetEqImpl::InsertPacket(const RTPHeader& rtp_header, return kOK; } -void NetEqImpl::InsertEmptyPacket(const RTPHeader& /*rtp_header*/) { - // TODO(henrik.lundin) Handle NACK as well. This will make use of the - // rtp_header parameter. - // https://bugs.chromium.org/p/webrtc/issues/detail?id=7611 +void NetEqImpl::InsertEmptyPacket(const RTPHeader& rtp_header) { MutexLock lock(&mutex_); + if (nack_enabled_) { + nack_->UpdateLastReceivedPacket(rtp_header.sequenceNumber, + rtp_header.timestamp); + } controller_->RegisterEmptyPacket(); }