Changes to allow testing interface for new NaCl Proxy

ppapi/proxy/ppb_audio_proxy.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/proxy/ppb_audio_proxy.h"
 
 #include "base/compiler_specific.h"
 #include "base/threading/simple_thread.h"
 #include "media/audio/shared_memory_util.h"
 #include "ppapi/c/pp_errors.h"
@@ skipping 56 matching lines @@
              PP_Resource config_id,
              PPB_Audio_Callback callback,
              void* user_data)
     : Resource(OBJECT_IS_PROXY, audio_id),
       config_(config_id) {
   SetCallback(callback, user_data);
   PpapiGlobals::Get()->GetResourceTracker()->AddRefResource(config_);
 }
 
 Audio::~Audio() {
+#if defined(OS_NACL)
+  // Invoke StopPlayback() to ensure audio back-end has a chance to send the
+  // escape value over the sync socket, which will terminate the client side
+  // audio callback loop.  This is required for NaCl Plugins that can't escape
+  // by shutting down the sync_socket.
+  StopPlayback();
+#endif

[bbudge, 2012/09/19 22:31:31]

Thanks for the fix. I just noticed it on Windows (doesn't work after stopping).
BTW, this could have been its own CL.

   PpapiGlobals::Get()->GetResourceTracker()->ReleaseResource(config_);
 }
 
 PPB_Audio_API* Audio::AsPPB_Audio_API() {
   return this;
 }
 
 PP_Resource Audio::GetCurrentConfig() {
   // AddRef for the caller.
   PpapiGlobals::Get()->GetResourceTracker()->AddRefResource(config_);
@@ skipping 245 matching lines @@
     static_cast<Audio*>(enter.object())->SetStreamInfo(
         enter.resource()->pp_instance(), handle.shmem(),
         media::PacketSizeInBytes(handle.size()),
         IPC::PlatformFileForTransitToPlatformFile(socket_handle.descriptor()),
         config.object()->GetSampleFrameCount());
   }
 }
 
 }  // namespace proxy
 }  // namespace ppapi