Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(569)

Unified Diff: talk/app/webrtc/java/jni/peerconnection_jni.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. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | talk/app/webrtc/mediastreaminterface.h » ('j') | talk/media/base/videorenderer.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: talk/app/webrtc/java/jni/peerconnection_jni.cc
diff --git a/talk/app/webrtc/java/jni/peerconnection_jni.cc b/talk/app/webrtc/java/jni/peerconnection_jni.cc
index c31859def809318b3bf61e400504dc112bce1478..bbaef7c98d4fc1d037b0e2390e0b64c41bc39a13 100644
--- a/talk/app/webrtc/java/jni/peerconnection_jni.cc
+++ b/talk/app/webrtc/java/jni/peerconnection_jni.cc
@@ -740,23 +740,14 @@ class VideoRendererWrapper : public VideoRendererInterface {
virtual ~VideoRendererWrapper() {}
- // This wraps VideoRenderer which still has SetSize.
void RenderFrame(const cricket::VideoFrame* video_frame) override {
ScopedLocalRefFrame local_ref_frame(AttachCurrentThreadIfNeeded());
- const cricket::VideoFrame* frame =
- video_frame->GetCopyWithRotationApplied();
- if (width_ != frame->GetWidth() || height_ != frame->GetHeight()) {
- width_ = frame->GetWidth();
- height_ = frame->GetHeight();
- renderer_->SetSize(width_, height_, 0);
- }
- renderer_->RenderFrame(frame);
+ renderer_->RenderFrame(video_frame->GetCopyWithRotationApplied());
}
private:
explicit VideoRendererWrapper(cricket::VideoRenderer* renderer)
- : width_(0), height_(0), renderer_(renderer) {}
- int width_, height_;
+ : renderer_(renderer) {}
scoped_ptr<cricket::VideoRenderer> renderer_;
};
« no previous file with comments | « no previous file | talk/app/webrtc/mediastreaminterface.h » ('j') | talk/media/base/videorenderer.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698