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

Unified Diff: webrtc/modules/audio_processing/level_controller/level_controller.h

Issue 2337083002: Reland of added functionality for specifying the initial signal level to use for the gain estimation (Closed)
Patch Set: Changes in response to reviewer comments Created 4 years, 2 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/level_controller/level_controller.h
diff --git a/webrtc/modules/audio_processing/level_controller/level_controller.h b/webrtc/modules/audio_processing/level_controller/level_controller.h
index 1d8e043cce410adbb1b07e970b41bf75f08a4395..8a96e2ce1e90fab0aeea9971e3ddcbce10010347 100644
--- a/webrtc/modules/audio_processing/level_controller/level_controller.h
+++ b/webrtc/modules/audio_processing/level_controller/level_controller.h
@@ -15,6 +15,7 @@
#include <vector>
#include "webrtc/base/constructormagic.h"
+#include "webrtc/base/gtest_prod_util.h"
the sun 2016/10/06 07:21:14 remove
peah-webrtc 2016/10/07 21:03:58 Done.
#include "webrtc/base/optional.h"
#include "webrtc/modules/audio_processing/include/audio_processing.h"
#include "webrtc/modules/audio_processing/level_controller/gain_applier.h"
@@ -38,6 +39,10 @@ class LevelController {
void Process(AudioBuffer* audio);
float GetLastGain() { return last_gain_; }
+ // TODO(peah): This method is a temporary solution as the the aim is to
+ // instead apply the config inside the constructor. Therefore this is likely
+ // to change.
+ void ApplyConfig(const AudioProcessing::Config::LevelController& config);
// Validates a config.
static bool Validate(const AudioProcessing::Config::LevelController& config);
// Dumps a config to a string.
@@ -80,6 +85,8 @@ class LevelController {
float dc_level_[2];
float dc_forgetting_factor_;
float last_gain_;
+ bool gain_jumpstart_ = false;
+ AudioProcessing::Config::LevelController config_;
RTC_DISALLOW_COPY_AND_ASSIGN(LevelController);
};

Powered by Google App Engine
This is Rietveld 408576698