Bug 1351896 - Fix Android JNI error while running android native vp8 encoder; r?jolin
MozReview-Commit-ID: Gm9WwfgMjh8
--- a/media/webrtc/signaling/src/media-conduit/WebrtcMediaCodecVP8VideoCodec.cpp
+++ b/media/webrtc/signaling/src/media-conduit/WebrtcMediaCodecVP8VideoCodec.cpp
@@ -455,40 +455,32 @@ public:
void ReleaseOutputBuffer(int32_t index, bool flag) {
mCoder->ReleaseOutputBuffer(index, flag);
}
jobjectArray GetInputBuffers() {
JNIEnv* env = jsjni_GetJNIForThread();
- if (mInputBuffers) {
- env->DeleteGlobalRef(mInputBuffers);
- }
-
nsresult res;
jni::ObjectArray::LocalRef inputBuffers;
res = mCoder->GetInputBuffers(&inputBuffers);
mInputBuffers = (jobjectArray) env->NewGlobalRef(inputBuffers.Get());
if (NS_FAILED(res)) {
CSFLogDebug(logTag, "WebrtcAndroidMediaCodec::%s, GetInputBuffers return err = %d",
__FUNCTION__, (int)res);
return nullptr;
}
return mInputBuffers;
}
jobjectArray GetOutputBuffers() {
JNIEnv* env = jsjni_GetJNIForThread();
- if (mOutputBuffers) {
- env->DeleteGlobalRef(mOutputBuffers);
- }
-
nsresult res;
jni::ObjectArray::LocalRef outputBuffers;
res = mCoder->GetOutputBuffers(&outputBuffers);
mOutputBuffers = (jobjectArray) env->NewGlobalRef(outputBuffers.Get());
if (NS_FAILED(res)) {
CSFLogDebug(logTag, "WebrtcAndroidMediaCodec::%s, GetOutputBuffers return err = %d",
__FUNCTION__, (int)res);
return nullptr;