Chromium Code Reviews

Unified Diff: talk/media/webrtc/webrtcvideoengine2.cc

Issue 1574963002: Deleted renderer-related SetSize methods, and all uses. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
« talk/media/base/videorenderer.h ('K') | « talk/media/base/videorenderer.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: talk/media/webrtc/webrtcvideoengine2.cc
diff --git a/talk/media/webrtc/webrtcvideoengine2.cc b/talk/media/webrtc/webrtcvideoengine2.cc
index 55c07426d090598877f1c4a6656d5936f20d5c99..97764acad0e786268d8bde63455745e09771c909 100644
--- a/talk/media/webrtc/webrtcvideoengine2.cc
+++ b/talk/media/webrtc/webrtcvideoengine2.cc
@@ -2494,9 +2494,6 @@ VideoRenderer* WebRtcVideoChannel2::WebRtcVideoReceiveStream::GetRenderer() {
void WebRtcVideoChannel2::WebRtcVideoReceiveStream::SetSize(int width,
pbos-webrtc 2016/01/11 10:25:01 I think you should be able to remove this method c
nisse-webrtc 2016/01/11 12:26:06 I'm deleting the SetSize method. But last_width_ a
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;
}
« talk/media/base/videorenderer.h ('K') | « talk/media/base/videorenderer.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine