X-Git-Url: http://review.tizen.org/git/?a=blobdiff_plain;f=src%2Fcontent%2Frenderer%2Fmedia%2Fremote_media_stream_impl.h;h=0c230bac73fb3b43d173eb26032c2630efe21799;hb=1afa4dd80ef85af7c90efaea6959db1d92330844;hp=6b1554aff066465eeb37da516759f4e789031bd2;hpb=90762837333c13ccf56f2ad88e4481fc71e8d281;p=platform%2Fframework%2Fweb%2Fcrosswalk.git

diff --git a/src/content/renderer/media/remote_media_stream_impl.h b/src/content/renderer/media/remote_media_stream_impl.h
index 6b1554a..0c230ba 100644
--- a/src/content/renderer/media/remote_media_stream_impl.h
+++ b/src/content/renderer/media/remote_media_stream_impl.h
@@ -8,8 +8,10 @@
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "base/memory/ref_counted.h"
-#include "base/memory/scoped_vector.h"
-#include "base/threading/non_thread_safe.h"
+#include "base/memory/scoped_ptr.h"
+#include "base/memory/weak_ptr.h"
+#include "base/single_thread_task_runner.h"
+#include "base/threading/thread_checker.h"
 #include "content/common/content_export.h"
 #include "third_party/libjingle/source/talk/app/webrtc/mediastreaminterface.h"
 #include "third_party/WebKit/public/platform/WebMediaStream.h"
@@ -18,29 +20,82 @@
 
 namespace content {
 
-class RemoteMediaStreamTrackAdapter;
+class RemoteAudioTrackAdapter;
+class RemoteVideoTrackAdapter;
 
 // RemoteMediaStreamImpl serves as a container and glue between remote webrtc
 // MediaStreams and WebKit MediaStreams. For each remote MediaStream received
 // on a PeerConnection a RemoteMediaStreamImpl instance is created and
 // owned by RtcPeerConnection.
-class CONTENT_EXPORT RemoteMediaStreamImpl
-    : NON_EXPORTED_BASE(public webrtc::ObserverInterface),
-      NON_EXPORTED_BASE(public base::NonThreadSafe) {
+class CONTENT_EXPORT RemoteMediaStreamImpl {
  public:
-  explicit RemoteMediaStreamImpl(
+  RemoteMediaStreamImpl(
+      const scoped_refptr<base::SingleThreadTaskRunner>& main_thread,
       webrtc::MediaStreamInterface* webrtc_stream);
-  virtual ~RemoteMediaStreamImpl();
+  ~RemoteMediaStreamImpl();
 
   const blink::WebMediaStream& webkit_stream() { return webkit_stream_; }
+  const scoped_refptr<webrtc::MediaStreamInterface>& webrtc_stream() {
+    return observer_->stream();
+  }
 
  private:
-  // webrtc::ObserverInterface implementation.
-  virtual void OnChanged() OVERRIDE;
+  typedef std::vector<scoped_refptr<RemoteAudioTrackAdapter>>
+      RemoteAudioTrackAdapters;
+  typedef std::vector<scoped_refptr<RemoteVideoTrackAdapter>>
+      RemoteVideoTrackAdapters;
 
-  scoped_refptr<webrtc::MediaStreamInterface> webrtc_stream_;
-  ScopedVector<RemoteMediaStreamTrackAdapter> video_track_observers_;
-  ScopedVector<RemoteMediaStreamTrackAdapter> audio_track_observers_;
+  void InitializeOnMainThread(const std::string& label);
+
+  class Observer
+      : NON_EXPORTED_BASE(public webrtc::ObserverInterface),
+        public base::RefCountedThreadSafe<Observer> {
+   public:
+    Observer(const base::WeakPtr<RemoteMediaStreamImpl>& media_stream,
+             const scoped_refptr<base::SingleThreadTaskRunner>& main_thread,
+             webrtc::MediaStreamInterface* webrtc_stream);
+
+    const scoped_refptr<webrtc::MediaStreamInterface>& stream() const {
+      return webrtc_stream_;
+    }
+
+    const scoped_refptr<base::SingleThreadTaskRunner>& main_thread() const {
+      return main_thread_;
+    }
+
+    void InitializeOnMainThread(const std::string& label);
+
+    // Uninitializes the observer, unregisteres from receiving notifications
+    // and releases the webrtc stream.
+    // Note: Must be called from the main thread before releasing the main
+    // reference.
+    void Unregister();
+
+   private:
+    friend class base::RefCountedThreadSafe<Observer>;
+    ~Observer() override;
+
+    // webrtc::ObserverInterface implementation.
+    void OnChanged() override;
+
+    void OnChangedOnMainThread(
+        scoped_ptr<RemoteAudioTrackAdapters> audio_tracks,
+        scoped_ptr<RemoteVideoTrackAdapters> video_tracks);
+
+    base::WeakPtr<RemoteMediaStreamImpl> media_stream_;
+    const scoped_refptr<base::SingleThreadTaskRunner> main_thread_;
+    scoped_refptr<webrtc::MediaStreamInterface> webrtc_stream_;
+  };
+
+  void OnChanged(scoped_ptr<RemoteAudioTrackAdapters> audio_tracks,
+                 scoped_ptr<RemoteVideoTrackAdapters> video_tracks);
+
+  const scoped_refptr<base::SingleThreadTaskRunner> signaling_thread_;
+  base::WeakPtrFactory<RemoteMediaStreamImpl> weak_factory_;
+  const scoped_refptr<Observer> observer_;  // must be after weak_factory_
+
+  RemoteVideoTrackAdapters video_track_observers_;
+  RemoteAudioTrackAdapters audio_track_observers_;
   blink::WebMediaStream webkit_stream_;
 
   DISALLOW_COPY_AND_ASSIGN(RemoteMediaStreamImpl);