Reland of added functionality for specifying the initial signal level to use for the gain estimation

webrtc/modules/audio_processing/include/audio_processing.h

 /*
  *  Copyright (c) 2012 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.
  */
 
 #ifndef WEBRTC_MODULES_AUDIO_PROCESSING_INCLUDE_AUDIO_PROCESSING_H_
 #define WEBRTC_MODULES_AUDIO_PROCESSING_INCLUDE_AUDIO_PROCESSING_H_
 
 // MSVC++ requires this to be set before any other includes to get M_PI.
 #define _USE_MATH_DEFINES
 
 #include <math.h>
 #include <stddef.h>  // size_t
 #include <stdio.h>  // FILE
 #include <vector>
 
 #include "webrtc/base/arraysize.h"
+#include "webrtc/base/optional.h"

[the sun, 2016/09/16 12:40:24]

remove

[peah-webrtc, 2016/09/19 16:37:10]

Done.

 #include "webrtc/base/platform_file.h"
 #include "webrtc/modules/audio_processing/beamformer/array_util.h"
 #include "webrtc/modules/audio_processing/include/config.h"
 #include "webrtc/typedefs.h"
 
 namespace webrtc {
 
 struct AecCore;
 
 class AudioFrame;
@@ skipping 212 matching lines @@
   // introduced, it is prone to change.
   // TODO(peah): Remove this comment once the new config scheme is fully rolled
   // out.
   //
   // The parameters and behavior of the audio processing module are controlled
   // by changing the default values in the AudioProcessing::Config struct.
   // The config is applied by passing the struct to the ApplyConfig method.
   struct Config {
     struct LevelController {
       bool enabled = false;
+
+      // Sets the initial peak level to use inside the level controller in order
+      // to compute the signal gain. The unit for the peak level is dBFS and
+      // the allowed range is [-100, 0].
+      float initial_peak_level = -6.0206;

[the sun, 2016/09/16 12:40:24]

-6.0206f

adding _db to the parameter name would make code using it easier to understand.

[hlundin-webrtc, 2016/09/19 12:03:03]

Acknowledged.

[peah-webrtc, 2016/09/19 16:37:10]

Done.

[peah-webrtc, 2016/09/19 16:37:10]

Acknowledged.

     } level_controller;
   };
 
   // TODO(mgraczyk): Remove once all methods that use ChannelLayout are gone.
   enum ChannelLayout {
     kMono,
     // Left, right.
     kStereo,
     // Mono, keyboard, and mic.
     kMonoAndKeyboard,
@@ skipping 730 matching lines @@
   // This does not impact the size of frames passed to |ProcessStream()|.
   virtual int set_frame_size_ms(int size) = 0;
   virtual int frame_size_ms() const = 0;
 
  protected:
   virtual ~VoiceDetection() {}
 };
 }  // namespace webrtc
 
 #endif  // WEBRTC_MODULES_AUDIO_PROCESSING_INCLUDE_AUDIO_PROCESSING_H_