Move iOS noise suppression override to default settings
Bug: None
Change-Id: I2cd642dd29a9b5e7e6141a54609b95318eb7fc85
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/195442
Reviewed-by: Per Ã…hgren <peah@webrtc.org>
Commit-Queue: Sam Zackrisson <saza@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#32702}
diff --git a/media/engine/webrtc_voice_engine.cc b/media/engine/webrtc_voice_engine.cc
index 0153554..bc382d0 100644
--- a/media/engine/webrtc_voice_engine.cc
+++ b/media/engine/webrtc_voice_engine.cc
@@ -337,21 +337,26 @@
AudioOptions options;
options.echo_cancellation = true;
options.auto_gain_control = true;
+#if defined(WEBRTC_IOS)
+ // On iOS, VPIO provides built-in NS.
+ options.noise_suppression = false;
+ options.typing_detection = false;
+#else
options.noise_suppression = true;
+ options.typing_detection = true;
+#endif
+ options.experimental_ns = false;
options.highpass_filter = true;
options.stereo_swapping = false;
options.audio_jitter_buffer_max_packets = 200;
options.audio_jitter_buffer_fast_accelerate = false;
options.audio_jitter_buffer_min_delay_ms = 0;
options.audio_jitter_buffer_enable_rtx_handling = false;
- options.typing_detection = true;
options.experimental_agc = false;
- options.experimental_ns = false;
options.residual_echo_detector = true;
bool error = ApplyOptions(options);
RTC_DCHECK(error);
}
-
initialized_ = true;
}
@@ -398,14 +403,8 @@
use_mobile_software_aec = true;
#endif
-// Set and adjust noise suppressor options.
-#if defined(WEBRTC_IOS)
- // On iOS, VPIO provides built-in NS.
- options.noise_suppression = false;
- options.typing_detection = false;
- options.experimental_ns = false;
- RTC_LOG(LS_INFO) << "Always disable NS on iOS. Use built-in instead.";
-#elif defined(WEBRTC_ANDROID)
+// Override noise suppression options for Android.
+#if defined(WEBRTC_ANDROID)
options.typing_detection = false;
options.experimental_ns = false;
#endif