Remove custom header support from DevSupportHttpClient (#55884)

Summary:
Pull Request resolved: #55884

The addRequestHeader/removeRequestHeader methods don't need to live inside devsupport.

Instead we can use the `setOkHttpClientFactory` to provide this same logic.

Changelog: [Internal]

Reviewed By: javache, cipolleschi

Differential Revision: D95059080

fbshipit-source-id: e9124123911a44291799368d6aa4d7b8cb0d4d53

Author: cortinico

packages/react-native/ReactAndroid/src/main/java/com/facebook/react/devsupport/inspector/DevSupportHttpClient.kt

@@ -9,31 +9,17 @@
 
 package com.facebook.react.devsupport.inspector
 
-import java.util.concurrent.ConcurrentHashMap
 import java.util.concurrent.TimeUnit
-import okhttp3.Interceptor
 import okhttp3.OkHttpClient
 
 /**
  * Shared [OkHttpClient] instances for devsupport networking. Uses a single connection pool and
- * dispatcher across all dev support HTTP and WebSocket usage. Supports injecting custom request
- * headers that are applied to every outgoing request via an OkHttp interceptor.
+ * dispatcher across all dev support HTTP and WebSocket usage.
  */
 internal object DevSupportHttpClient {
-  private val customHeaders = ConcurrentHashMap<String, String>()
-
-  private val headerInterceptor = Interceptor { chain ->
-    val builder = chain.request().newBuilder()
-    for ((name, value) in customHeaders) {
-      builder.header(name, value)
-    }
-    chain.proceed(builder.build())
-  }
-
   /** Client for HTTP requests: connect=5s, write=disabled, read=disabled. */
   internal val httpClient: OkHttpClient =
       OkHttpClient.Builder()
-          .addInterceptor(headerInterceptor)
           .connectTimeout(5, TimeUnit.SECONDS)
           .writeTimeout(0, TimeUnit.MILLISECONDS)
           .readTimeout(0, TimeUnit.MINUTES)
@@ -47,16 +33,6 @@ internal object DevSupportHttpClient {
           .writeTimeout(10, TimeUnit.SECONDS)
           .build()
 
-  /** Add a custom header to be included in all requests made through both clients. */
-  internal fun addRequestHeader(name: String, value: String) {
-    customHeaders[name] = value
-  }
-
-  /** Remove a previously added custom header. */
-  internal fun removeRequestHeader(name: String) {
-    customHeaders.remove(name)
-  }
-
   /**
    * Returns the appropriate HTTP scheme ("http" or "https") for the given host. Uses "https" when
    * the host specifies port 443 explicitly (e.g. "example.com:443").