Deleted renderer-related SetSize methods, and all uses.

talk/media/webrtc/webrtcvideoengine2.cc

 /*
  * libjingle
  * Copyright 2014 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 2473 matching lines @@
   }
 }
 
 VideoRenderer* WebRtcVideoChannel2::WebRtcVideoReceiveStream::GetRenderer() {
   // TODO(pbos): Remove GetRenderer and all uses of it, it's thread-unsafe by
   // design.
   rtc::CritScope crit(&renderer_lock_);
   return renderer_;
 }
 
 void WebRtcVideoChannel2::WebRtcVideoReceiveStream::SetSize(int width,

[pbos-webrtc, 2016/01/11 10:25:01]

I think you should be able to remove this method completely, as well as
last_width_ and last_height_.

[nisse-webrtc, 2016/01/11 12:26:06]

I'm deleting the SetSize method. But last_width_ and last_height_ are used to
populate the VideoReceiverInfo struct, so it's not obvious to me that they are
unused.

                                                             int height) {
   rtc::CritScope crit(&renderer_lock_);
-  if (!renderer_->SetSize(width, height, 0)) {
-    LOG(LS_ERROR) << "Could not set renderer size.";
-  }
   last_width_ = width;
   last_height_ = height;
 }
 
 std::string
 WebRtcVideoChannel2::WebRtcVideoReceiveStream::GetCodecNameFromPayloadType(
     int payload_type) {
   for (const webrtc::VideoReceiveStream::Decoder& decoder : config_.decoders) {
     if (decoder.payload_type == payload_type) {
       return decoder.payload_name;
@@ skipping 153 matching lines @@
       video_codecs[i].rtx_payload_type = rtx_mapping[video_codecs[i].codec.id];
     }
   }
 
   return video_codecs;
 }
 
 }  // namespace cricket
 
 #endif  // HAVE_WEBRTC_VIDEO