Switch to literals in playout delay tests
It is important that these numbers do not change, so instead of referring to constants we will use literals here. If we need to update them we will simply have to update this test as well. Bug: webrtc:7452 Change-Id: I2808ef08d2236c10666258a8670cc2fd08543143 Reviewed-on: https://webrtc-review.googlesource.com/c/114160 Reviewed-by: Magnus Jedvert <magjed@webrtc.org> Commit-Queue: Paulina Hensman <phensman@webrtc.org> Cr-Commit-Position: refs/heads/master@{#25991}
This commit is contained in:

committed by
Commit Bot

parent
c13f4be5f4
commit
3d5df13f52
@ -746,13 +746,13 @@ TEST_F(AudioDeviceTest,
|
||||
// verifies that the audio device reports correct delay estimate given the
|
||||
// selected audio layer. Note that, this delay estimate will only be utilized
|
||||
// if the HW AEC is disabled.
|
||||
// Delay should be 75 ms in high latency and 25 ms in low latency.
|
||||
TEST_F(AudioDeviceTest, UsesCorrectDelayEstimateForHighLatencyOutputPath) {
|
||||
EXPECT_EQ(kHighLatencyModeDelayEstimateInMilliseconds / 2,
|
||||
TestDelayOnAudioLayer(AudioDeviceModule::kAndroidJavaAudio));
|
||||
EXPECT_EQ(75, TestDelayOnAudioLayer(AudioDeviceModule::kAndroidJavaAudio));
|
||||
}
|
||||
|
||||
TEST_F(AudioDeviceTest, UsesCorrectDelayEstimateForLowLatencyOutputPath) {
|
||||
EXPECT_EQ(kLowLatencyModeDelayEstimateInMilliseconds / 2,
|
||||
EXPECT_EQ(25,
|
||||
TestDelayOnAudioLayer(
|
||||
AudioDeviceModule::kAndroidJavaInputAndOpenSLESOutputAudio));
|
||||
}
|
||||
|
Reference in New Issue
Block a user