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

Unified Diff: webrtc/media/engine/webrtcvoiceengine.cc

Issue 2489343002: Corrected the way that the settings for the new APM parameter scheme are applied in VoiceEngine (Closed)
Patch Set: Rebase Created 4 years, 1 month 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
« no previous file with comments | « webrtc/media/engine/webrtcvoiceengine.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/media/engine/webrtcvoiceengine.cc
diff --git a/webrtc/media/engine/webrtcvoiceengine.cc b/webrtc/media/engine/webrtcvoiceengine.cc
index 57921acb70dd46182ebe6a43bedf829014bfa947..1f88cb7245568a1da6a5c1e52aaf751945232e92 100644
--- a/webrtc/media/engine/webrtcvoiceengine.cc
+++ b/webrtc/media/engine/webrtcvoiceengine.cc
@@ -912,17 +912,16 @@ bool WebRtcVoiceEngine::ApplyOptions(const AudioOptions& options_in) {
LOG(LS_INFO) << "Level control: "
<< (!!level_control_ ? *level_control_ : -1);
- webrtc::AudioProcessing::Config apm_config;
if (level_control_) {
- apm_config.level_controller.enabled = *level_control_;
+ apm_config_.level_controller.enabled = *level_control_;
if (options.level_control_initial_peak_level_dbfs) {
- apm_config.level_controller.initial_peak_level_dbfs =
+ apm_config_.level_controller.initial_peak_level_dbfs =
*options.level_control_initial_peak_level_dbfs;
}
}
apm()->SetExtraOptions(config);
- apm()->ApplyConfig(apm_config);
+ apm()->ApplyConfig(apm_config_);
if (options.recording_sample_rate) {
LOG(LS_INFO) << "Recording sample rate is "
« no previous file with comments | « webrtc/media/engine/webrtcvoiceengine.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698