commit 11539f0b29462bfc3e92a4ade65cb295b191cd09
author Gustaf Ullberg <gustaf@webrtc.org> Mon Oct 15 13:40:29 2018 +0200
committer Commit Bot <commit-bot@chromium.org> Mon Oct 15 13:31:50 2018 +0000
tree bdbf7acc47a7ec327749208498113cf37285c21e
parent e07864ea6ec179562bf3915aaac6f5b55f700c0e

AEC3: Simplify render buffering

This CL simplifies the buffering of render data. Instead of making assumptions
about the worst possible platform, it leverages recent improvements in
the delay estimator to quickly adapt when the conditions change.

Pros:
- No capture delay, delay is found ~200 ms faster.
- Cleaner code that makes the concept of delay more clear.
- Allows for removal of one matched filter because of the jitter headroom
removal.

Cons:
- Delay estimator needs to re-adapt when the call jitter increases.

The code can be deactivated by a kill switch. When the kill switch is
pulled the CL is bit exact.

Bug: webrtc:9726,chromium:895338
Change-Id: Ie2f9c8c5ce5b5a4510b4bdb95db2b970b57cd5d0
Reviewed-on: https://webrtc-review.googlesource.com/c/96920
Commit-Queue: Gustaf Ullberg <gustaf@webrtc.org>
Reviewed-by: Per Åhgren <peah@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#25169}

modules/audio_processing/aec3/render_delay_controller.h

diff --git a/modules/audio_processing/aec3/render_delay_controller.h b/modules/audio_processing/aec3/render_delay_controller.h
index ddd9548..a0cbe89 100644
--- a/modules/audio_processing/aec3/render_delay_controller.h
+++ b/modules/audio_processing/aec3/render_delay_controller.h
@@ -27,6 +27,8 @@
   static RenderDelayController* Create(const EchoCanceller3Config& config,
                                        int non_causal_offset,
                                        int sample_rate_hz);
+  static RenderDelayController* Create2(const EchoCanceller3Config& config,
+                                        int sample_rate_hz);
   virtual ~RenderDelayController() = default;
 
   // Resets the delay controller.