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

Unified Diff: webrtc/modules/audio_processing/aec3/echo_path_delay_estimator.h

Issue 2644123002: Adding full initial version of delay estimation functionality in echo canceller 3 (Closed)
Patch Set: Created 3 years, 11 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/aec3/echo_path_delay_estimator.h
diff --git a/webrtc/modules/audio_processing/aec3/echo_path_delay_estimator.h b/webrtc/modules/audio_processing/aec3/echo_path_delay_estimator.h
index ae8ab4318e66a52ee84dddd743caa104b4b347de..831a6f437dd1abc502552717811abb9a919f0c0e 100644
--- a/webrtc/modules/audio_processing/aec3/echo_path_delay_estimator.h
+++ b/webrtc/modules/audio_processing/aec3/echo_path_delay_estimator.h
@@ -1,3 +1,4 @@
+
hlundin-webrtc 2017/02/01 08:30:02 WAT?
peah-webrtc 2017/02/02 14:04:47 Done.
/*
* Copyright (c) 2017 The WebRTC project authors. All Rights Reserved.
*
@@ -15,19 +16,32 @@
#include "webrtc/base/constructormagic.h"
#include "webrtc/base/optional.h"
+#include "webrtc/modules/audio_processing/aec3/correlator_lag_aggregator.h"
+#include "webrtc/modules/audio_processing/aec3/correlator.h"
+#include "webrtc/modules/audio_processing/aec3/down_sampler_4khz.h"
namespace webrtc {
class ApmDataDumper;
+// Estimates the delay of the echo path.
class EchoPathDelayEstimator {
public:
EchoPathDelayEstimator(ApmDataDumper* data_dumper, int sample_rate_hz);
~EchoPathDelayEstimator();
+
+ // Produce a delay estimate if such is avaliable.
rtc::Optional<size_t> EstimateDelay(rtc::ArrayView<const float> render,
rtc::ArrayView<const float> capture);
private:
+ ApmDataDumper* const data_dumper_;
+ DownSampler4kHz render_down_sampler_;
+ DownSampler4kHz capture_down_sampler_;
+ const int down_sampling_factor_;
+ Correlator correlator_;
+ CorrelatorLagAggregator correlator_lag_aggregator_;
+
RTC_DISALLOW_IMPLICIT_CONSTRUCTORS(EchoPathDelayEstimator);
};
} // namespace webrtc

Powered by Google App Engine
This is Rietveld 408576698