Audio output debug recording.

media/audio/audio_debug_recording_manager.h

+// Copyright 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef MEDIA_AUDIO_AUDIO_DEBUG_RECORDING_MANAGER_H_
+#define MEDIA_AUDIO_AUDIO_DEBUG_RECORDING_MANAGER_H_
+
+#include <map>
+#include <memory>
+#include <string>
+#include <utility>
+
+#include "base/callback.h"
+#include "base/files/file_path.h"
+#include "base/gtest_prod_util.h"
+#include "base/memory/weak_ptr.h"
+#include "base/threading/thread_checker.h"
+#include "media/audio/audio_debug_recording_helper.h"
+#include "media/audio/audio_file_writer.h"
+#include "media/base/audio_parameters.h"
+#include "media/base/media_export.h"
+
+namespace base {
+class FilePath;
+class SingleThreadTaskRunner;
+}
+
+namespace media {
+
+class AudioDebugRecordingHelper;
+
+// A manager for audio debug recording that handles registration of data
+// sources and hands them a recorder (AudioDebugRecordingHelper) to feed data
+// to. The recorder will unregister with the manager automatically when deleted.
+// When debug recording is enabled, it is enabled on all recorders and
+// constructs a unique file name for each recorder by using a running ID.
+// A somewhat simplified diagram of the the debug recording infrastructure,
+// interfaces omitted:
+//
+//                                 AudioFileWriter
+//                                        ^
+//                                        | owns
+//                         owns           |
+//   OnMoreDataConverter  -----> AudioDebugRecordingHelper
+//            ^                           ^
+//            | owns several              | raw pointer to several
+//            |                           |
+//   AudioOutputResampler        AudioDebugRecordingManager
+//            ^                           ^
+//            |                           | owns
+//            | owns several              |
+//             ------------------  AudioManagerBase
+//

[o1ka, 2017/02/10 15:21:53]

This is awesome. Could you also indicate which of owned instances are null when
recording is disabled?

[Henrik Grunell, 2017/02/13 15:16:48]

Thanks. Sounds good, done. It involves what content/ is doing, but since it's
the main user I think that's fine. I added a reference to the content code.

+class MEDIA_EXPORT AudioDebugRecordingManager {
+ public:
+  AudioDebugRecordingManager(
+      AudioFileWriter::CreateCallback create_audio_file_writer_callback,
+      scoped_refptr<base::SingleThreadTaskRunner> task_runner);
+  virtual ~AudioDebugRecordingManager();
+
+  // Enables and disables debug recording.
+  virtual void EnableDebugRecording(const base::FilePath& base_file_name);
+  virtual void DisableDebugRecording();
+
+  // Registers a source and returns a wrapped recorder. |file_name_extension| is
+  // added to the base filename, along with a unique running ID.
+  std::unique_ptr<AudioDebugRecorder> RegisterDebugRecordingSource(
+      const std::string& file_name_extension,
+      const AudioParameters& params);
+
+ protected:
+  // Creates a AudioDebugRecordingHelper. Overridden by test.
+  virtual std::unique_ptr<AudioDebugRecordingHelper>
+  CreateAudioDebugRecordingHelper(
+      const AudioParameters& params,
+      const AudioFileWriter::CreateCallback& create_audio_file_writer_callback,
+      scoped_refptr<base::SingleThreadTaskRunner> task_runner,
+      base::OnceClosure on_destruction_closure);
+
+  // Callback for creating AudioFileWriter objects.
+  const AudioFileWriter::CreateCallback create_audio_file_writer_callback_;
+
+  // The task runner this class lives on. Also handed to
+  // AudioDebugRecordingHelpers.
+  scoped_refptr<base::SingleThreadTaskRunner> task_runner_;
+
+ private:
+  FRIEND_TEST_ALL_PREFIXES(AudioDebugRecordingManagerTest,
+                           RegisterAutomaticUnregisterAtDelete);
+  FRIEND_TEST_ALL_PREFIXES(AudioDebugRecordingManagerTest,
+                           RegisterEnableDisable);
+  FRIEND_TEST_ALL_PREFIXES(AudioDebugRecordingManagerTest,
+                           EnableRegisterDisable);
+
+  // Map type from source id to recorder and its filename extension.
+  using DebugRecordingHelperMap =
+      std::map<int, std::pair<AudioDebugRecordingHelper*, std::string>>;
+
+  // Unregisters a source.
+  void UnregisterDebugRecordingSource(int id);
+
+  bool IsDebugRecordingEnabled();
+
+  // Recorders, one per source.
+  DebugRecordingHelperMap debug_recording_helpers_;
+
+  // The base file name for debug recording files. If this is non-empty, debug
+  // recording is enabled.
+  base::FilePath debug_recording_base_file_name_;
+
+  base::WeakPtrFactory<AudioDebugRecordingManager> weak_factory_;
+  DISALLOW_COPY_AND_ASSIGN(AudioDebugRecordingManager);
+};
+
+}  // namespace media
+
+#endif  // MEDIA_AUDIO_AUDIO_DEBUG_RECORDING_MANAGER_H_