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

Unified Diff: media/base/multi_channel_resampler_unittest.cc

Issue 10918098: Introduce AudioOutputResampler for browser side resampling. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: First draft! Created 8 years, 3 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/multi_channel_resampler_unittest.cc
diff --git a/media/base/multi_channel_resampler_unittest.cc b/media/base/multi_channel_resampler_unittest.cc
index d32396eb3e3a3173faab091871a92c6b63fb45a3..f046257046f946e822eefce151e411b96cad369e 100644
--- a/media/base/multi_channel_resampler_unittest.cc
+++ b/media/base/multi_channel_resampler_unittest.cc
@@ -63,6 +63,11 @@ class MultiChannelResamplerTest
base::Unretained(this)));
resampler.Resample(audio_bus_.get(), frames);
TestValues(expected_max_rms_error, expected_max_error);
+
+ // Verify Flush() of our internal state.
+ EXPECT_NE(resampler.last_frame_count_, 0);
+ resampler.Flush();
+ EXPECT_EQ(resampler.last_frame_count_, 0);
}
void HighLatencyTest(int channels) {

Powered by Google App Engine
This is Rietveld 408576698