Make VideoTrack and VideoTrackRenderers implement rtc::VideoSourceInterface.

webrtc/api/test/fakevideotrackrenderer.h

Index: webrtc/api/test/fakevideotrackrenderer.h
diff --git a/webrtc/api/test/fakevideotrackrenderer.h b/webrtc/api/test/fakevideotrackrenderer.h
index 816b44744fc2acc35e6d9e62119fab5702331552..bbd6b55c52eff5025e7ee0c89c570727df2e53f5 100644
--- a/webrtc/api/test/fakevideotrackrenderer.h
+++ b/webrtc/api/test/fakevideotrackrenderer.h
@@ -16,19 +16,20 @@
 
 namespace webrtc {
 
-class FakeVideoTrackRenderer : public VideoRendererInterface {
+class FakeVideoTrackRenderer
+    : public rtc::VideoSinkInterface<cricket::VideoFrame> {

[perkj_webrtc, 2016/02/12 14:40:14]

Se should continue testing both until all clients has been updated. ie -
android, chrome etc.

[nisse-webrtc, 2016/02/15 08:54:48]

Unclear what you suggest...

I think it should be possible to leave this class unchanged, until the backwards
compatibility methods AddSink/RemoveSink are deleted (VideoRendererInterface
inherits VideoSinkInterface). Would that help clients?

[perkj_webrtc, 2016/02/15 12:36:27]

Sorry- I meant that currently clients such as chrome uses AddRenderer -
RemoveRenderer - therefor- as long as those methods has not been removed, we
should keep the old tests that tests AddRenderer and RemoveRenderer.
If we add new methods, AddSink and RemoveSink, there should be tests for them
too.

[nisse-webrtc, 2016/02/15 14:51:53]

Made two variants of the VideoTrack.RenderVideo test case and the
FakeVideoTrackRenderer class.

  public:
   FakeVideoTrackRenderer(VideoTrackInterface* video_track)
       : video_track_(video_track), last_frame_(NULL) {
-    video_track_->AddRenderer(this);
+    video_track_->AddOrUpdateSink(this, rtc::VideoSinkWants());
   }
   ~FakeVideoTrackRenderer() {
-    video_track_->RemoveRenderer(this);
+    video_track_->RemoveSink(this);
   }
 
-  virtual void RenderFrame(const cricket::VideoFrame* video_frame) override {
-    last_frame_ = const_cast<cricket::VideoFrame*>(video_frame);
-    fake_renderer_.RenderFrame(video_frame);
+  virtual void OnFrame(const cricket::VideoFrame& video_frame) override {
+    last_frame_ = const_cast<cricket::VideoFrame*>(&video_frame);
+    fake_renderer_.RenderFrame(&video_frame);
   }
 
   int errors() const { return fake_renderer_.errors(); }