From 6a94734d4d2f6b83072c18a558b7e1cdc8e6aaae Mon Sep 17 00:00:00 2001 From: "bjornv@webrtc.org" Date: Thu, 16 Jan 2014 08:41:09 +0000 Subject: [PATCH] Adds back set_sample_rate_hz() when Init is called in recordings. Recordings that had a AnalyzeReverseStream() call prior to ProcessStream() where aborted due to sample rates being set upon call by ProcessStream(). That change was done in r5346. Before we have a smarter handling on how to set sample rate automatically, this CL adds back that setting. BUG= TESTED=trybots, modules_unittests R=andrew@webrtc.org Review URL: https://webrtc-codereview.appspot.com/7189004 git-svn-id: http://webrtc.googlecode.com/svn/trunk@5394 4adac7df-926f-26a2-2b94-8c16560cd09d --- webrtc/modules/audio_processing/test/process_test.cc | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/webrtc/modules/audio_processing/test/process_test.cc b/webrtc/modules/audio_processing/test/process_test.cc index 94259055e1..8a5d0f5fd6 100644 --- a/webrtc/modules/audio_processing/test/process_test.cc +++ b/webrtc/modules/audio_processing/test/process_test.cc @@ -627,6 +627,9 @@ void void_main(int argc, char* argv[]) { const Init msg = event_msg.init(); ASSERT_TRUE(msg.has_sample_rate()); + // TODO(bjornv): Replace set_sample_rate_hz() when we have a smarter + // AnalyzeReverseStream(). + ASSERT_EQ(apm->kNoError, apm->set_sample_rate_hz(msg.sample_rate())); ASSERT_TRUE(msg.has_device_sample_rate()); ASSERT_EQ(apm->kNoError, apm->echo_cancellation()->set_device_sample_rate_hz( @@ -814,6 +817,10 @@ void void_main(int argc, char* argv[]) { 1, event_file)); + // TODO(bjornv): Replace set_sample_rate_hz() when we have a smarter + // AnalyzeReverseStream(). + ASSERT_EQ(apm->kNoError, apm->set_sample_rate_hz(sample_rate_hz)); + ASSERT_EQ(apm->kNoError, apm->echo_cancellation()->set_device_sample_rate_hz( device_sample_rate_hz));