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

Unified Diff: webrtc/modules/audio_processing/high_pass_filter_impl.h

Issue 2415403002: Introduced the new parameter setting scheme for activating the high-pass filter in APM (Closed)
Patch Set: New approach for introducing the new parameter setting scheme Created 4 years, 2 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/high_pass_filter_impl.h
diff --git a/webrtc/modules/audio_processing/high_pass_filter_impl.h b/webrtc/modules/audio_processing/high_pass_filter_impl.h
index 5cb18f12d08d06b56a8802964fdddd5d2810f3bf..5092deb95e68ad4da360c5f5efe9fca4e88f053d 100644
--- a/webrtc/modules/audio_processing/high_pass_filter_impl.h
+++ b/webrtc/modules/audio_processing/high_pass_filter_impl.h
@@ -15,7 +15,7 @@
#include <vector>
#include "webrtc/base/constructormagic.h"
-#include "webrtc/base/criticalsection.h"
+#include "webrtc/modules/audio_processing/audio_processing_impl.h"
#include "webrtc/modules/audio_processing/include/audio_processing.h"
namespace webrtc {
@@ -24,22 +24,15 @@ class AudioBuffer;
class HighPassFilterImpl : public HighPassFilter {
public:
- explicit HighPassFilterImpl(rtc::CriticalSection* crit);
+ explicit HighPassFilterImpl(AudioProcessingImpl* apm);
~HighPassFilterImpl() override;
- // TODO(peah): Fold into ctor, once public API is removed.
- void Initialize(size_t channels, int sample_rate_hz);
- void ProcessCaptureAudio(AudioBuffer* audio);
-
// HighPassFilter implementation.
int Enable(bool enable) override;
bool is_enabled() const override;
private:
- class BiquadFilter;
- rtc::CriticalSection* const crit_ = nullptr;
- bool enabled_ GUARDED_BY(crit_) = false;
- std::vector<std::unique_ptr<BiquadFilter>> filters_ GUARDED_BY(crit_);
+ AudioProcessingImpl* apm_;
RTC_DISALLOW_IMPLICIT_CONSTRUCTORS(HighPassFilterImpl);
};
} // namespace webrtc

Powered by Google App Engine
This is Rietveld 408576698