Set audio format before adding pepper sink to audio track.

content/renderer/pepper/pepper_media_stream_audio_track_host.h

 // Copyright 2014 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_MEDIA_STREAM_AUDIO_TRACK_HOST_H_
 #define CONTENT_RENDERER_PEPPER_PEPPER_MEDIA_STREAM_AUDIO_TRACK_HOST_H_
 
 #include <deque>
 
 #include "base/compiler_specific.h"
@@ skipping 35 matching lines @@
     void EnqueueBuffer(int32_t index);
 
     // This function is called on the main thread.
     int32_t Configure(int32_t number_of_buffers, int32_t duration,
                       const ppapi::host::ReplyMessageContext& context);
 
     // Send a reply to the currently pending |Configure()| request.
     void SendConfigureReply(int32_t result);
 
    private:
+    // To allow |DidConnectPendingHostToResource()| to call |OnSetFormat()|.
+    friend class PepperMediaStreamAudioTrackHost;

[dmichael (off chromium), 2015/01/20 17:07:03]

I think I'd rather you either:
1) Make OnSetFormat public (OnData can be as well)
or
2) At the place you need to call OnSetFormat,
static_cast<MediaStreamAudioSink*>(host_)

Either option would better preserve encapsulation of AudioSink.

[Anand Mistry (off Chromium), 2015/01/27 02:12:10]

Done #1.

+
     // Initializes buffers on the main thread.
     void SetFormatOnMainThread(int bytes_per_second, int bytes_per_frame);
 
     void InitBuffers();
 
     // Send enqueue buffer message on the main thread.
     void SendEnqueueBufferMessageOnMainThread(int32_t index,
                                               int32_t buffers_generation);
 
     // MediaStreamAudioSink overrides:
@@ skipping 96 matching lines @@
   bool connected_;
 
   AudioSink audio_sink_;
 
   DISALLOW_COPY_AND_ASSIGN(PepperMediaStreamAudioTrackHost);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_RENDERER_PEPPER_PEPPER_MEDIA_STREAM_AUDIO_TRACK_HOST_H_