henrik.lundin@webrtc.org | d94659d | 2013-01-29 12:09:21 +0000 | [diff] [blame] | 1 | /* |
| 2 | * Copyright (c) 2012 The WebRTC project authors. All Rights Reserved. |
| 3 | * |
| 4 | * Use of this source code is governed by a BSD-style license |
| 5 | * that can be found in the LICENSE file in the root of the source |
| 6 | * tree. An additional intellectual property rights grant can be found |
| 7 | * in the file PATENTS. All contributing project authors may |
| 8 | * be found in the AUTHORS file in the root of the source tree. |
| 9 | */ |
| 10 | |
| 11 | // Unit tests for DelayManager class. |
| 12 | |
Mirko Bonadei | 92ea95e | 2017-09-15 06:47:31 +0200 | [diff] [blame] | 13 | #include "modules/audio_coding/neteq/delay_manager.h" |
henrik.lundin@webrtc.org | d94659d | 2013-01-29 12:09:21 +0000 | [diff] [blame] | 14 | |
| 15 | #include <math.h> |
| 16 | |
Mirko Bonadei | 317a1f0 | 2019-09-17 17:06:18 +0200 | [diff] [blame] | 17 | #include <memory> |
| 18 | |
Jakob Ivarsson | 7dff9f3 | 2020-11-11 15:26:10 +0100 | [diff] [blame^] | 19 | #include "absl/types/optional.h" |
Jakob Ivarsson | 1eb3d7e | 2019-02-21 15:42:31 +0100 | [diff] [blame] | 20 | #include "modules/audio_coding/neteq/histogram.h" |
Jakob Ivarsson | 1eb3d7e | 2019-02-21 15:42:31 +0100 | [diff] [blame] | 21 | #include "modules/audio_coding/neteq/mock/mock_histogram.h" |
Jakob Ivarsson | 4450708 | 2019-03-05 16:59:03 +0100 | [diff] [blame] | 22 | #include "modules/audio_coding/neteq/mock/mock_statistics_calculator.h" |
Ruslan Burakov | 4a68fb9 | 2019-02-13 14:25:39 +0100 | [diff] [blame] | 23 | #include "rtc_base/checks.h" |
Minyue Li | 002fbb8 | 2018-10-04 11:31:03 +0200 | [diff] [blame] | 24 | #include "test/field_trial.h" |
Mirko Bonadei | 92ea95e | 2017-09-15 06:47:31 +0200 | [diff] [blame] | 25 | #include "test/gmock.h" |
| 26 | #include "test/gtest.h" |
henrik.lundin@webrtc.org | d94659d | 2013-01-29 12:09:21 +0000 | [diff] [blame] | 27 | |
| 28 | namespace webrtc { |
| 29 | |
Jakob Ivarsson | db42ed2 | 2019-02-27 10:08:09 +0100 | [diff] [blame] | 30 | namespace { |
| 31 | constexpr int kMaxNumberOfPackets = 240; |
| 32 | constexpr int kMinDelayMs = 0; |
Jakob Ivarsson | 7dff9f3 | 2020-11-11 15:26:10 +0100 | [diff] [blame^] | 33 | constexpr int kMaxHistoryMs = 2000; |
Jakob Ivarsson | db42ed2 | 2019-02-27 10:08:09 +0100 | [diff] [blame] | 34 | constexpr int kTimeStepMs = 10; |
| 35 | constexpr int kFs = 8000; |
| 36 | constexpr int kFrameSizeMs = 20; |
| 37 | constexpr int kTsIncrement = kFrameSizeMs * kFs / 1000; |
| 38 | constexpr int kMaxBufferSizeMs = kMaxNumberOfPackets * kFrameSizeMs; |
| 39 | constexpr int kDefaultHistogramQuantile = 1020054733; |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 40 | constexpr int kNumBuckets = 100; |
Jakob Ivarsson | db42ed2 | 2019-02-27 10:08:09 +0100 | [diff] [blame] | 41 | constexpr int kForgetFactor = 32745; |
| 42 | } // namespace |
| 43 | |
henrik.lundin@webrtc.org | d94659d | 2013-01-29 12:09:21 +0000 | [diff] [blame] | 44 | class DelayManagerTest : public ::testing::Test { |
| 45 | protected: |
henrik.lundin@webrtc.org | d94659d | 2013-01-29 12:09:21 +0000 | [diff] [blame] | 46 | DelayManagerTest(); |
| 47 | virtual void SetUp(); |
Minyue Li | 002fbb8 | 2018-10-04 11:31:03 +0200 | [diff] [blame] | 48 | void RecreateDelayManager(); |
Ivo Creusen | 53a31f7 | 2019-10-24 15:20:39 +0200 | [diff] [blame] | 49 | absl::optional<int> InsertNextPacket(); |
henrik.lundin@webrtc.org | d94659d | 2013-01-29 12:09:21 +0000 | [diff] [blame] | 50 | void IncreaseTime(int inc_ms); |
| 51 | |
Minyue Li | 002fbb8 | 2018-10-04 11:31:03 +0200 | [diff] [blame] | 52 | std::unique_ptr<DelayManager> dm_; |
henrik.lundin | f393370 | 2016-04-28 01:53:52 -0700 | [diff] [blame] | 53 | TickTimer tick_timer_; |
Jakob Ivarsson | 4450708 | 2019-03-05 16:59:03 +0100 | [diff] [blame] | 54 | MockStatisticsCalculator stats_; |
Jakob Ivarsson | 1eb3d7e | 2019-02-21 15:42:31 +0100 | [diff] [blame] | 55 | MockHistogram* mock_histogram_; |
henrik.lundin@webrtc.org | d94659d | 2013-01-29 12:09:21 +0000 | [diff] [blame] | 56 | uint32_t ts_; |
Jakob Ivarsson | db42ed2 | 2019-02-27 10:08:09 +0100 | [diff] [blame] | 57 | bool use_mock_histogram_ = false; |
Jakob Ivarsson | 7dff9f3 | 2020-11-11 15:26:10 +0100 | [diff] [blame^] | 58 | absl::optional<int> resample_interval_ms_; |
henrik.lundin@webrtc.org | d94659d | 2013-01-29 12:09:21 +0000 | [diff] [blame] | 59 | }; |
| 60 | |
| 61 | DelayManagerTest::DelayManagerTest() |
Jakob Ivarsson | 39b934b | 2019-01-10 10:28:23 +0100 | [diff] [blame] | 62 | : dm_(nullptr), |
Jakob Ivarsson | 39b934b | 2019-01-10 10:28:23 +0100 | [diff] [blame] | 63 | ts_(0x12345678) {} |
henrik.lundin@webrtc.org | d94659d | 2013-01-29 12:09:21 +0000 | [diff] [blame] | 64 | |
| 65 | void DelayManagerTest::SetUp() { |
Minyue Li | 002fbb8 | 2018-10-04 11:31:03 +0200 | [diff] [blame] | 66 | RecreateDelayManager(); |
| 67 | } |
| 68 | |
| 69 | void DelayManagerTest::RecreateDelayManager() { |
Jakob Ivarsson | 1eb3d7e | 2019-02-21 15:42:31 +0100 | [diff] [blame] | 70 | if (use_mock_histogram_) { |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 71 | mock_histogram_ = new MockHistogram(kNumBuckets, kForgetFactor); |
Jakob Ivarsson | db42ed2 | 2019-02-27 10:08:09 +0100 | [diff] [blame] | 72 | std::unique_ptr<Histogram> histogram(mock_histogram_); |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 73 | dm_ = std::make_unique<DelayManager>(kMaxNumberOfPackets, kMinDelayMs, |
| 74 | kDefaultHistogramQuantile, |
Jakob Ivarsson | 7dff9f3 | 2020-11-11 15:26:10 +0100 | [diff] [blame^] | 75 | resample_interval_ms_, kMaxHistoryMs, |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 76 | &tick_timer_, std::move(histogram)); |
Jakob Ivarsson | 1eb3d7e | 2019-02-21 15:42:31 +0100 | [diff] [blame] | 77 | } else { |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 78 | dm_ = DelayManager::Create(kMaxNumberOfPackets, kMinDelayMs, &tick_timer_); |
Jakob Ivarsson | 1eb3d7e | 2019-02-21 15:42:31 +0100 | [diff] [blame] | 79 | } |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 80 | dm_->SetPacketAudioLength(kFrameSizeMs); |
henrik.lundin@webrtc.org | d94659d | 2013-01-29 12:09:21 +0000 | [diff] [blame] | 81 | } |
| 82 | |
Ivo Creusen | 53a31f7 | 2019-10-24 15:20:39 +0200 | [diff] [blame] | 83 | absl::optional<int> DelayManagerTest::InsertNextPacket() { |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 84 | auto relative_delay = dm_->Update(ts_, kFs); |
henrik.lundin@webrtc.org | d94659d | 2013-01-29 12:09:21 +0000 | [diff] [blame] | 85 | ts_ += kTsIncrement; |
Ivo Creusen | 53a31f7 | 2019-10-24 15:20:39 +0200 | [diff] [blame] | 86 | return relative_delay; |
henrik.lundin@webrtc.org | d94659d | 2013-01-29 12:09:21 +0000 | [diff] [blame] | 87 | } |
| 88 | |
| 89 | void DelayManagerTest::IncreaseTime(int inc_ms) { |
| 90 | for (int t = 0; t < inc_ms; t += kTimeStepMs) { |
henrik.lundin | f393370 | 2016-04-28 01:53:52 -0700 | [diff] [blame] | 91 | tick_timer_.Increment(); |
henrik.lundin@webrtc.org | d94659d | 2013-01-29 12:09:21 +0000 | [diff] [blame] | 92 | } |
| 93 | } |
Ruslan Burakov | 1e193fa | 2019-05-15 14:31:22 +0200 | [diff] [blame] | 94 | |
henrik.lundin@webrtc.org | d94659d | 2013-01-29 12:09:21 +0000 | [diff] [blame] | 95 | TEST_F(DelayManagerTest, CreateAndDestroy) { |
| 96 | // Nothing to do here. The test fixture creates and destroys the DelayManager |
| 97 | // object. |
| 98 | } |
| 99 | |
henrik.lundin@webrtc.org | d94659d | 2013-01-29 12:09:21 +0000 | [diff] [blame] | 100 | TEST_F(DelayManagerTest, UpdateNormal) { |
henrik.lundin@webrtc.org | d94659d | 2013-01-29 12:09:21 +0000 | [diff] [blame] | 101 | // First packet arrival. |
| 102 | InsertNextPacket(); |
| 103 | // Advance time by one frame size. |
| 104 | IncreaseTime(kFrameSizeMs); |
| 105 | // Second packet arrival. |
henrik.lundin@webrtc.org | d94659d | 2013-01-29 12:09:21 +0000 | [diff] [blame] | 106 | InsertNextPacket(); |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 107 | EXPECT_EQ(20, dm_->TargetDelayMs()); |
henrik.lundin@webrtc.org | d94659d | 2013-01-29 12:09:21 +0000 | [diff] [blame] | 108 | } |
| 109 | |
| 110 | TEST_F(DelayManagerTest, UpdateLongInterArrivalTime) { |
henrik.lundin@webrtc.org | d94659d | 2013-01-29 12:09:21 +0000 | [diff] [blame] | 111 | // First packet arrival. |
| 112 | InsertNextPacket(); |
| 113 | // Advance time by two frame size. |
| 114 | IncreaseTime(2 * kFrameSizeMs); |
| 115 | // Second packet arrival. |
henrik.lundin@webrtc.org | d94659d | 2013-01-29 12:09:21 +0000 | [diff] [blame] | 116 | InsertNextPacket(); |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 117 | EXPECT_EQ(40, dm_->TargetDelayMs()); |
henrik.lundin@webrtc.org | d94659d | 2013-01-29 12:09:21 +0000 | [diff] [blame] | 118 | } |
| 119 | |
Niels Möller | 18f1adc | 2018-08-23 08:40:41 +0200 | [diff] [blame] | 120 | TEST_F(DelayManagerTest, MaxDelay) { |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 121 | const int kExpectedTarget = 5 * kFrameSizeMs; |
turaj@webrtc.org | f1efc57 | 2013-08-16 23:44:24 +0000 | [diff] [blame] | 122 | // First packet arrival. |
| 123 | InsertNextPacket(); |
| 124 | // Second packet arrival. |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 125 | IncreaseTime(kExpectedTarget); |
turaj@webrtc.org | f1efc57 | 2013-08-16 23:44:24 +0000 | [diff] [blame] | 126 | InsertNextPacket(); |
| 127 | |
| 128 | // No limit is set. |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 129 | EXPECT_EQ(kExpectedTarget, dm_->TargetDelayMs()); |
turaj@webrtc.org | f1efc57 | 2013-08-16 23:44:24 +0000 | [diff] [blame] | 130 | |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 131 | const int kMaxDelayMs = 3 * kFrameSizeMs; |
turaj@webrtc.org | f1efc57 | 2013-08-16 23:44:24 +0000 | [diff] [blame] | 132 | EXPECT_TRUE(dm_->SetMaximumDelay(kMaxDelayMs)); |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 133 | IncreaseTime(kFrameSizeMs); |
turaj@webrtc.org | f1efc57 | 2013-08-16 23:44:24 +0000 | [diff] [blame] | 134 | InsertNextPacket(); |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 135 | EXPECT_EQ(kMaxDelayMs, dm_->TargetDelayMs()); |
turaj@webrtc.org | f1efc57 | 2013-08-16 23:44:24 +0000 | [diff] [blame] | 136 | |
| 137 | // Target level at least should be one packet. |
| 138 | EXPECT_FALSE(dm_->SetMaximumDelay(kFrameSizeMs - 1)); |
| 139 | } |
| 140 | |
Niels Möller | 18f1adc | 2018-08-23 08:40:41 +0200 | [diff] [blame] | 141 | TEST_F(DelayManagerTest, MinDelay) { |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 142 | const int kExpectedTarget = 5 * kFrameSizeMs; |
turaj@webrtc.org | f1efc57 | 2013-08-16 23:44:24 +0000 | [diff] [blame] | 143 | // First packet arrival. |
| 144 | InsertNextPacket(); |
| 145 | // Second packet arrival. |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 146 | IncreaseTime(kExpectedTarget); |
turaj@webrtc.org | f1efc57 | 2013-08-16 23:44:24 +0000 | [diff] [blame] | 147 | InsertNextPacket(); |
| 148 | |
| 149 | // No limit is applied. |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 150 | EXPECT_EQ(kExpectedTarget, dm_->TargetDelayMs()); |
turaj@webrtc.org | f1efc57 | 2013-08-16 23:44:24 +0000 | [diff] [blame] | 151 | |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 152 | int kMinDelayMs = 7 * kFrameSizeMs; |
turaj@webrtc.org | f1efc57 | 2013-08-16 23:44:24 +0000 | [diff] [blame] | 153 | dm_->SetMinimumDelay(kMinDelayMs); |
Jakob Ivarsson | 507f434 | 2019-09-03 13:04:41 +0200 | [diff] [blame] | 154 | IncreaseTime(kFrameSizeMs); |
turaj@webrtc.org | f1efc57 | 2013-08-16 23:44:24 +0000 | [diff] [blame] | 155 | InsertNextPacket(); |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 156 | EXPECT_EQ(kMinDelayMs, dm_->TargetDelayMs()); |
turaj@webrtc.org | f1efc57 | 2013-08-16 23:44:24 +0000 | [diff] [blame] | 157 | } |
| 158 | |
Ruslan Burakov | 4a68fb9 | 2019-02-13 14:25:39 +0100 | [diff] [blame] | 159 | TEST_F(DelayManagerTest, BaseMinimumDelayCheckValidRange) { |
Ruslan Burakov | 4a68fb9 | 2019-02-13 14:25:39 +0100 | [diff] [blame] | 160 | // Base minimum delay should be between [0, 10000] milliseconds. |
| 161 | EXPECT_FALSE(dm_->SetBaseMinimumDelay(-1)); |
| 162 | EXPECT_FALSE(dm_->SetBaseMinimumDelay(10001)); |
| 163 | EXPECT_EQ(dm_->GetBaseMinimumDelay(), 0); |
| 164 | |
| 165 | EXPECT_TRUE(dm_->SetBaseMinimumDelay(7999)); |
| 166 | EXPECT_EQ(dm_->GetBaseMinimumDelay(), 7999); |
| 167 | } |
| 168 | |
| 169 | TEST_F(DelayManagerTest, BaseMinimumDelayLowerThanMinimumDelay) { |
Ruslan Burakov | 4a68fb9 | 2019-02-13 14:25:39 +0100 | [diff] [blame] | 170 | constexpr int kBaseMinimumDelayMs = 100; |
| 171 | constexpr int kMinimumDelayMs = 200; |
| 172 | |
| 173 | // Base minimum delay sets lower bound on minimum. That is why when base |
| 174 | // minimum delay is lower than minimum delay we use minimum delay. |
| 175 | RTC_DCHECK_LT(kBaseMinimumDelayMs, kMinimumDelayMs); |
| 176 | |
| 177 | EXPECT_TRUE(dm_->SetBaseMinimumDelay(kBaseMinimumDelayMs)); |
| 178 | EXPECT_TRUE(dm_->SetMinimumDelay(kMinimumDelayMs)); |
| 179 | EXPECT_EQ(dm_->effective_minimum_delay_ms_for_test(), kMinimumDelayMs); |
| 180 | } |
| 181 | |
| 182 | TEST_F(DelayManagerTest, BaseMinimumDelayGreaterThanMinimumDelay) { |
Ruslan Burakov | 4a68fb9 | 2019-02-13 14:25:39 +0100 | [diff] [blame] | 183 | constexpr int kBaseMinimumDelayMs = 70; |
| 184 | constexpr int kMinimumDelayMs = 30; |
| 185 | |
| 186 | // Base minimum delay sets lower bound on minimum. That is why when base |
| 187 | // minimum delay is greater than minimum delay we use base minimum delay. |
| 188 | RTC_DCHECK_GT(kBaseMinimumDelayMs, kMinimumDelayMs); |
| 189 | |
| 190 | EXPECT_TRUE(dm_->SetBaseMinimumDelay(kBaseMinimumDelayMs)); |
| 191 | EXPECT_TRUE(dm_->SetMinimumDelay(kMinimumDelayMs)); |
| 192 | EXPECT_EQ(dm_->effective_minimum_delay_ms_for_test(), kBaseMinimumDelayMs); |
| 193 | } |
| 194 | |
| 195 | TEST_F(DelayManagerTest, BaseMinimumDelayGreaterThanBufferSize) { |
Ruslan Burakov | 4a68fb9 | 2019-02-13 14:25:39 +0100 | [diff] [blame] | 196 | constexpr int kBaseMinimumDelayMs = kMaxBufferSizeMs + 1; |
| 197 | constexpr int kMinimumDelayMs = 12; |
Ruslan Burakov | b35bacc | 2019-02-20 13:41:59 +0100 | [diff] [blame] | 198 | constexpr int kMaximumDelayMs = 20; |
Ruslan Burakov | 4a68fb9 | 2019-02-13 14:25:39 +0100 | [diff] [blame] | 199 | constexpr int kMaxBufferSizeMsQ75 = 3 * kMaxBufferSizeMs / 4; |
| 200 | |
Ruslan Burakov | b35bacc | 2019-02-20 13:41:59 +0100 | [diff] [blame] | 201 | EXPECT_TRUE(dm_->SetMaximumDelay(kMaximumDelayMs)); |
| 202 | |
Ruslan Burakov | 4a68fb9 | 2019-02-13 14:25:39 +0100 | [diff] [blame] | 203 | // Base minimum delay is greater than minimum delay, that is why we clamp |
| 204 | // it to current the highest possible value which is maximum delay. |
| 205 | RTC_DCHECK_GT(kBaseMinimumDelayMs, kMinimumDelayMs); |
| 206 | RTC_DCHECK_GT(kBaseMinimumDelayMs, kMaxBufferSizeMs); |
Ruslan Burakov | b35bacc | 2019-02-20 13:41:59 +0100 | [diff] [blame] | 207 | RTC_DCHECK_GT(kBaseMinimumDelayMs, kMaximumDelayMs); |
| 208 | RTC_DCHECK_LT(kMaximumDelayMs, kMaxBufferSizeMsQ75); |
Ruslan Burakov | 4a68fb9 | 2019-02-13 14:25:39 +0100 | [diff] [blame] | 209 | |
| 210 | EXPECT_TRUE(dm_->SetMinimumDelay(kMinimumDelayMs)); |
| 211 | EXPECT_TRUE(dm_->SetBaseMinimumDelay(kBaseMinimumDelayMs)); |
| 212 | |
| 213 | // Unset maximum value. |
| 214 | EXPECT_TRUE(dm_->SetMaximumDelay(0)); |
| 215 | |
| 216 | // With maximum value unset, the highest possible value now is 75% of |
| 217 | // currently possible maximum buffer size. |
| 218 | EXPECT_EQ(dm_->effective_minimum_delay_ms_for_test(), kMaxBufferSizeMsQ75); |
| 219 | } |
| 220 | |
| 221 | TEST_F(DelayManagerTest, BaseMinimumDelayGreaterThanMaximumDelay) { |
Ruslan Burakov | 4a68fb9 | 2019-02-13 14:25:39 +0100 | [diff] [blame] | 222 | constexpr int kMaximumDelayMs = 400; |
| 223 | constexpr int kBaseMinimumDelayMs = kMaximumDelayMs + 1; |
| 224 | constexpr int kMinimumDelayMs = 20; |
| 225 | |
| 226 | // Base minimum delay is greater than minimum delay, that is why we clamp |
| 227 | // it to current the highest possible value which is kMaximumDelayMs. |
| 228 | RTC_DCHECK_GT(kBaseMinimumDelayMs, kMinimumDelayMs); |
| 229 | RTC_DCHECK_GT(kBaseMinimumDelayMs, kMaximumDelayMs); |
| 230 | RTC_DCHECK_LT(kMaximumDelayMs, kMaxBufferSizeMs); |
| 231 | |
| 232 | EXPECT_TRUE(dm_->SetMaximumDelay(kMaximumDelayMs)); |
| 233 | EXPECT_TRUE(dm_->SetMinimumDelay(kMinimumDelayMs)); |
| 234 | EXPECT_TRUE(dm_->SetBaseMinimumDelay(kBaseMinimumDelayMs)); |
| 235 | EXPECT_EQ(dm_->effective_minimum_delay_ms_for_test(), kMaximumDelayMs); |
| 236 | } |
| 237 | |
| 238 | TEST_F(DelayManagerTest, BaseMinimumDelayLowerThanMaxSize) { |
Ruslan Burakov | 4a68fb9 | 2019-02-13 14:25:39 +0100 | [diff] [blame] | 239 | constexpr int kMaximumDelayMs = 400; |
| 240 | constexpr int kBaseMinimumDelayMs = kMaximumDelayMs - 1; |
| 241 | constexpr int kMinimumDelayMs = 20; |
| 242 | |
| 243 | // Base minimum delay is greater than minimum delay, and lower than maximum |
| 244 | // delays that is why it is used. |
| 245 | RTC_DCHECK_GT(kBaseMinimumDelayMs, kMinimumDelayMs); |
| 246 | RTC_DCHECK_LT(kBaseMinimumDelayMs, kMaximumDelayMs); |
| 247 | |
| 248 | EXPECT_TRUE(dm_->SetMaximumDelay(kMaximumDelayMs)); |
| 249 | EXPECT_TRUE(dm_->SetMinimumDelay(kMinimumDelayMs)); |
| 250 | EXPECT_TRUE(dm_->SetBaseMinimumDelay(kBaseMinimumDelayMs)); |
| 251 | EXPECT_EQ(dm_->effective_minimum_delay_ms_for_test(), kBaseMinimumDelayMs); |
| 252 | } |
| 253 | |
| 254 | TEST_F(DelayManagerTest, MinimumDelayMemorization) { |
| 255 | // Check that when we increase base minimum delay to value higher than |
| 256 | // minimum delay then minimum delay is still memorized. This allows to |
| 257 | // restore effective minimum delay to memorized minimum delay value when we |
| 258 | // decrease base minimum delay. |
Ruslan Burakov | 4a68fb9 | 2019-02-13 14:25:39 +0100 | [diff] [blame] | 259 | constexpr int kBaseMinimumDelayMsLow = 10; |
| 260 | constexpr int kMinimumDelayMs = 20; |
| 261 | constexpr int kBaseMinimumDelayMsHigh = 30; |
| 262 | |
| 263 | EXPECT_TRUE(dm_->SetBaseMinimumDelay(kBaseMinimumDelayMsLow)); |
| 264 | EXPECT_TRUE(dm_->SetMinimumDelay(kMinimumDelayMs)); |
| 265 | // Minimum delay is used as it is higher than base minimum delay. |
| 266 | EXPECT_EQ(dm_->effective_minimum_delay_ms_for_test(), kMinimumDelayMs); |
| 267 | |
| 268 | EXPECT_TRUE(dm_->SetBaseMinimumDelay(kBaseMinimumDelayMsHigh)); |
| 269 | // Base minimum delay is used as it is now higher than minimum delay. |
| 270 | EXPECT_EQ(dm_->effective_minimum_delay_ms_for_test(), |
| 271 | kBaseMinimumDelayMsHigh); |
| 272 | |
| 273 | EXPECT_TRUE(dm_->SetBaseMinimumDelay(kBaseMinimumDelayMsLow)); |
| 274 | // Check that minimum delay is memorized and is used again. |
| 275 | EXPECT_EQ(dm_->effective_minimum_delay_ms_for_test(), kMinimumDelayMs); |
| 276 | } |
| 277 | |
| 278 | TEST_F(DelayManagerTest, BaseMinimumDelay) { |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 279 | const int kExpectedTarget = 5 * kFrameSizeMs; |
Jakob Ivarsson | b1ae5cc | 2020-10-06 15:37:28 +0000 | [diff] [blame] | 280 | // First packet arrival. |
| 281 | InsertNextPacket(); |
| 282 | // Second packet arrival. |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 283 | IncreaseTime(kExpectedTarget); |
Jakob Ivarsson | b1ae5cc | 2020-10-06 15:37:28 +0000 | [diff] [blame] | 284 | InsertNextPacket(); |
| 285 | |
| 286 | // No limit is applied. |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 287 | EXPECT_EQ(kExpectedTarget, dm_->TargetDelayMs()); |
Jakob Ivarsson | ff9f646 | 2020-10-07 12:46:42 +0000 | [diff] [blame] | 288 | |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 289 | constexpr int kBaseMinimumDelayMs = 7 * kFrameSizeMs; |
Jakob Ivarsson | ff9f646 | 2020-10-07 12:46:42 +0000 | [diff] [blame] | 290 | EXPECT_TRUE(dm_->SetBaseMinimumDelay(kBaseMinimumDelayMs)); |
| 291 | EXPECT_EQ(dm_->GetBaseMinimumDelay(), kBaseMinimumDelayMs); |
| 292 | |
| 293 | IncreaseTime(kFrameSizeMs); |
| 294 | InsertNextPacket(); |
| 295 | EXPECT_EQ(dm_->GetBaseMinimumDelay(), kBaseMinimumDelayMs); |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 296 | EXPECT_EQ(kBaseMinimumDelayMs, dm_->TargetDelayMs()); |
Jakob Ivarsson | ff9f646 | 2020-10-07 12:46:42 +0000 | [diff] [blame] | 297 | } |
| 298 | |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 299 | TEST_F(DelayManagerTest, BaseMinimumDelayAffectsTargetDelay) { |
Jakob Ivarsson | ff9f646 | 2020-10-07 12:46:42 +0000 | [diff] [blame] | 300 | const int kExpectedTarget = 5; |
| 301 | const int kTimeIncrement = kExpectedTarget * kFrameSizeMs; |
Jakob Ivarsson | ff9f646 | 2020-10-07 12:46:42 +0000 | [diff] [blame] | 302 | // First packet arrival. |
| 303 | InsertNextPacket(); |
| 304 | // Second packet arrival. |
| 305 | IncreaseTime(kTimeIncrement); |
| 306 | InsertNextPacket(); |
| 307 | |
| 308 | // No limit is applied. |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 309 | EXPECT_EQ(kTimeIncrement, dm_->TargetDelayMs()); |
Ruslan Burakov | 4a68fb9 | 2019-02-13 14:25:39 +0100 | [diff] [blame] | 310 | |
| 311 | // Minimum delay is lower than base minimum delay, that is why base minimum |
| 312 | // delay is used to calculate target level. |
| 313 | constexpr int kMinimumDelayPackets = kExpectedTarget + 1; |
| 314 | constexpr int kBaseMinimumDelayPackets = kExpectedTarget + 2; |
| 315 | |
| 316 | constexpr int kMinimumDelayMs = kMinimumDelayPackets * kFrameSizeMs; |
| 317 | constexpr int kBaseMinimumDelayMs = kBaseMinimumDelayPackets * kFrameSizeMs; |
| 318 | |
| 319 | EXPECT_TRUE(kMinimumDelayMs < kBaseMinimumDelayMs); |
| 320 | EXPECT_TRUE(dm_->SetMinimumDelay(kMinimumDelayMs)); |
| 321 | EXPECT_TRUE(dm_->SetBaseMinimumDelay(kBaseMinimumDelayMs)); |
| 322 | EXPECT_EQ(dm_->GetBaseMinimumDelay(), kBaseMinimumDelayMs); |
| 323 | |
Jakob Ivarsson | 507f434 | 2019-09-03 13:04:41 +0200 | [diff] [blame] | 324 | IncreaseTime(kFrameSizeMs); |
Ruslan Burakov | 4a68fb9 | 2019-02-13 14:25:39 +0100 | [diff] [blame] | 325 | InsertNextPacket(); |
| 326 | EXPECT_EQ(dm_->GetBaseMinimumDelay(), kBaseMinimumDelayMs); |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 327 | EXPECT_EQ(kBaseMinimumDelayMs, dm_->TargetDelayMs()); |
henrik.lundin | b8c55b1 | 2017-05-10 07:38:01 -0700 | [diff] [blame] | 328 | } |
| 329 | |
henrik.lundin@webrtc.org | d94659d | 2013-01-29 12:09:21 +0000 | [diff] [blame] | 330 | TEST_F(DelayManagerTest, Failures) { |
| 331 | // Wrong sample rate. |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 332 | EXPECT_EQ(absl::nullopt, dm_->Update(0, -1)); |
henrik.lundin@webrtc.org | d94659d | 2013-01-29 12:09:21 +0000 | [diff] [blame] | 333 | // Wrong packet size. |
| 334 | EXPECT_EQ(-1, dm_->SetPacketAudioLength(0)); |
| 335 | EXPECT_EQ(-1, dm_->SetPacketAudioLength(-1)); |
turaj@webrtc.org | f1efc57 | 2013-08-16 23:44:24 +0000 | [diff] [blame] | 336 | |
| 337 | // Minimum delay higher than a maximum delay is not accepted. |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 338 | EXPECT_TRUE(dm_->SetMaximumDelay(20)); |
| 339 | EXPECT_FALSE(dm_->SetMinimumDelay(40)); |
turaj@webrtc.org | f1efc57 | 2013-08-16 23:44:24 +0000 | [diff] [blame] | 340 | |
| 341 | // Maximum delay less than minimum delay is not accepted. |
| 342 | EXPECT_TRUE(dm_->SetMaximumDelay(100)); |
| 343 | EXPECT_TRUE(dm_->SetMinimumDelay(80)); |
| 344 | EXPECT_FALSE(dm_->SetMaximumDelay(60)); |
henrik.lundin@webrtc.org | d94659d | 2013-01-29 12:09:21 +0000 | [diff] [blame] | 345 | } |
| 346 | |
Jakob Ivarsson | db42ed2 | 2019-02-27 10:08:09 +0100 | [diff] [blame] | 347 | TEST_F(DelayManagerTest, DelayHistogramFieldTrial) { |
| 348 | { |
| 349 | test::ScopedFieldTrials field_trial( |
| 350 | "WebRTC-Audio-NetEqDelayHistogram/Enabled-96-0.998/"); |
| 351 | RecreateDelayManager(); |
Jakob Ivarsson | db42ed2 | 2019-02-27 10:08:09 +0100 | [diff] [blame] | 352 | EXPECT_EQ(1030792151, dm_->histogram_quantile()); // 0.96 in Q30. |
Jakob Ivarsson | e9a2ee2 | 2019-05-22 16:54:09 +0200 | [diff] [blame] | 353 | EXPECT_EQ( |
| 354 | 32702, |
| 355 | dm_->histogram()->base_forget_factor_for_testing()); // 0.998 in Q15. |
| 356 | EXPECT_FALSE(dm_->histogram()->start_forget_weight_for_testing()); |
Jakob Ivarsson | db42ed2 | 2019-02-27 10:08:09 +0100 | [diff] [blame] | 357 | } |
| 358 | { |
| 359 | test::ScopedFieldTrials field_trial( |
| 360 | "WebRTC-Audio-NetEqDelayHistogram/Enabled-97.5-0.998/"); |
| 361 | RecreateDelayManager(); |
Jakob Ivarsson | db42ed2 | 2019-02-27 10:08:09 +0100 | [diff] [blame] | 362 | EXPECT_EQ(1046898278, dm_->histogram_quantile()); // 0.975 in Q30. |
Jakob Ivarsson | e9a2ee2 | 2019-05-22 16:54:09 +0200 | [diff] [blame] | 363 | EXPECT_EQ( |
| 364 | 32702, |
| 365 | dm_->histogram()->base_forget_factor_for_testing()); // 0.998 in Q15. |
| 366 | EXPECT_FALSE(dm_->histogram()->start_forget_weight_for_testing()); |
Jakob Ivarsson | db42ed2 | 2019-02-27 10:08:09 +0100 | [diff] [blame] | 367 | } |
Jakob Ivarsson | e9a2ee2 | 2019-05-22 16:54:09 +0200 | [diff] [blame] | 368 | // Test parameter for new call start adaptation. |
| 369 | { |
| 370 | test::ScopedFieldTrials field_trial( |
| 371 | "WebRTC-Audio-NetEqDelayHistogram/Enabled-96-0.998-1/"); |
| 372 | RecreateDelayManager(); |
| 373 | EXPECT_EQ(dm_->histogram()->start_forget_weight_for_testing().value(), 1.0); |
| 374 | } |
| 375 | { |
| 376 | test::ScopedFieldTrials field_trial( |
| 377 | "WebRTC-Audio-NetEqDelayHistogram/Enabled-96-0.998-1.5/"); |
| 378 | RecreateDelayManager(); |
| 379 | EXPECT_EQ(dm_->histogram()->start_forget_weight_for_testing().value(), 1.5); |
| 380 | } |
| 381 | { |
| 382 | test::ScopedFieldTrials field_trial( |
| 383 | "WebRTC-Audio-NetEqDelayHistogram/Enabled-96-0.998-0.5/"); |
| 384 | RecreateDelayManager(); |
| 385 | EXPECT_FALSE(dm_->histogram()->start_forget_weight_for_testing()); |
Jakob Ivarsson | db42ed2 | 2019-02-27 10:08:09 +0100 | [diff] [blame] | 386 | } |
| 387 | } |
| 388 | |
Jakob Ivarsson | bd5874a | 2020-01-07 17:07:40 +0100 | [diff] [blame] | 389 | TEST_F(DelayManagerTest, RelativeArrivalDelay) { |
Jakob Ivarsson | db42ed2 | 2019-02-27 10:08:09 +0100 | [diff] [blame] | 390 | use_mock_histogram_ = true; |
| 391 | RecreateDelayManager(); |
| 392 | |
Jakob Ivarsson | db42ed2 | 2019-02-27 10:08:09 +0100 | [diff] [blame] | 393 | InsertNextPacket(); |
| 394 | |
| 395 | IncreaseTime(kFrameSizeMs); |
| 396 | EXPECT_CALL(*mock_histogram_, Add(0)); // Not delayed. |
| 397 | InsertNextPacket(); |
| 398 | |
| 399 | IncreaseTime(2 * kFrameSizeMs); |
| 400 | EXPECT_CALL(*mock_histogram_, Add(1)); // 20ms delayed. |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 401 | dm_->Update(ts_, kFs); |
Jakob Ivarsson | db42ed2 | 2019-02-27 10:08:09 +0100 | [diff] [blame] | 402 | |
| 403 | IncreaseTime(2 * kFrameSizeMs); |
| 404 | EXPECT_CALL(*mock_histogram_, Add(2)); // 40ms delayed. |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 405 | dm_->Update(ts_ + kTsIncrement, kFs); |
Jakob Ivarsson | db42ed2 | 2019-02-27 10:08:09 +0100 | [diff] [blame] | 406 | |
| 407 | EXPECT_CALL(*mock_histogram_, Add(1)); // Reordered, 20ms delayed. |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 408 | dm_->Update(ts_, kFs); |
| 409 | } |
| 410 | |
| 411 | TEST_F(DelayManagerTest, ReorderedPackets) { |
| 412 | use_mock_histogram_ = true; |
| 413 | RecreateDelayManager(); |
| 414 | |
| 415 | // Insert first packet. |
| 416 | InsertNextPacket(); |
| 417 | |
| 418 | // Insert reordered packet. |
| 419 | EXPECT_CALL(*mock_histogram_, Add(4)); |
| 420 | dm_->Update(ts_ - 5 * kTsIncrement, kFs); |
| 421 | |
| 422 | // Insert another reordered packet. |
| 423 | EXPECT_CALL(*mock_histogram_, Add(1)); |
| 424 | dm_->Update(ts_ - 2 * kTsIncrement, kFs); |
| 425 | |
| 426 | // Insert the next packet in order and verify that the relative delay is |
| 427 | // estimated based on the first inserted packet. |
| 428 | IncreaseTime(4 * kFrameSizeMs); |
| 429 | EXPECT_CALL(*mock_histogram_, Add(3)); |
| 430 | InsertNextPacket(); |
Jakob Ivarsson | db42ed2 | 2019-02-27 10:08:09 +0100 | [diff] [blame] | 431 | } |
| 432 | |
Jakob Ivarsson | 74154e6 | 2019-08-22 15:00:16 +0200 | [diff] [blame] | 433 | TEST_F(DelayManagerTest, MaxDelayHistory) { |
Jakob Ivarsson | 74154e6 | 2019-08-22 15:00:16 +0200 | [diff] [blame] | 434 | use_mock_histogram_ = true; |
| 435 | RecreateDelayManager(); |
| 436 | |
Jakob Ivarsson | 74154e6 | 2019-08-22 15:00:16 +0200 | [diff] [blame] | 437 | InsertNextPacket(); |
| 438 | |
| 439 | // Insert 20 ms iat delay in the delay history. |
| 440 | IncreaseTime(2 * kFrameSizeMs); |
| 441 | EXPECT_CALL(*mock_histogram_, Add(1)); // 20ms delayed. |
| 442 | InsertNextPacket(); |
| 443 | |
| 444 | // Insert next packet with a timestamp difference larger than maximum history |
| 445 | // size. This removes the previously inserted iat delay from the history. |
| 446 | constexpr int kMaxHistoryMs = 2000; |
| 447 | IncreaseTime(kMaxHistoryMs + kFrameSizeMs); |
| 448 | ts_ += kFs * kMaxHistoryMs / 1000; |
| 449 | EXPECT_CALL(*mock_histogram_, Add(0)); // Not delayed. |
Jakob Ivarsson | 80fb978 | 2020-10-09 13:41:06 +0200 | [diff] [blame] | 450 | dm_->Update(ts_, kFs); |
Jakob Ivarsson | 74154e6 | 2019-08-22 15:00:16 +0200 | [diff] [blame] | 451 | } |
| 452 | |
Jakob Ivarsson | 4450708 | 2019-03-05 16:59:03 +0100 | [diff] [blame] | 453 | TEST_F(DelayManagerTest, RelativeArrivalDelayStatistic) { |
Ivo Creusen | 53a31f7 | 2019-10-24 15:20:39 +0200 | [diff] [blame] | 454 | EXPECT_EQ(absl::nullopt, InsertNextPacket()); |
Jakob Ivarsson | 4450708 | 2019-03-05 16:59:03 +0100 | [diff] [blame] | 455 | IncreaseTime(kFrameSizeMs); |
Ivo Creusen | 53a31f7 | 2019-10-24 15:20:39 +0200 | [diff] [blame] | 456 | EXPECT_EQ(0, InsertNextPacket()); |
Jakob Ivarsson | 4450708 | 2019-03-05 16:59:03 +0100 | [diff] [blame] | 457 | IncreaseTime(2 * kFrameSizeMs); |
Ivo Creusen | 53a31f7 | 2019-10-24 15:20:39 +0200 | [diff] [blame] | 458 | |
| 459 | EXPECT_EQ(20, InsertNextPacket()); |
Jakob Ivarsson | 4450708 | 2019-03-05 16:59:03 +0100 | [diff] [blame] | 460 | } |
| 461 | |
Jakob Ivarsson | 7dff9f3 | 2020-11-11 15:26:10 +0100 | [diff] [blame^] | 462 | TEST_F(DelayManagerTest, ResamplePacketDelays) { |
| 463 | use_mock_histogram_ = true; |
| 464 | resample_interval_ms_ = 500; |
| 465 | RecreateDelayManager(); |
| 466 | |
| 467 | // The histogram should be updated once with the maximum delay observed for |
| 468 | // the following sequence of packets. |
| 469 | EXPECT_CALL(*mock_histogram_, Add(5)).Times(1); |
| 470 | |
| 471 | EXPECT_EQ(absl::nullopt, InsertNextPacket()); |
| 472 | |
| 473 | IncreaseTime(kFrameSizeMs); |
| 474 | EXPECT_EQ(0, InsertNextPacket()); |
| 475 | IncreaseTime(3 * kFrameSizeMs); |
| 476 | EXPECT_EQ(2 * kFrameSizeMs, InsertNextPacket()); |
| 477 | IncreaseTime(4 * kFrameSizeMs); |
| 478 | EXPECT_EQ(5 * kFrameSizeMs, InsertNextPacket()); |
| 479 | |
| 480 | for (int i = 4; i >= 0; --i) { |
| 481 | EXPECT_EQ(i * kFrameSizeMs, InsertNextPacket()); |
| 482 | } |
| 483 | for (int i = 0; i < *resample_interval_ms_ / kFrameSizeMs; ++i) { |
| 484 | IncreaseTime(kFrameSizeMs); |
| 485 | EXPECT_EQ(0, InsertNextPacket()); |
| 486 | } |
| 487 | } |
| 488 | |
henrik.lundin@webrtc.org | d94659d | 2013-01-29 12:09:21 +0000 | [diff] [blame] | 489 | } // namespace webrtc |