diff --git a/webrtc/modules/audio_processing/include/audio_processing.cc b/webrtc/modules/audio_processing/include/audio_processing.cc index eec4db6179..52a57aa6e7 100644 --- a/webrtc/modules/audio_processing/include/audio_processing.cc +++ b/webrtc/modules/audio_processing/include/audio_processing.cc @@ -11,9 +11,6 @@ #include "webrtc/modules/audio_processing/include/audio_processing.h" #include "webrtc/base/checks.h" -#include "webrtc/modules/audio_processing/include/aec_dump.h" -// TODO(aleloi): remove AecDump header usage when internal projcets -// have updated. See https://bugs.webrtc.org/7404. namespace webrtc { @@ -35,21 +32,4 @@ Beamforming::Beamforming(bool enabled, target_direction(target_direction) {} Beamforming::~Beamforming() {} - -// TODO(aleloi): make pure virtual when internal projects have -// updated. See https://bugs.webrtc.org/7404 -void AudioProcessing::AttachAecDump(std::unique_ptr aec_dump) { - RTC_NOTREACHED(); -} - -// If no AecDump is attached, this has no effect. If an AecDump is -// attached, it's destructor is called. The d-tor may block until -// all pending logging tasks are completed. -// -// TODO(aleloi): make pure virtual when internal projects have -// updated. See https://bugs.webrtc.org/7404 -void AudioProcessing::DetachAecDump() { - RTC_NOTREACHED(); -} - } // namespace webrtc diff --git a/webrtc/modules/audio_processing/include/audio_processing.h b/webrtc/modules/audio_processing/include/audio_processing.h index 33fdff24b6..6a5a81d3b5 100644 --- a/webrtc/modules/audio_processing/include/audio_processing.h +++ b/webrtc/modules/audio_processing/include/audio_processing.h @@ -463,18 +463,12 @@ class AudioProcessing { // with a new one. This causes the d-tor of the earlier AecDump to // be called. The d-tor call may block until all pending logging // tasks are completed. - // - // TODO(aleloi): make pure virtual when internal projects have - // updated. See https://bugs.webrtc.org/7404 - virtual void AttachAecDump(std::unique_ptr aec_dump); + virtual void AttachAecDump(std::unique_ptr aec_dump) = 0; // If no AecDump is attached, this has no effect. If an AecDump is // attached, it's destructor is called. The d-tor may block until // all pending logging tasks are completed. - // - // TODO(aleloi): make pure virtual when internal projects have - // updated. See https://bugs.webrtc.org/7404 - virtual void DetachAecDump(); + virtual void DetachAecDump() = 0; // Starts recording debugging information to a file specified by |filename|, // a NULL-terminated string. If there is an ongoing recording, the old file