NetEq: Change NetEq's ramp-up behavior after expansions
NetEq tapers down the audio produced through loss concealment when the
expansion has been going on for some time. When the audio packets starts
coming in again, there is a ramp-up that happens. This ramp-up could
before this change extend over more than one 10 ms block, which made
keeping track of the scaling factor necessary. With this change, we make
this ramp-up quicker in the rare cases when it lasted more than 10 ms,
so that it always ramps up to 100% within one block. This way, we can
remove the mute_factor_array.
This change breaks bit-exactness, but careful listening could not reveal
an audible difference.
This change is a part of a larger refactoring of NetEq's PLC code.
Bug: webrtc:9180
Change-Id: I4c513ce3ed8d66f9beec2abfb1f0c7ffaac7a21e
Reviewed-on: https://webrtc-review.googlesource.com/77180
Commit-Queue: Henrik Lundin <henrik.lundin@webrtc.org>
Reviewed-by: Minyue Li <minyue@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#23342}
diff --git a/modules/audio_coding/neteq/neteq_unittest.cc b/modules/audio_coding/neteq/neteq_unittest.cc
index 42dbcf3..dff4568 100644
--- a/modules/audio_coding/neteq/neteq_unittest.cc
+++ b/modules/audio_coding/neteq/neteq_unittest.cc
@@ -463,18 +463,18 @@
webrtc::test::ResourcePath("audio_coding/neteq_universal_new", "rtp");
const std::string output_checksum = PlatformChecksum(
- "09fa7646e2ad032a0b156177b95f09012430f81f",
- "1c64eb8b55ce8878676c6a1e6ddd78f48de0668b",
+ "0c6dc227f781c81a229970f8fceda1a012498cba",
+ "15c4a2202877a414515e218bdb7992f0ad53e5af",
"not used",
- "09fa7646e2ad032a0b156177b95f09012430f81f",
- "759fef89a5de52bd17e733dc255c671ce86be909");
+ "0c6dc227f781c81a229970f8fceda1a012498cba",
+ "25fc4c863caa499aa447a5b8d059f5452cbcc500");
const std::string network_stats_checksum =
- PlatformChecksum("5b4262ca328e5f066af5d34f3380521583dd20de",
- "80235b6d727281203acb63b98f9a9e85d95f7ec0",
+ PlatformChecksum("4b2370f5c794741d2a46be5c7935c66ef3fb53e9",
+ "e339cb2adf5ab3dfc21cb7205d670a34751e8336",
"not used",
- "5b4262ca328e5f066af5d34f3380521583dd20de",
- "5b4262ca328e5f066af5d34f3380521583dd20de");
+ "4b2370f5c794741d2a46be5c7935c66ef3fb53e9",
+ "4b2370f5c794741d2a46be5c7935c66ef3fb53e9");
const std::string rtcp_stats_checksum = PlatformChecksum(
"b8880bf9fed2487efbddcb8d94b9937a29ae521d",
@@ -502,18 +502,18 @@
webrtc::test::ResourcePath("audio_coding/neteq_opus", "rtp");
const std::string output_checksum = PlatformChecksum(
- "e0e4063d55941792a725fde1f781ebbbe83b8165",
- "b77d2f25db1e59e054160a89e7defc7bfdb12f31",
- "a8a525c476a922456a35b5b13861b55cd8c9c7b8",
- "e0e4063d55941792a725fde1f781ebbbe83b8165",
- "e0e4063d55941792a725fde1f781ebbbe83b8165");
+ "14a63b3c7b925c82296be4bafc71bec85f2915c2",
+ "b7b7ed802b0e18ee416973bf3b9ae98599b0181d",
+ "5876e52dda90d5ca433c3726555b907b97c86374",
+ "14a63b3c7b925c82296be4bafc71bec85f2915c2",
+ "14a63b3c7b925c82296be4bafc71bec85f2915c2");
const std::string network_stats_checksum =
- PlatformChecksum("9e72233c78baf685e500dd6c94212b30a4c5f27d",
- "c237d7ca04cbb1ea2e3b27a7c8963015deb985e7",
- "4f1e9734bc80a290faaf9d611efcb8d7802dbc4f",
- "9e72233c78baf685e500dd6c94212b30a4c5f27d",
- "9e72233c78baf685e500dd6c94212b30a4c5f27d");
+ PlatformChecksum("adb3272498e436d1c019cbfd71610e9510c54497",
+ "fa935a91abc7291db47428a2d7c5361b98713a92",
+ "42106aa5267300f709f63737707ef07afd9dac61",
+ "adb3272498e436d1c019cbfd71610e9510c54497",
+ "adb3272498e436d1c019cbfd71610e9510c54497");
const std::string rtcp_stats_checksum = PlatformChecksum(
"e37c797e3de6a64dda88c9ade7a013d022a2e1e0",