commit 2bde85046ad85757279d7bd0bf701ee0f70a0272
author Sami Kalliomäki <sakal@webrtc.org> Thu Feb 15 13:58:15 2018 +0100
committer Commit Bot <commit-bot@chromium.org> Thu Feb 15 14:26:45 2018 +0000
tree 00fdcb3e38e2ef1e2225f6461e851bd9eb3effe3
parent 2c599d663d45dd760aac8f0b37c1d65d52c19698

Fix memory leak in NativeToJavaVideoFrame.

This method used to just wrap frame when passed a native frame and
create a new one when passed non-native frame. This caused a memory
leak when a new frame was returned because the caller didn't release
the frame. Now the method always returns a new frame and the caller is
responsible for releasing it.

Bug: webrtc:8892, b/72675429
Change-Id: I06d67a6ed4c059cae1d709c51b0266f9c72fef1a
Reviewed-on: https://webrtc-review.googlesource.com/53840
Reviewed-by: Anders Carlsson <andersc@webrtc.org>
Commit-Queue: Sami Kalliomäki <sakal@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#22033}

sdk/android/api/org/webrtc/VideoFrame.java

diff --git a/sdk/android/api/org/webrtc/VideoFrame.java b/sdk/android/api/org/webrtc/VideoFrame.java
index 3e8a1e4..3b73543 100644
--- a/sdk/android/api/org/webrtc/VideoFrame.java
+++ b/sdk/android/api/org/webrtc/VideoFrame.java
@@ -177,6 +177,7 @@
     buffer.retain();
   }
 
+  @CalledByNative
   public void release() {
     buffer.release();
   }