Plumb render view ID from audio-related code in renderer through IPCs to AudioRendererHost in brows…

content/renderer/pepper/pepper_platform_audio_input_impl.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 CONTENT_RENDERER_PEPPER_PEPPER_PLATFORM_AUDIO_INPUT_IMPL_H_
 #define CONTENT_RENDERER_PEPPER_PEPPER_PLATFORM_AUDIO_INPUT_IMPL_H_
 
 #include <string>
 
 #include "base/basictypes.h"
@@ skipping 21 matching lines @@
 // I/O thread.
 
 class PepperPlatformAudioInputImpl
     : public webkit::ppapi::PluginDelegate::PlatformAudioInput,
       public media::AudioInputIPCDelegate,
       public base::RefCountedThreadSafe<PepperPlatformAudioInputImpl> {
  public:
   // Factory function, returns NULL on failure. StreamCreated() will be called
   // when the stream is created.
   static PepperPlatformAudioInputImpl* Create(
+      int render_view_id,

[jamesr, 2012/10/29 22:51:02]

you don't need this - you're passing in a WeakPtr to a PepperPluginDelegateImpl
which provides a RenderViewImpl* getter.  Just grab the ID off that pointer
if/when you need it

       const base::WeakPtr<PepperPluginDelegateImpl>& plugin_delegate,
       const std::string& device_id,
       int sample_rate,
       int frames_per_buffer,
       webkit::ppapi::PluginDelegate::PlatformAudioInputClient* client);
 
   // PlatformAudioInput implementation (called on main thread).
   virtual void StartCapture() OVERRIDE;
   virtual void StopCapture() OVERRIDE;
   virtual void ShutDown() OVERRIDE;
 
   // media::AudioInputIPCDelegate.
   virtual void OnStreamCreated(base::SharedMemoryHandle handle,
                                base::SyncSocket::Handle socket_handle,
                                int length) OVERRIDE;
   virtual void OnVolume(double volume) OVERRIDE;
   virtual void OnStateChanged(
       media::AudioInputIPCDelegate::State state) OVERRIDE;
   virtual void OnDeviceReady(const std::string&) OVERRIDE;
   virtual void OnIPCClosed() OVERRIDE;
 
  protected:
   virtual ~PepperPlatformAudioInputImpl();
 
  private:
   friend class base::RefCountedThreadSafe<PepperPlatformAudioInputImpl>;
 
-  PepperPlatformAudioInputImpl();
+  explicit PepperPlatformAudioInputImpl(int render_view_id);

[jamesr, 2012/10/29 22:51:02]

Unnecessary

 
   bool Initialize(
       const base::WeakPtr<PepperPluginDelegateImpl>& plugin_delegate,
       const std::string& device_id,
       int sample_rate,
       int frames_per_buffer,
       webkit::ppapi::PluginDelegate::PlatformAudioInputClient* client);
 
   // I/O thread backends to above functions.
   void InitializeOnIOThread(int session_id);
   void StartCaptureOnIOThread();
   void StopCaptureOnIOThread();
   void ShutDownOnIOThread();
 
   void OnDeviceOpened(int request_id,
                       bool succeeded,
                       const std::string& label);
   void CloseDevice();
   void NotifyStreamCreationFailed();
 
   // The client to notify when the stream is created. THIS MUST ONLY BE
   // ACCESSED ON THE MAIN THREAD.
   webkit::ppapi::PluginDelegate::PlatformAudioInputClient* client_;
 
   // Used to send/receive IPC. THIS MUST ONLY BE ACCESSED ON THE
   // I/O thread except to send messages and get the message loop.
-  media::AudioInputIPC* ipc_;
+  scoped_ptr<media::AudioInputIPC> ipc_;
 
   // Our ID on the MessageFilter. THIS MUST ONLY BE ACCESSED ON THE I/O THREAD
   // or else you could race with the initialize function which sets it.
   int32 stream_id_;
 
   base::MessageLoopProxy* main_message_loop_proxy_;
 
   // THIS MUST ONLY BE ACCESSED ON THE MAIN THREAD.
   base::WeakPtr<PepperPluginDelegateImpl> plugin_delegate_;
 
   // The unique ID to identify the opened device. THIS MUST ONLY BE ACCESSED ON
   // THE MAIN THREAD.
   std::string label_;
 
   // Whether ShutDownOnIOThread() has been called. THIS MUST ONLY BE ACCESSED ON
   // THE I/O THREAD.
   bool shutdown_called_;
 
   // Initialized on the main thread and accessed on the I/O thread afterwards.
   media::AudioParameters params_;
 
-  DISALLOW_COPY_AND_ASSIGN(PepperPlatformAudioInputImpl);
+  DISALLOW_IMPLICIT_CONSTRUCTORS(PepperPlatformAudioInputImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_RENDERER_PEPPER_PEPPER_PLATFORM_AUDIO_INPUT_IMPL_H_