commit 552c83171bc63ae6eef82317e45022433653c978
author Jakob Ivarsson <jakobi@webrtc.org> Thu Sep 09 08:09:28 2021
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> Thu Sep 09 10:40:35 2021
tree 6df65f5f41b37a972d10b639c8582db0c856a81f
parent 11ddcf8b539b2ea4feb9d3933f0901237a069ed4

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 <ivoc@webrtc.org>
Commit-Queue: Jakob Ivarsson <jakobi@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#34958}

modules/audio_coding/neteq/neteq_impl.cc

diff --git a/modules/audio_coding/neteq/neteq_impl.cc b/modules/audio_coding/neteq/neteq_impl.cc
index 624d8d3..c7b98a5 100644
--- a/modules/audio_coding/neteq/neteq_impl.cc
+++ b/modules/audio_coding/neteq/neteq_impl.cc
@@ -195,11 +195,12 @@
   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();
 }