Deleted renderer-related SetSize methods, and all uses.

talk/media/base/videorenderer.h

Index: talk/media/base/videorenderer.h
diff --git a/talk/media/base/videorenderer.h b/talk/media/base/videorenderer.h
index 0a0ee5181712596e2afe1463efef9b6e356b0515..0b159f3af7864ff2db12290f570f6ef99fdc1847 100644
--- a/talk/media/base/videorenderer.h
+++ b/talk/media/base/videorenderer.h
@@ -42,11 +42,6 @@ class VideoFrame;
 class VideoRenderer {

[nisse-webrtc, 2016/01/11 08:30:52]

Except for the RenderFrame return value, and the protection of the destructor,
this class declaration is identical to webrtc::VideoRendererInterface. Can we
simply delete one of the classes (and if so, which one to keep)?

I'd also prefer to rename it to something like VideoFrameSink, but that's less
important.

[tommi, 2016/01/11 10:17:42]

keep the webrtc:: one. It's a part of the public interface but cricket:: is an
implementation detail.

  public:
   virtual ~VideoRenderer() {}
-  // Called when the video has changed size. This is also used as an
-  // initialization method to set the UI size before any video frame
-  // rendered. webrtc::ExternalRenderer's FrameSizeChange will invoke this when
-  // it's called or later when a VideoRenderer is attached.
-  virtual bool SetSize(int width, int height, int reserved) = 0;
   // Called when a new frame is available for display.
   virtual bool RenderFrame(const VideoFrame *frame) = 0;