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

Side by Side Diff: media/audio/audio_input_writer.h

Issue 2390153006: Audio input debug recording refactoring to reduce thread hops and simplify object ownership (Closed)
Patch Set: review comments addressed 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef MEDIA_AUDIO_AUDIO_INPUT_WRITER_H_ 5 #ifndef MEDIA_AUDIO_AUDIO_INPUT_WRITER_H_
6 #define MEDIA_AUDIO_AUDIO_INPUT_WRITER_H_ 6 #define MEDIA_AUDIO_AUDIO_INPUT_WRITER_H_
7 7
8 #include <memory> 8 #include <memory>
9 9
10 namespace media { 10 namespace media {
11 11
12 class AudioBus; 12 class AudioBus;
13 13
14 // A writer interface used by AudioInputController for writing audio data to 14 // A writer interface used by AudioInputController for writing audio data to
15 // file for debugging purposes. 15 // file for debugging purposes.
16 class AudioInputWriter { 16 class AudioInputWriter {
17 public: 17 public:
18 virtual ~AudioInputWriter() {} 18 virtual ~AudioInputWriter() {}
19 19
20 // Must be called before calling Write() for the first time after creation or
21 // Stop() call. Can be called on any sequence; Write() and Stop() must be
22 // called on the same sequence as Start().
23 virtual void Start(const base::FilePath& file) = 0;
24
25 // Must be called to finish recording. Each call to Start() requires a call to
26 // Stop(). Will be automatically called on destruction.
27 virtual void Stop() = 0;
28
20 // Write |data| to file. 29 // Write |data| to file.
21 virtual void Write(std::unique_ptr<AudioBus> data) = 0; 30 virtual void Write(std::unique_ptr<AudioBus> data) = 0;
31
32 // Returns true if it makes sense to schedule Write() calls. Can be called
Henrik Grunell 2016/10/11 13:07:24 And when does it make sense? :) Describe, so that
o1ka 2016/10/11 14:04:37 Done.
33 // from any sequence.
34 virtual bool WillWrite() = 0;
22 }; 35 };
23 36
24 } // namspace media 37 } // namspace media
25 38
26 #endif // MEDIA_AUDIO_AUDIO_INPUT_WRITER_H_ 39 #endif // MEDIA_AUDIO_AUDIO_INPUT_WRITER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698