Switch OnMoreData() to use AudioBus.

ppapi/shared_impl/ppb_audio_shared.cc

 // 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.
 
 #include "ppapi/shared_impl/ppb_audio_shared.h"
 
 #include "base/logging.h"
 #include "media/audio/shared_memory_util.h"
 #include "ppapi/shared_impl/ppapi_globals.h"
 
+// Hard coded values from PepperPlatformAudioOutputImpl.
+// TODO(???): PPAPI shouldn't hard code these values for all clients.
+enum { kChannels = 2, kBytesPerSample = 2 };
+
 namespace ppapi {
 
 #if defined(OS_NACL)
 namespace {
 // Because this is static, the function pointers will be NULL initially.
 PP_ThreadFunctions thread_functions;
 }
 #endif  // defined(OS_NACL)
 
 PPB_Audio_Shared::PPB_Audio_Shared()
     : playing_(false),
       shared_memory_size_(0),
 #if defined(OS_NACL)
       thread_id_(0),
       thread_active_(false),
 #endif
       callback_(NULL),
-      user_data_(NULL) {
+      user_data_(NULL),
+      audio_buffer_size_(0) {
 }
 
 PPB_Audio_Shared::~PPB_Audio_Shared() {
   StopThread();
 }
 
 void PPB_Audio_Shared::SetCallback(PPB_Audio_Callback callback,
                                    void* user_data) {
   callback_ = callback;
   user_data_ = user_data;
@@ skipping 27 matching lines @@
     size_t shared_memory_size,
     base::SyncSocket::Handle socket_handle) {
   socket_.reset(new base::CancelableSyncSocket(socket_handle));
   shared_memory_.reset(new base::SharedMemory(shared_memory_handle, false));
   shared_memory_size_ = shared_memory_size;
 
   if (!shared_memory_->Map(
           media::TotalSharedMemorySizeInBytes(shared_memory_size_))) {
     PpapiGlobals::Get()->LogWithSource(instance, PP_LOGLEVEL_WARNING, "",
       "Failed to map shared memory for PPB_Audio_Shared.");
+  } else {
+    // Deduce the frame count from the size using hard coded channel count.
+    audio_bus_ = media::AudioBus::WrapMemory(
+        kChannels, shared_memory_size_ / (sizeof(float) * kChannels),
+        shared_memory_.get());
+    // Setup integer audio buffer for user audio data.
+    audio_buffer_size_ =
+        audio_bus_->frames() * audio_bus_->channels() * kBytesPerSample;
+    audio_buffer_.reset(new uint8_t[audio_buffer_size_]);
   }
 
   StartThread();
 }
 
 void PPB_Audio_Shared::StartThread() {
   // Don't start the thread unless all our state is set up correctly.
-  if (!playing_ || !callback_ || !socket_.get() || !shared_memory_->memory())
+  if (!playing_ || !callback_ || !socket_.get() || !shared_memory_->memory() ||
+      !audio_bus_.get() || !audio_buffer_.get())
     return;
   // Clear contents of shm buffer before starting audio thread. This will
   // prevent a burst of static if for some reason the audio thread doesn't
   // start up quickly enough.
   memset(shared_memory_->memory(), 0, shared_memory_size_);
+  memset(audio_buffer_.get(), 0, audio_buffer_size_);
 #if !defined(OS_NACL)
   DCHECK(!audio_thread_.get());
   audio_thread_.reset(new base::DelegateSimpleThread(
       this, "plugin_audio_thread"));
   audio_thread_->Start();
 #else
   // Use NaCl's special API for IRT code that creates threads that call back
   // into user code.
   if (NULL == thread_functions.thread_create ||
       NULL == thread_functions.thread_join)
@@ skipping 34 matching lines @@
 
 // static
 void PPB_Audio_Shared::CallRun(void* self) {
   PPB_Audio_Shared* audio = static_cast<PPB_Audio_Shared*>(self);
   audio->Run();
 }
 #endif
 
 void PPB_Audio_Shared::Run() {
   int pending_data;
-  void* buffer = shared_memory_->memory();
 
   while (sizeof(pending_data) ==
       socket_->Receive(&pending_data, sizeof(pending_data)) &&
       pending_data != media::kPauseMark) {
-    callback_(buffer, shared_memory_size_, user_data_);
+    callback_(audio_buffer_.get(), audio_buffer_size_, user_data_);
+
+    // Deinterleave the audio data into the shared memory as float.
+    audio_bus_->FromInterleaved(
+        audio_buffer_.get(), audio_bus_->frames(), kBytesPerSample);
 
     // Let the host know we are done.
     media::SetActualDataSizeInBytes(
         shared_memory_.get(), shared_memory_size_, shared_memory_size_);
   }
 }
 
 }  // namespace ppapi