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

Unified Diff: content/renderer/media/media_stream_audio_processor_unittest.cc

Issue 1357013006: Add detection for repeated audio in capturing. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fixing dll Created 5 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: content/renderer/media/media_stream_audio_processor_unittest.cc
diff --git a/content/renderer/media/media_stream_audio_processor_unittest.cc b/content/renderer/media/media_stream_audio_processor_unittest.cc
index 20eb181240206123bcaa6e75fb764533747dd93d..a00a500afed2f18d3c15df84ebff4bfe3af36780 100644
--- a/content/renderer/media/media_stream_audio_processor_unittest.cc
+++ b/content/renderer/media/media_stream_audio_processor_unittest.cc
@@ -456,7 +456,7 @@ TEST_F(MediaStreamAudioProcessorTest, MAYBE_TestAllSampleRates) {
EXPECT_TRUE(audio_processor->has_audio_processing());
static const int kSupportedSampleRates[] =
- { 8000, 16000, 22050, 32000, 44100, 48000, 88200, 96000 };
+ { 8000, 16000, 32000, 44100, 48000, 88200, 96000 };
tommi (sloooow) - chröme 2015/09/25 08:57:14 why this change? (I didn't see anything in the des
minyue 2015/09/25 14:40:06 The problem is 10 ms @ 22050 Hz is 220.5 samples.
tommi (sloooow) - chröme 2015/09/25 14:56:58 Hmm... from that answer it sounds like it's a corn
Henrik Grunell 2015/09/27 07:34:11 According to Henrik A, Chrome doesn't support 2205
minyue 2015/09/28 05:22:00 22050 has been added back
for (size_t i = 0; i < arraysize(kSupportedSampleRates); ++i) {
int buffer_size = (kSupportedSampleRates[i] / 100) < 128 ?
kSupportedSampleRates[i] / 100 : 128;

Powered by Google App Engine
This is Rietveld 408576698