Allow audio system to handle synchronized low-latency audio I/O

media/audio/audio_output_controller.h

 // Copyright (c) 2012 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_OUTPUT_CONTROLLER_H_
 #define MEDIA_AUDIO_AUDIO_OUTPUT_CONTROLLER_H_
 
 #include "base/callback.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/weak_ptr.h"
@@ skipping 74 matching lines @@
   // synchronous reading.
   class SyncReader {
    public:
     virtual ~SyncReader() {}
 
     // Notify the synchronous reader the number of bytes in the
     // AudioOutputController not yet played. This is used by SyncReader to
     // prepare more data and perform synchronization.
     virtual void UpdatePendingBytes(uint32 bytes) = 0;
 
-    // Attempt to completely fill |audio_bus|, return the actual number of
+    // Attempt to completely fill |dest|, return the actual number of
     // frames that could be read.
-    virtual int Read(AudioBus* audio_bus) = 0;
+    // |source| may optionally be provided for input data.
+    virtual int Read(AudioBus* source, AudioBus* dest) = 0;

[DaleCurtis, 2012/09/11 09:44:39]

This is a bit confusing now; SyncReader and Read() don't really make sense as
names now that we're both reading and writing data to the shared memory.

[scherkus (not reviewing), 2012/09/11 12:13:00]

Do you have a suggestion for a name? Perhaps we can TODO-ify a follow-up CL to
rename some parts of this class.

[DaleCurtis, 2012/09/11 12:22:23]

TODO-ify is fine with me. No suggestions that I actually like: SyncReaderWriter,
AudioTransferAgent. 

 
     // Close this synchronous reader.
     virtual void Close() = 0;
 
     // Poll if data is ready.
     // Not reliable, as there is no guarantee that renderer is "new-style"
     // renderer that writes metadata into buffer. After several unsuccessful
     // attempts caller should assume the data is ready even if that function
     // returns false.
     virtual bool DataReady() = 0;
@@ skipping 31 matching lines @@
   // will have no effect.
   void Close(const base::Closure& closed_task);
 
   // Sets the volume of the audio output stream.
   void SetVolume(double volume);
 
   ///////////////////////////////////////////////////////////////////////////
   // AudioSourceCallback methods.
   virtual int OnMoreData(AudioBus* audio_bus,
                          AudioBuffersState buffers_state) OVERRIDE;
+  virtual int OnMoreIOData(AudioBus* source,
+                           AudioBus* dest,
+                           AudioBuffersState buffers_state) OVERRIDE;
   virtual void OnError(AudioOutputStream* stream, int code) OVERRIDE;
   virtual void WaitTillDataReady() OVERRIDE;
 
  protected:
     // Internal state of the source.
   enum State {
     kEmpty,
     kCreated,
     kPlaying,
     kStarting,
@@ skipping 65 matching lines @@
   // shutdown and force it to wait for the most delayed task.
   // Also, if we're shutting down, we do not want to poll for more data.
   base::WeakPtrFactory<AudioOutputController> weak_this_;
 
   DISALLOW_COPY_AND_ASSIGN(AudioOutputController);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_AUDIO_AUDIO_OUTPUT_CONTROLLER_H_