Reducing calling to SmoothingFilter in Fec Controller.

webrtc/modules/audio_coding/audio_network_adaptor/fec_controller.cc

 /*
  *  Copyright (c) 2016 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 57 matching lines @@
 
 void FecController::MakeDecision(
     const NetworkMetrics& metrics,
     AudioNetworkAdaptor::EncoderRuntimeConfig* config) {
   RTC_DCHECK(!config->enable_fec);
   RTC_DCHECK(!config->uplink_packet_loss_fraction);
 
   if (metrics.uplink_packet_loss_fraction)
     packet_loss_smoothed_->AddSample(*metrics.uplink_packet_loss_fraction);
 
-  fec_enabled_ = fec_enabled_ ? !FecDisablingDecision(metrics)
-                              : FecEnablingDecision(metrics);
+  const auto& packet_loss = packet_loss_smoothed_->GetAverage();
+
+  fec_enabled_ = fec_enabled_ ? !FecDisablingDecision(metrics, packet_loss)
+                              : FecEnablingDecision(metrics, packet_loss);
 
   config->enable_fec = rtc::Optional<bool>(fec_enabled_);
 
   auto packet_loss_fraction = packet_loss_smoothed_->GetAverage();
   config->uplink_packet_loss_fraction = rtc::Optional<float>(
       packet_loss_fraction ? *packet_loss_fraction : 0.0);
 }
 
 FecController::ThresholdInfo::ThresholdInfo(
     const Config::Threshold& threshold) {
@@ skipping 10 matching lines @@
     int bandwidth_bps,
     const Config::Threshold& threshold,
     const ThresholdInfo& threshold_info) const {
   if (bandwidth_bps < threshold.low_bandwidth_bps)
     return std::numeric_limits<float>::max();
   if (bandwidth_bps >= threshold.high_bandwidth_bps)
     return threshold.high_bandwidth_packet_loss;
   return threshold_info.offset + threshold_info.slope * bandwidth_bps;
 }
 
-bool FecController::FecEnablingDecision(const NetworkMetrics& metrics) const {
+bool FecController::FecEnablingDecision(
+    const NetworkMetrics& metrics,
+    const rtc::Optional<float>& packet_loss) const {
   if (!metrics.uplink_bandwidth_bps)
     return false;
-
-  auto packet_loss = packet_loss_smoothed_->GetAverage();
   if (!packet_loss)
     return false;
-
   return *packet_loss >= GetPacketLossThreshold(*metrics.uplink_bandwidth_bps,
                                                 config_.fec_enabling_threshold,
                                                 fec_enabling_threshold_info_);
 }
 
-bool FecController::FecDisablingDecision(const NetworkMetrics& metrics) const {
+bool FecController::FecDisablingDecision(
+    const NetworkMetrics& metrics,
+    const rtc::Optional<float>& packet_loss) const {
   if (!metrics.uplink_bandwidth_bps)
     return false;
-
-  auto packet_loss = packet_loss_smoothed_->GetAverage();
   if (!packet_loss)
     return false;
-
   return *packet_loss <= GetPacketLossThreshold(*metrics.uplink_bandwidth_bps,
                                                 config_.fec_disabling_threshold,
                                                 fec_disabling_threshold_info_);
 }
 
 }  // namespace webrtc