Audio output debug recording.

content/browser/renderer_host/media/audio_debug_file_writer.h

Index: content/browser/renderer_host/media/audio_debug_file_writer.h
diff --git a/content/browser/renderer_host/media/audio_debug_file_writer.h b/content/browser/renderer_host/media/audio_debug_file_writer.h
index 1ef10f186b18553f73abb4704db74359396bf592..7e26413c7a49e82ef6d1b66baf11cb1a9f4a802d 100644
--- a/content/browser/renderer_host/media/audio_debug_file_writer.h
+++ b/content/browser/renderer_host/media/audio_debug_file_writer.h
@@ -25,18 +25,25 @@ class AudioBus;
 
 namespace content {
 
-// Writes audio input data used for debugging purposes. All operations are
-// non-blocking.
+// Writes audio data to a 16 bit PCM WAVE file used for debugging purposes. All
+// operations are non-blocking.
 class CONTENT_EXPORT AudioDebugFileWriter
     : public NON_EXPORTED_BASE(media::AudioFileWriter) {
  public:
   explicit AudioDebugFileWriter(const media::AudioParameters& params);

[Guido Urdaneta, 2017/02/14 10:03:42]

Should this constructor become private?

[Henrik Grunell, 2017/02/14 11:24:22]

The Create() function's purpose is to be passed as a callback, and it returns an
AudioFileWriter intentionally so it's doesn't quite replace the constructor.
There's no particular reason for hiding the ctor either.

   ~AudioDebugFileWriter() override;
 
+  // Number of channels and sample rate are used from |params|, the other
+  // parameters are ignored. The number of channels in the data passed to
+  // Write() must match |params|.
+  static std::unique_ptr<media::AudioFileWriter> Create(
+      const media::AudioParameters& params);
+
   void Start(const base::FilePath& file) override;
   void Stop() override;
   void Write(std::unique_ptr<media::AudioBus> data) override;
   bool WillWrite() override;
+  const char* GetFileNameExtension() override;
 
  private:
   class AudioFileWriter;