Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(252)

Unified Diff: media/base/audio_pull_fifo_unittest.cc

Issue 1596523005: Drop WebRTC audio data if OS has skipped frames. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Code review (tommi@) Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: media/base/audio_pull_fifo_unittest.cc
diff --git a/media/base/audio_pull_fifo_unittest.cc b/media/base/audio_pull_fifo_unittest.cc
index e6a7362ce2da5c15ae549bd427a1a4198efda230..4f1d98ed5fa2859724bb5d8d5a481c34eebf0f46 100644
--- a/media/base/audio_pull_fifo_unittest.cc
+++ b/media/base/audio_pull_fifo_unittest.cc
@@ -28,11 +28,15 @@ class AudioPullFifoTest
: public testing::TestWithParam<int> {
public:
AudioPullFifoTest()
- : pull_fifo_(kChannels, kMaxFramesInFifo, base::Bind(
- &AudioPullFifoTest::ProvideInput, base::Unretained(this))),
- audio_bus_(AudioBus::Create(kChannels, kMaxFramesInFifo)),
- fill_value_(0),
- last_frame_delay_(-1) {}
+ : pull_fifo_(kChannels,
+ kMaxFramesInFifo,
+ base::Bind(&AudioPullFifoTest::ProvideInput,
+ base::Unretained(this))),
+ audio_bus_(AudioBus::Create(kChannels, kMaxFramesInFifo)),
+ fill_value_(0),
+ last_frame_delay_(-1) {
+ EXPECT_EQ(kMaxFramesInFifo, pull_fifo_.SizeInFrames());
+ }
virtual ~AudioPullFifoTest() {}
void VerifyValue(const float data[], int size, float start_value) {
« content/renderer/media/webrtc_audio_renderer.cc ('K') | « media/base/audio_pull_fifo.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698