commit ecdedac3dabbbc0345e5a0f98bc2063cd38d236f
author Jakob Ivarsson <jakobi@webrtc.org> Tue Apr 25 18:42:38 2023
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Apr 26 13:07:12 2023
tree a768dd8d2910cb4b8f526568dec1a25c04035038
parent d1771e925dfb35e22963dd6ad41b91a93a840339

Remove NetEq simulation step size restriction.

This should not be relevant anymore and is causing some issues due to
SetMinimumDelay events early in the log.

Bug: None
Change-Id: Ib7e3c624608c9bceaed31bd6669db59887d24659
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/303580
Reviewed-by: Henrik Lundin <henrik.lundin@webrtc.org>
Commit-Queue: Jakob Ivarsson‎ <jakobi@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#39956}

modules/audio_coding/neteq/tools/neteq_test.cc

diff --git a/modules/audio_coding/neteq/tools/neteq_test.cc b/modules/audio_coding/neteq/tools/neteq_test.cc
index 82dd0ae..ab9a3e2 100644
--- a/modules/audio_coding/neteq/tools/neteq_test.cc
+++ b/modules/audio_coding/neteq/tools/neteq_test.cc
@@ -273,17 +273,8 @@
       prev_lifetime_stats_ = lifetime_stats;
       const bool no_more_packets_to_decode =
           !input_->NextPacketTime() && !operations_state.next_packet_available;
-      // End the simulation if the gap is too large. This indicates an issue
-      // with the event log file.
-      const bool simulation_step_too_large = result.simulation_step_ms > 1000;
-      if (simulation_step_too_large) {
-        // If we don't reset the step time, the large gap will be included in
-        // the simulation time, which can be a large distortion.
-        result.simulation_step_ms = 10;
-      }
-      result.is_simulation_finished = simulation_step_too_large ||
-                                      no_more_packets_to_decode ||
-                                      input_->ended();
+      result.is_simulation_finished =
+          no_more_packets_to_decode || input_->ended();
       prev_ops_state_ = operations_state;
       return result;
     }