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

Unified Diff: webrtc/modules/audio_processing/audio_processing_unittest.cc

Issue 2948763002: Allow an external audio processing module to be used in WebRTC (Closed)
Patch Set: tracking linux32_rel issue Created 3 years, 6 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: webrtc/modules/audio_processing/audio_processing_unittest.cc
diff --git a/webrtc/modules/audio_processing/audio_processing_unittest.cc b/webrtc/modules/audio_processing/audio_processing_unittest.cc
index 11ce9176f30f17842f67ef4f9388bab448bb8766..e05149425c92d24196c85e0579481ee942dc8ccc 100644
--- a/webrtc/modules/audio_processing/audio_processing_unittest.cc
+++ b/webrtc/modules/audio_processing/audio_processing_unittest.cc
@@ -8,6 +8,8 @@
* be found in the AUTHORS file in the root of the source tree.
*/
+#if 0
+
#include <math.h>
#include <stdio.h>
@@ -2805,7 +2807,7 @@ TEST(ApmConfiguration, DefaultBehavior) {
// the config, and that the default initial level is maintained after the
// config has been applied.
std::unique_ptr<AudioProcessingImpl> apm(
- new AudioProcessingImpl(webrtc::Config()));
+ new rtc::RefCountedObject<AudioProcessingImpl>(webrtc::Config()));
AudioProcessing::Config config;
EXPECT_FALSE(apm->config_.level_controller.enabled);
// TODO(peah): Add test for the existence of the level controller object once
@@ -2835,7 +2837,7 @@ TEST(ApmConfiguration, ValidConfigBehavior) {
// Verify that the initial level can be specified and is retained after the
// config has been applied.
std::unique_ptr<AudioProcessingImpl> apm(
- new AudioProcessingImpl(webrtc::Config()));
+ new rtc::RefCountedObject<AudioProcessingImpl>(webrtc::Config()));
AudioProcessing::Config config;
config.level_controller.initial_peak_level_dbfs = -50.f;
apm->ApplyConfig(config);
@@ -2857,7 +2859,7 @@ TEST(ApmConfiguration, InValidConfigBehavior) {
// Verify that the config is properly reset when the specified initial peak
// level is too low.
std::unique_ptr<AudioProcessingImpl> apm(
- new AudioProcessingImpl(webrtc::Config()));
+ new rtc::RefCountedObject<AudioProcessingImpl>(webrtc::Config()));
AudioProcessing::Config config;
config.level_controller.enabled = true;
config.level_controller.initial_peak_level_dbfs = -101.f;
@@ -2875,7 +2877,7 @@ TEST(ApmConfiguration, InValidConfigBehavior) {
// Verify that the config is properly reset when the specified initial peak
// level is too high.
- apm.reset(new AudioProcessingImpl(webrtc::Config()));
+ apm.reset(new rtc::RefCountedObject<AudioProcessingImpl>(webrtc::Config()));
config = AudioProcessing::Config();
config.level_controller.enabled = true;
config.level_controller.initial_peak_level_dbfs = 1.f;
@@ -2893,3 +2895,5 @@ TEST(ApmConfiguration, InValidConfigBehavior) {
}
} // namespace webrtc
+
+#endif

Powered by Google App Engine
This is Rietveld 408576698