Remove unnecessary OkHttpCompat (#51624)

Summary:
Pull Request resolved: #51624

Let's remove this class and inline all the methods to the callsites.
This class was also public which would make this a breaking change but I couldn't find any meaningful usage:
https://github.com/search?type=code&q=NOT+is%3Afork+NOT+org%3Afacebook+NOT+repo%3Areact-native-tvos%2Freact-native-tvos+NOT+repo%3Anuagoz%2Freact-native+NOT+repo%3A2lambda123%2Freact-native+NOT+repo%3Abeanchips%2Ffacebookreactnative+NOT+repo%3AfabOnReact%2Freact-native-notes+NOT+user%3Ahuntie+NOT+user%3Acortinico+NOT+repo%3AMaxdev18%2Fpowersync_app+NOT+repo%3Acarter-0%2Finstagram-decompiled+NOT+repo%3Am0mosenpai%2Finstadamn+NOT+repo%3AA-Star100%2FA-Star100-AUG2-2024+NOT+repo%3Alclnrd%2Fdetox-scrollview-reproductible+NOT+repo%3ADionisisChytiris%2FWorldWiseTrivia_Main+NOT+repo%3Apast3l%2Fhi2+NOT+repo%3AoneDotpy%2FCaribouQuest+NOT+repo%3Abejayoharen%2Fdailytodo+NOT+repo%3Amolangning%2Freversing-discord+NOT+repo%3AScottPrzy%2Freact-native+NOT+repo%3Agabrieldonadel%2Freact-native-visionos+NOT+repo%3AGabriel2308%2FTestes-Soft+NOT+repo%3Adawnzs03%2FflakyBuild+NOT+repo%3Acga2351%2Fcode+NOT+repo%3Astreeg%2Ftcc+NOT+repo%3Asoftware-mansion-labs%2Freact-native-swiftui+NOT+repo%3Apkcsecurity%2Fdecompiled-lightbulb+com.facebook.react.modules.network.OkHttpCompat

Changelog:
[Internal] [Changed] -

Reviewed By: javache

Differential Revision: D75448162

fbshipit-source-id: 9f4793809c9115a299ac72d79dbfb44073320d54

Author: cortinico

packages/react-native/ReactAndroid/api/ReactAndroid.api

@@ -3025,12 +3025,6 @@ public final class com/facebook/react/modules/network/OkHttpClientProvider {
 	public static final fun setOkHttpClientFactory (Lcom/facebook/react/modules/network/OkHttpClientFactory;)V
 }
 
-public final class com/facebook/react/modules/network/OkHttpCompat {
-	public static final field INSTANCE Lcom/facebook/react/modules/network/OkHttpCompat;
-	public final fun getCookieJarContainer (Lokhttp3/OkHttpClient;)Lcom/facebook/react/modules/network/CookieJarContainer;
-	public final fun getHeadersFromMap (Ljava/util/Map;)Lokhttp3/Headers;
-}
-
 public abstract interface class com/facebook/react/modules/network/ProgressListener {
 	public abstract fun onProgress (JJZ)V
 }

packages/react-native/ReactAndroid/src/main/java/com/facebook/react/modules/fresco/FrescoModule.kt

@@ -22,9 +22,9 @@ import com.facebook.react.bridge.ReactContextBaseJavaModule
 import com.facebook.react.common.ReactConstants
 import com.facebook.react.module.annotations.ReactModule
 import com.facebook.react.modules.common.ModuleDataCleaner
+import com.facebook.react.modules.network.CookieJarContainer
 import com.facebook.react.modules.network.ForwardingCookieHandler
 import com.facebook.react.modules.network.OkHttpClientProvider
-import com.facebook.react.modules.network.OkHttpCompat
 import com.facebook.react.turbomodule.core.interfaces.TurboModule
 import okhttp3.JavaNetCookieJar
 
@@ -156,7 +156,10 @@ constructor(
 
       // make sure to forward cookies for any requests via the okHttpClient
       // so that image requests to endpoints that use cookies still work
-      val container = OkHttpCompat.getCookieJarContainer(client)
+
+      @Suppress("DEPRECATION_ERROR") // Conflicting okhttp versions
+      val container = client.cookieJar() as CookieJarContainer
+
       val handler = ForwardingCookieHandler()
       container.setCookieJar(JavaNetCookieJar(handler))
       val builder =

packages/react-native/ReactAndroid/src/main/java/com/facebook/react/modules/fresco/ReactOkHttpNetworkFetcher.kt

@@ -11,9 +11,9 @@ import android.os.SystemClock
 import com.facebook.imagepipeline.backends.okhttp3.OkHttpNetworkFetcher
 import com.facebook.imagepipeline.producers.NetworkFetcher
 import com.facebook.react.bridge.ReadableMap
-import com.facebook.react.modules.network.OkHttpCompat
 import java.util.concurrent.TimeUnit
 import okhttp3.CacheControl
+import okhttp3.Headers
 import okhttp3.OkHttpClient
 import okhttp3.Request
 
@@ -57,7 +57,10 @@ internal class ReactOkHttpNetworkFetcher(okHttpClient: OkHttpClient) :
     } else {
       cacheControlBuilder.noStore()
     }
-    val headers = OkHttpCompat.getHeadersFromMap(requestHeaders)
+
+    @Suppress("DEPRECATION_ERROR") // Conflicting okhttp versions
+    val headers = Headers.of(requestHeaders ?: emptyMap())
+
     val request =
         Request.Builder()
             .headers(headers)