Reland "APM: Remove the usage of AudioFrame in the AudioProcessing interface"

This is a reland of 12e2d4ddb235da6ec7a5c1c3a83ac33d394920b0

Original change's description:
> APM: Remove the usage of AudioFrame in the AudioProcessing interface
> 
> This CL removes the AudioFrame-based APIs from the AudioProcessing
> interface.
> 
> Bug: webrtc:5298
> Change-Id: Iab470b26b10e06dcf29c543851ae0085bc5b66f0
> Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/172939
> Reviewed-by: Sam Zackrisson <saza@webrtc.org>
> Commit-Queue: Per Åhgren <peah@webrtc.org>
> Cr-Commit-Position: refs/heads/master@{#31016}

Bug: webrtc:5298
Change-Id: I70e6d59afc3716ee6109d8b9dc384abc71c93624
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/173476
Reviewed-by: Sam Zackrisson <saza@webrtc.org>
Commit-Queue: Per Åhgren <peah@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#31066}
This commit is contained in:
Per Åhgren
2020-04-07 11:33:16 +02:00
committed by Commit Bot
parent 43955465b4
commit fea8b94591
5 changed files with 4 additions and 38 deletions

View File

@ -44,7 +44,6 @@
namespace webrtc {
class ApmDataDumper;
class AudioFrame;
class AudioConverter;
class AudioProcessingImpl : public AudioProcessing {
@ -81,9 +80,6 @@ class AudioProcessingImpl : public AudioProcessing {
// Capture-side exclusive methods possibly running APM in a
// multi-threaded manner. Acquire the capture lock.
int ProcessStream(AudioFrame* frame) override {
return ProcessAudioFrame(this, frame);
}
int ProcessStream(const int16_t* const src,
const StreamConfig& input_config,
const StreamConfig& output_config,
@ -102,9 +98,6 @@ class AudioProcessingImpl : public AudioProcessing {
// Render-side exclusive methods possibly running APM in a
// multi-threaded manner. Acquire the render lock.
int ProcessReverseStream(AudioFrame* frame) override {
return ProcessReverseAudioFrame(this, frame);
}
int ProcessReverseStream(const int16_t* const src,
const StreamConfig& input_config,
const StreamConfig& output_config,