New rtc::VideoSinkInterface.

talk/media/base/capturemanager.h

 /*
  * libjingle
  * Copyright 2012 Google Inc.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are met:
  *
  *  1. Redistributions of source code must retain the above copyright notice,
  *     this list of conditions and the following disclaimer.
  *  2. Redistributions in binary form must reproduce the above copyright notice,
@@ skipping 61 matching lines @@
 
   // Possibly restarts the capturer. If |options| is set to kRequestRestart,
   // the CaptureManager chooses whether this request can be handled with the
   // current state or if a restart is actually needed. If |options| is set to
   // kForceRestart, the capturer is restarted.
   virtual bool RestartVideoCapture(VideoCapturer* video_capturer,
                                    const VideoFormat& previous_format,
                                    const VideoFormat& desired_format,
                                    RestartOptions options);
 
-  virtual bool AddVideoRenderer(VideoCapturer* video_capturer,
-                                VideoRenderer* video_renderer);
-  virtual bool RemoveVideoRenderer(VideoCapturer* video_capturer,
-                                   VideoRenderer* video_renderer);
+  // TODO(nisse): Return values are mostly useless. Currently, these
+  // methods tolerate nullptr inputs, probably not needed.
+  virtual bool AddVideoSink(VideoCapturer* video_capturer,
+                            rtc::VideoSinkInterface<VideoFrame>* sink);

[pthatcher1, 2016/01/27 01:11:03]

Can you just make these void?  You can either return true in
ChannelManager::AddVideoRenderer (the only place that uses it) or change it also
to be void (since the only place that calls ChannelManager::AddVideoRenderer
ignores the return value).

[nisse-webrtc, 2016/01/27 09:08:51]

Done, deleting return values also for the ChannelManager methods.

+  virtual bool RemoveVideoSink(VideoCapturer* video_capturer,
+                               rtc::VideoSinkInterface<VideoFrame>* sink);
 
   sigslot::repeater2<VideoCapturer*, CaptureState> SignalCapturerStateChange;
 
  private:
   typedef std::map<VideoCapturer*, VideoCapturerState*> CaptureStates;
 
   bool IsCapturerRegistered(VideoCapturer* video_capturer) const;
   bool RegisterVideoCapturer(VideoCapturer* video_capturer);
   void UnregisterVideoCapturer(VideoCapturerState* capture_state);
 
   bool StartWithBestCaptureFormat(VideoCapturerState* capture_info,
                                   VideoCapturer* video_capturer);
 
   VideoCapturerState* GetCaptureState(VideoCapturer* video_capturer) const;
   CaptureRenderAdapter* GetAdapter(VideoCapturer* video_capturer) const;
 
   rtc::ThreadChecker thread_checker_;
   CaptureStates capture_states_;
 };
 
 }  // namespace cricket
 
 #endif  // TALK_MEDIA_BASE_CAPTUREMANAGER_H_