Change VideoRtpReceiver to create remote VideoTrack and VideoTrackSource.

webrtc/api/videotracksource.h

Index: webrtc/api/videotracksource.h
diff --git a/webrtc/api/videotracksource.h b/webrtc/api/videotracksource.h
index 59dae42950b9597d958190dc5e2c980646940263..5e2437c6b0d113f251f6332b8305c61e054eb47d 100644
--- a/webrtc/api/videotracksource.h
+++ b/webrtc/api/videotracksource.h
@@ -24,21 +24,27 @@ class VideoTrackSource : public Notifier<VideoTrackSourceInterface> {
   VideoTrackSource(rtc::VideoSourceInterface<cricket::VideoFrame>* source,
                    rtc::Thread* worker_thread,
                    bool remote);
-
   void SetState(SourceState new_state);
+  // OnSourceDestroyed clears this instance pointer to |source_|. It is useful
+  // when the underlying rtc::VideoSourceInterface is destroyed before the
+  // reference counted VideoTrackSource.
+  void OnSourceDestroyed();
+
   SourceState state() const override { return state_; }
   bool remote() const override { return remote_; }
 
   void Stop() override{};
   void Restart() override{};
 
-  virtual bool is_screencast() const { return false; };
-  virtual bool needs_denoising() const { return false; };
+  virtual bool is_screencast() const { return false; }
+  virtual bool needs_denoising() const { return false; }
 
   void AddOrUpdateSink(rtc::VideoSinkInterface<cricket::VideoFrame>* sink,
                        const rtc::VideoSinkWants& wants) override;
   void RemoveSink(rtc::VideoSinkInterface<cricket::VideoFrame>* sink) override;
 
+  cricket::VideoCapturer* GetVideoCapturer() override { return nullptr; }

[pthatcher1, 2016/03/09 18:16:49]

Why do we still need this?

[perkj_webrtc, 2016/03/09 21:22:59]

It is part of the public interface until nisse can land
https://codereview.webrtc.org/1766653002/

+
  protected:
   rtc::Thread* worker_thread() { return worker_thread_; }