Reland "Make relative arrival delay mode default in NetEq delay manager."
This is a reland of 77c71d1488b1c821b2b3481f23a3264f1b1d37a5 Original change's description: > Make relative arrival delay mode default in NetEq delay manager. > > Bug: webrtc:10333 > Change-Id: I9b1e0bec0b1813cf31259492f83eb2ca86a44d3f > Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/150782 > Reviewed-by: Sebastian Jansson <srte@webrtc.org> > Reviewed-by: Henrik Lundin <henrik.lundin@webrtc.org> > Reviewed-by: Minyue Li <minyue@webrtc.org> > Commit-Queue: Jakob Ivarsson <jakobi@webrtc.org> > Cr-Commit-Position: refs/heads/master@{#29075} Bug: webrtc:10333 Change-Id: I9c726cec1afc1147a4618fc224404a83962e6ae2 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/152281 Reviewed-by: Minyue Li <minyue@webrtc.org> Reviewed-by: Henrik Lundin <henrik.lundin@webrtc.org> Reviewed-by: Sebastian Jansson <srte@webrtc.org> Commit-Queue: Jakob Ivarsson <jakobi@webrtc.org> Cr-Commit-Position: refs/heads/master@{#29136}
This commit is contained in:
committed by
Commit Bot
parent
3354157d36
commit
507f43465b
@ -53,20 +53,6 @@ class TargetDelayTest : public ::testing::Test {
|
||||
EXPECT_EQ(-1, SetMinimumDelay(10001));
|
||||
}
|
||||
|
||||
void WithTargetDelayBufferNotChanging() {
|
||||
// A target delay that is one packet larger than jitter.
|
||||
const int kTargetDelayMs =
|
||||
(kInterarrivalJitterPacket + 1) * kNum10msPerFrame * 10;
|
||||
ASSERT_EQ(0, SetMinimumDelay(kTargetDelayMs));
|
||||
for (int n = 0; n < 30; ++n) // Run enough iterations to fill the buffer.
|
||||
Run(true);
|
||||
int clean_optimal_delay = GetCurrentOptimalDelayMs();
|
||||
EXPECT_EQ(kTargetDelayMs, clean_optimal_delay);
|
||||
Run(false); // Run with jitter.
|
||||
int jittery_optimal_delay = GetCurrentOptimalDelayMs();
|
||||
EXPECT_EQ(jittery_optimal_delay, clean_optimal_delay);
|
||||
}
|
||||
|
||||
void TargetDelayBufferMinMax() {
|
||||
const int kTargetMinDelayMs = kNum10msPerFrame * 10;
|
||||
ASSERT_EQ(0, SetMinimumDelay(kTargetMinDelayMs));
|
||||
@ -162,17 +148,6 @@ TEST_F(TargetDelayTest, MAYBE_OutOfRangeInput) {
|
||||
OutOfRangeInput();
|
||||
}
|
||||
|
||||
// Flaky on iOS: webrtc:7057.
|
||||
#if defined(WEBRTC_ANDROID) || defined(WEBRTC_IOS)
|
||||
#define MAYBE_WithTargetDelayBufferNotChanging \
|
||||
DISABLED_WithTargetDelayBufferNotChanging
|
||||
#else
|
||||
#define MAYBE_WithTargetDelayBufferNotChanging WithTargetDelayBufferNotChanging
|
||||
#endif
|
||||
TEST_F(TargetDelayTest, MAYBE_WithTargetDelayBufferNotChanging) {
|
||||
WithTargetDelayBufferNotChanging();
|
||||
}
|
||||
|
||||
// Flaky on iOS: webrtc:7057.
|
||||
#if defined(WEBRTC_ANDROID) || defined(WEBRTC_IOS)
|
||||
#define MAYBE_TargetDelayBufferMinMax DISABLED_TargetDelayBufferMinMax
|
||||
|
||||
Reference in New Issue
Block a user