Cleanup sdk android code from using VideoEncoderFactory::CreateVideoEncoder

To force webrtc::Environment propagation through android VideoEncoderFactory wrappers

Bug: webrtc:15860
Change-Id: Icad7249b47a5c738b089e09714c729f91eddaf5d
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/342960
Commit-Queue: Danil Chapovalov <[email protected]>
Reviewed-by: Sergey Silkin <[email protected]>
Cr-Commit-Position: refs/heads/main@{#41936}

Author: DanilChapovalov

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

@@ -11,17 +11,11 @@
 package org.webrtc;
 
 public class LibaomAv1Encoder extends WrappedNativeVideoEncoder {
-  @Override
-  public long createNativeVideoEncoder() {
-    return nativeCreateEncoder();
-  }
-
   @Override
   public long createNative(long webrtcEnvRef) {
     return nativeCreate(webrtcEnvRef);
   }
 
-  static native long nativeCreateEncoder();
   static native long nativeCreate(long webrtcEnvRef);
 
   @Override

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

@@ -11,17 +11,11 @@
 package org.webrtc;
 
 public class LibvpxVp8Encoder extends WrappedNativeVideoEncoder {
-  @Override
-  public long createNativeVideoEncoder() {
-    return nativeCreateEncoder();
-  }
-
   @Override
   public long createNative(long webrtcEnvRef) {
     return nativeCreate(webrtcEnvRef);
   }
 
-  static native long nativeCreateEncoder();
   static native long nativeCreate(long webrtcEnvRef);
 
   @Override

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

@@ -11,17 +11,11 @@
 package org.webrtc;
 
 public class LibvpxVp9Encoder extends WrappedNativeVideoEncoder {
-  @Override
-  public long createNativeVideoEncoder() {
-    return nativeCreateEncoder();
-  }
-
   @Override
   public long createNative(long webrtcEnvRef) {
     return nativeCreate(webrtcEnvRef);
   }
 
-  static native long nativeCreateEncoder();
   static native long nativeCreate(long webrtcEnvRef);
 
   @Override

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

@@ -32,11 +32,6 @@ public VideoEncoder createEncoder(VideoCodecInfo info) {
     }
 
     return new WrappedNativeVideoEncoder() {
-      @Override
-      public long createNativeVideoEncoder() {
-        return nativeCreateEncoder(nativeFactory, info);
-      }
-
       @Override
       public long createNative(long webrtcEnvRef) {
         return nativeCreate(nativeFactory, webrtcEnvRef, info);
@@ -56,8 +51,6 @@ public VideoCodecInfo[] getSupportedCodecs() {
 
   private static native long nativeCreateFactory();
 
-  private static native long nativeCreateEncoder(long factory, VideoCodecInfo videoCodecInfo);
-
   private static native boolean nativeIsSupported(long factory, VideoCodecInfo info);
 
   private static native long nativeCreate(long factory, long webrtcEnvRef, VideoCodecInfo info);

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

@@ -322,16 +322,9 @@ public interface Callback {
    * the Java interface methods declared below should thus throw an
    * UnsupportedOperationException.
    */
-
-  @CalledByNative
-  @Deprecated
-  default long createNativeVideoEncoder() {
-    return 0;
-  }
-
   @CalledByNative
   default long createNative(long webrtcEnvRef) {
-    return createNativeVideoEncoder();
+    return 0;
   }
 
   /**

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

@@ -22,11 +22,6 @@ public VideoEncoderFallback(VideoEncoder fallback, VideoEncoder primary) {
     this.primary = primary;
   }
 
-  @Override
-  public long createNativeVideoEncoder() {
-    return nativeCreateEncoder(fallback, primary);
-  }
-
   @Override
   public long createNative(long webrtcEnvRef) {
     return nativeCreate(webrtcEnvRef, fallback, primary);
@@ -37,7 +32,6 @@ public boolean isHardwareEncoder() {
     return primary.isHardwareEncoder();
   }
 
-  private static native long nativeCreateEncoder(VideoEncoder fallback, VideoEncoder primary);
   private static native long nativeCreate(
       long webrtcEnvRef, VideoEncoder fallback, VideoEncoder primary);
 }

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

@@ -14,9 +14,7 @@
  * Wraps a native webrtc::VideoEncoder.
  */
 public abstract class WrappedNativeVideoEncoder implements VideoEncoder {
-  // TODO: bugs.webrtc.org/15860 - uncomment when implemented by all
-  // derived classes instead of the createNativeVideoEncoder
-  // @Override public abstract long createNative(long webrtcEnvRef);
+  @Override public abstract long createNative(long webrtcEnvRef);
   @Override public abstract boolean isHardwareEncoder();
 
   @Override

sdk/android/src/jni/libaom_av1_encoder.cc

@@ -19,10 +19,6 @@
 namespace webrtc {
 namespace jni {
 
-static jlong JNI_LibaomAv1Encoder_CreateEncoder(JNIEnv* jni) {
-  return jlongFromPointer(webrtc::CreateLibaomAv1Encoder().release());
-}
-
 jlong JNI_LibaomAv1Encoder_Create(JNIEnv* jni, jlong j_webrtc_env_ref) {
   return NativeToJavaPointer(
       CreateLibaomAv1Encoder(

sdk/android/src/jni/software_video_encoder_factory.cc

@@ -45,22 +45,6 @@ jlong JNI_SoftwareVideoEncoderFactory_Create(
           .release());
 }
 
-static jlong JNI_SoftwareVideoEncoderFactory_CreateEncoder(
-    JNIEnv* env,
-    jlong j_factory,
-    const webrtc::JavaParamRef<jobject>& j_video_codec_info) {
-  auto* const native_factory =
-      reinterpret_cast<webrtc::VideoEncoderFactory*>(j_factory);
-  const auto video_format =
-      webrtc::jni::VideoCodecInfoToSdpVideoFormat(env, j_video_codec_info);
-
-  auto encoder = native_factory->CreateVideoEncoder(video_format);
-  if (encoder == nullptr) {
-    return 0;
-  }
-  return webrtc::NativeToJavaPointer(encoder.release());
-}
-
 static webrtc::ScopedJavaLocalRef<jobject>
 JNI_SoftwareVideoEncoderFactory_GetSupportedCodecs(JNIEnv* env,
                                                    jlong j_factory) {

sdk/android/src/jni/video_encoder_factory_wrapper.cc

@@ -92,18 +92,6 @@ VideoEncoderFactoryWrapper::VideoEncoderFactoryWrapper(
 }
 VideoEncoderFactoryWrapper::~VideoEncoderFactoryWrapper() = default;
 
-std::unique_ptr<VideoEncoder> VideoEncoderFactoryWrapper::CreateVideoEncoder(
-    const SdpVideoFormat& format) {
-  JNIEnv* jni = AttachCurrentThreadIfNeeded();
-  ScopedJavaLocalRef<jobject> j_codec_info =
-      SdpVideoFormatToVideoCodecInfo(jni, format);
-  ScopedJavaLocalRef<jobject> encoder = Java_VideoEncoderFactory_createEncoder(
-      jni, encoder_factory_, j_codec_info);
-  if (!encoder.obj())
-    return nullptr;
-  return JavaToNativeVideoEncoder(jni, encoder);
-}
-
 std::unique_ptr<VideoEncoder> VideoEncoderFactoryWrapper::Create(
     const Environment& env,
     const SdpVideoFormat& format) {