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

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: Changes in response to reviewer comments 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..22eb3a836dc5b9b4709feca08a8fa4086dcacb00 100644
--- a/webrtc/modules/audio_processing/aec3/echo_path_delay_estimator.h
+++ b/webrtc/modules/audio_processing/aec3/echo_path_delay_estimator.h
@@ -15,19 +15,31 @@
#include "webrtc/base/constructormagic.h"
#include "webrtc/base/optional.h"
+#include "webrtc/modules/audio_processing/aec3/matched_filter.h"
+#include "webrtc/modules/audio_processing/aec3/matched_filter_lag_aggregator.h"
+#include "webrtc/modules/audio_processing/aec3/decimator_by_4.h"
namespace webrtc {
class ApmDataDumper;
+// Estimates the delay of the echo path.
class EchoPathDelayEstimator {
public:
EchoPathDelayEstimator(ApmDataDumper* data_dumper, int sample_rate_hz);
aleloi 2017/02/03 15:46:48 Does the delay estimator need to know the sample r
peah-webrtc 2017/02/06 11:25:38 That is correct. It is no longer needed. I removed
~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_;
+ DecimatorBy4 render_decimator_;
+ DecimatorBy4 capture_decimator_;
+ MatchedFilter matched_filter_;
+ MatchedFilterLagAggregator matched_filter_lag_aggregator_;
+
RTC_DISALLOW_IMPLICIT_CONSTRUCTORS(EchoPathDelayEstimator);
};
} // namespace webrtc

Powered by Google App Engine
This is Rietveld 408576698