Switch OnMoreData() to use AudioBus.

media/audio/audio_device_thread.cc

Index: media/audio/audio_device_thread.cc
diff --git a/media/audio/audio_device_thread.cc b/media/audio/audio_device_thread.cc
index 3e14d3de53929ae88488fe03fc200c325529ca89..cccd7aff6344b4d913be76a91918097e7acc18c7 100644
--- a/media/audio/audio_device_thread.cc
+++ b/media/audio/audio_device_thread.cc
@@ -197,9 +197,7 @@ void AudioDeviceThread::Callback::InitializeOnAudioThread() {
   MapSharedMemory();
   DCHECK(shared_memory_.memory() != NULL);
 
-  // TODO(dalecurtis): Instead of creating a new AudioBus and memcpy'ing into
-  // the shared memory we should wrap the shared memory.
-  audio_bus_ = AudioBus::Create(audio_parameters_);
+  audio_bus_ = AudioBus::WrapMemory(audio_parameters_, shared_memory_.memory());

[Chris Rogers, 2012/08/24 20:20:26]

I think we should have a sanity check that the audio_parameters_ reflects the
actual size of the shared_memory_ (and not just accept it on faith from what the
browser gives back to us)

[DaleCurtis, 2012/08/24 23:53:12]

Done.

 }
 
 }  // namespace media.