Implementation of ppapi audio.

webkit/plugins/ppapi/ppb_audio_impl.cc

 // Copyright (c) 2011 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 "webkit/plugins/ppapi/ppb_audio_impl.h"
 
 #include "base/logging.h"
 #include "ppapi/c/pp_completion_callback.h"
 #include "ppapi/c/ppb_audio.h"
 #include "ppapi/c/ppb_audio_config.h"
@@ skipping 10 matching lines @@
 using ppapi::thunk::PPB_Audio_API;
 using ppapi::thunk::PPB_AudioConfig_API;
 
 namespace webkit {
 namespace ppapi {
 
 // PPB_Audio_Impl --------------------------------------------------------------
 
 PPB_Audio_Impl::PPB_Audio_Impl(PP_Instance instance)
     : Resource(instance),
-      audio_(NULL),
-      create_callback_pending_(false),
-      shared_memory_size_for_create_callback_(0) {
-  create_callback_ = PP_MakeCompletionCallback(NULL, NULL);
+      audio_(NULL) {
 }
 
 PPB_Audio_Impl::~PPB_Audio_Impl() {
   // Calling ShutDown() makes sure StreamCreated cannot be called anymore and
   // releases the audio data associated with the pointer. Note however, that
   // until ShutDown returns, StreamCreated may still be called. This will be
   // OK since we'll just immediately clean up the data it stored later in this
   // destructor.
   if (audio_) {
     audio_->ShutDown();
     audio_ = NULL;
   }
-
-  // If the completion callback hasn't fired yet, do so here
-  // with an error condition.
-  if (create_callback_pending_) {
-    PP_RunCompletionCallback(&create_callback_, PP_ERROR_ABORTED);
-    create_callback_pending_ = false;
-  }
 }
 
 // static
 PP_Resource PPB_Audio_Impl::Create(PP_Instance instance,
                                    PP_Resource config,
                                    PPB_Audio_Callback audio_callback,
                                    void* user_data) {
   scoped_refptr<PPB_Audio_Impl> audio(new PPB_Audio_Impl(instance));
   if (!audio->Init(config, audio_callback, user_data))
     return 0;
@@ skipping 70 matching lines @@
   DCHECK(!audio_);
   audio_ = plugin_delegate->CreateAudio(enter.object()->GetSampleRate(),
                                         enter.object()->GetSampleFrameCount(),
                                         this);
   if (!audio_)
     return PP_ERROR_FAILED;
 
   // At this point, we are guaranteeing ownership of the completion
   // callback.  Audio promises to fire the completion callback
   // once and only once.
-  create_callback_ = create_callback;
-  create_callback_pending_ = true;
+  SetCallbackInfo(true, create_callback);
+
   return PP_OK_COMPLETIONPENDING;
 }
 
 int32_t PPB_Audio_Impl::GetSyncSocket(int* sync_socket) {
-  if (socket_for_create_callback_.get()) {
-#if defined(OS_POSIX)
-    *sync_socket = socket_for_create_callback_->handle();
-#elif defined(OS_WIN)
-    *sync_socket = reinterpret_cast<int>(socket_for_create_callback_->handle());
-#else
-    #error "Platform not supported."
-#endif
-    return PP_OK;
-  }
-  return PP_ERROR_FAILED;
+    return AudioHelper::GetSyncSocket(sync_socket);

[viettrungluu, 2011/11/10 19:31:57]

nit: incorrect indentation

 }
 
-int32_t PPB_Audio_Impl::GetSharedMemory(int* shm_handle, uint32_t* shm_size) {
-  if (shared_memory_for_create_callback_.get()) {
-#if defined(OS_POSIX)
-    *shm_handle = shared_memory_for_create_callback_->handle().fd;
-#elif defined(OS_WIN)
-    *shm_handle = reinterpret_cast<int>(
-        shared_memory_for_create_callback_->handle());
-#else
-    #error "Platform not supported."
-#endif
-    *shm_size = shared_memory_size_for_create_callback_;
-    return PP_OK;
-  }
-  return PP_ERROR_FAILED;
+int32_t PPB_Audio_Impl::GetSharedMemory(int* shm_handle,
+                                        uint32_t* shm_size) {
+    return AudioHelper::GetSharedMemory(shm_handle, shm_size);

[viettrungluu, 2011/11/10 19:31:57]

"

+}
+
+void PPB_Audio_Impl::OnSetStreamInfo(
+        base::SharedMemoryHandle shared_memory_handle,
+        size_t shared_memory_size,
+        base::SyncSocket::Handle socket_handle) {
+    SetStreamInfo(shared_memory_handle, shared_memory_size, socket_handle);

[viettrungluu, 2011/11/10 19:31:57]

"

 }
 
 void PPB_Audio_Impl::StreamCreated(
-    base::SharedMemoryHandle shared_memory_handle,
-    size_t shared_memory_size,
-    base::SyncSocket::Handle socket_handle) {
-  if (create_callback_pending_) {
-    // Trusted side of proxy can specify a callback to recieve handles. In
-    // this case we don't need to map any data or start the thread since it
-    // will be handled by the proxy.
-    shared_memory_for_create_callback_.reset(
-        new base::SharedMemory(shared_memory_handle, false));
-    shared_memory_size_for_create_callback_ = shared_memory_size;
-    socket_for_create_callback_.reset(new base::SyncSocket(socket_handle));
-
-    PP_RunCompletionCallback(&create_callback_, 0);
-    create_callback_pending_ = false;
-
-    // It might be nice to close the handles here to free up some system
-    // resources, but we can't since there's a race condition. The handles must
-    // be valid until they're sent over IPC, which is done from the I/O thread
-    // which will often get done after this code executes. We could do
-    // something more elaborate like an ACK from the plugin or post a task to
-    // the I/O thread and back, but this extra complexity doesn't seem worth it
-    // just to clean up these handles faster.
-  } else {
-    SetStreamInfo(shared_memory_handle, shared_memory_size, socket_handle);
-  }
+        base::SharedMemoryHandle shared_memory_handle,
+        size_t shared_memory_size,
+        base::SyncSocket::Handle socket_handle) {
+    StreamCreated_(shared_memory_handle, shared_memory_size, socket_handle);

[viettrungluu, 2011/11/10 19:31:57]

"

 }
 
 }  // namespace ppapi
 }  // namespace webkit
+

[viettrungluu, 2011/11/10 19:31:57]

no need to add extraneous blank lines