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

Unified Diff: content/renderer/media/media_stream_audio_processor.h

Issue 187913002: Support the Aec dump for the APM in chrome (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 6 years, 10 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: content/renderer/media/media_stream_audio_processor.h
diff --git a/content/renderer/media/media_stream_audio_processor.h b/content/renderer/media/media_stream_audio_processor.h
index 46018b28fa615c4dec454ba21cc9161cf244bc47..bd63822837d5348e5a55dae309190e08f5c10451 100644
--- a/content/renderer/media/media_stream_audio_processor.h
+++ b/content/renderer/media/media_stream_audio_processor.h
@@ -6,6 +6,7 @@
#define CONTENT_RENDERER_MEDIA_MEDIA_STREAM_AUDIO_PROCESSOR_H_
#include "base/atomicops.h"
+#include "base/platform_file.h"
#include "base/synchronization/lock.h"
#include "base/threading/thread_checker.h"
#include "base/time/time.h"
@@ -73,7 +74,6 @@ class CONTENT_EXPORT MediaStreamAudioProcessor :
int* new_volume,
int16** out);
-
// The audio format of the input to the processor.
const media::AudioParameters& InputFormat() const;
@@ -83,6 +83,11 @@ class CONTENT_EXPORT MediaStreamAudioProcessor :
// Accessor to check if the audio processing is enabled or not.
bool has_audio_processing() const { return audio_processing_ != NULL; }
+ // Starts/Stops the Aec dump on the |audio_processing_|.
+ // Called on the main render thread.
Henrik Grunell 2014/03/06 10:12:20 Comment that it takes ownership of |aec_dump_file|
no longer working on chromium 2014/03/06 18:57:21 Done.
+ void StartAecDump(const base::PlatformFile& aec_dump_file);
+ void StopAecDump();
+
protected:
friend class base::RefCountedThreadSafe<MediaStreamAudioProcessor>;
virtual ~MediaStreamAudioProcessor();

Powered by Google App Engine
This is Rietveld 408576698