From c614823fa35cd25df3c3cbcf2872e279098c7025 Mon Sep 17 00:00:00 2001 From: sakal Date: Tue, 22 Aug 2017 00:21:48 -0700 Subject: [PATCH] Remove unnecessary release call in EglRenderer. VideoSink is not assumed to take ownership of the frame, so it is not necessary to release it unless it is retained for future use. BUG=webrtc:7749 Review-Url: https://codereview.webrtc.org/3003563002 Cr-Commit-Position: refs/heads/master@{#19440} --- webrtc/sdk/android/api/org/webrtc/EglRenderer.java | 1 - 1 file changed, 1 deletion(-) diff --git a/webrtc/sdk/android/api/org/webrtc/EglRenderer.java b/webrtc/sdk/android/api/org/webrtc/EglRenderer.java index 083b1ca076..08d66a9276 100644 --- a/webrtc/sdk/android/api/org/webrtc/EglRenderer.java +++ b/webrtc/sdk/android/api/org/webrtc/EglRenderer.java @@ -430,7 +430,6 @@ public class EglRenderer implements VideoRenderer.Callbacks, VideoSink { synchronized (handlerLock) { if (renderThreadHandler == null) { logD("Dropping frame - Not initialized or already released."); - frame.release(); return; } synchronized (frameLock) {