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

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: Rebase 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/test/fakevideotrackrenderer.h » ('j') | no next file with comments »
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 6059623daa753293b21477e20202c927ddbb084d..1e138f6b945962467e1735b2b50ec26582db07ac 100644
--- a/talk/app/webrtc/java/jni/peerconnection_jni.cc
+++ b/talk/app/webrtc/java/jni/peerconnection_jni.cc
@@ -724,23 +724,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/test/fakevideotrackrenderer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698