Support for remote audio into tracks

talk/app/webrtc/remoteaudiosource.h

Index: talk/app/webrtc/remoteaudiosource.h
diff --git a/talk/app/webrtc/remoteaudiosource.h b/talk/app/webrtc/remoteaudiosource.h
index e49aca56842091e3c29e17c6558c7c727ddc9e11..f518d9b607b8b5158d9b8d9120033401024f1a50 100644
--- a/talk/app/webrtc/remoteaudiosource.h
+++ b/talk/app/webrtc/remoteaudiosource.h
@@ -29,36 +29,65 @@
 #define TALK_APP_WEBRTC_REMOTEAUDIOSOURCE_H_
 
 #include <list>
+#include <string>
 
 #include "talk/app/webrtc/mediastreaminterface.h"
 #include "talk/app/webrtc/notifier.h"
+#include "talk/media/base/audiorenderer.h"
+#include "webrtc/audio/audio_sink.h"
+#include "webrtc/base/criticalsection.h"
+
+namespace rtc {
+struct Message;
+class Thread;
+}  // namespace rtc
 
 namespace webrtc {
 
-using webrtc::AudioSourceInterface;
+class AudioProviderInterface;
 
 // This class implements the audio source used by the remote audio track.
 class RemoteAudioSource : public Notifier<AudioSourceInterface> {
  public:
   // Creates an instance of RemoteAudioSource.
-  static rtc::scoped_refptr<RemoteAudioSource> Create();
+  static rtc::scoped_refptr<RemoteAudioSource> Create(
+      uint32_t ssrc,
+      AudioProviderInterface* provider);
+
+  // MediaSourceInterface implementation.
+  MediaSourceInterface::SourceState state() const override;
+
+  void AddSink(AudioTrackSinkInterface* sink);
+  void RemoveSink(AudioTrackSinkInterface* sink);
 
  protected:
   RemoteAudioSource();
-  virtual ~RemoteAudioSource();
+  ~RemoteAudioSource() override;
+
+  // Post construction initialize where we can do things like save a reference
+  // to ourselves (need to be fully constructed).
+  void Initialize(uint32_t ssrc, AudioProviderInterface* provider);
 
  private:
   typedef std::list<AudioObserver*> AudioObserverList;
 
-  // MediaSourceInterface implementation.
-  MediaSourceInterface::SourceState state() const override;
-
   // AudioSourceInterface implementation.
   void SetVolume(double volume) override;
   void RegisterAudioObserver(AudioObserver* observer) override;
   void UnregisterAudioObserver(AudioObserver* observer) override;
 
+  class Sink;
+  void OnData(const AudioSinkInterface::Data& audio);
+  void OnAudioProviderGone();
+
+  class MessageHandler;
+  void OnMessage(rtc::Message* msg);
+
   AudioObserverList audio_observers_;
+  rtc::CriticalSection sink_lock_;

[the sun, 2015/12/11 16:32:04]

Add thread checkers to document the expected usage by different threads here.

[tommi (sloooow) - chröme, 2015/12/11 17:51:18]

I'm using main_thread_ as my thread checker and there's a thread check in almost
every method. For the ones where there isn't, I'll either add one or add more
documentation.

[the sun, 2015/12/11 19:46:28]

Acknowledged.

+  std::list<AudioTrackSinkInterface*> sinks_;
+  rtc::Thread* const main_thread_;
+  SourceState state_;
 };
 
 }  // namespace webrtc