http: refactor to remove redundant argument of _deferToConnect

PR-URL: #38598
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>

Author: pd4d10

lib/_http_client.js

@@ -818,7 +818,7 @@ function onSocketNT(req, socket, err) {
 }
 
 ClientRequest.prototype._deferToConnect = _deferToConnect;
-function _deferToConnect(method, arguments_, cb) {
+function _deferToConnect(method, arguments_) {
   // This function is for calls that need to happen once the socket is
   // assigned to this request and writable. It's an important promisy
   // thing for all the socket calls that happen either now
@@ -828,9 +828,6 @@ function _deferToConnect(method, arguments_, cb) {
   const callSocketMethod = () => {
     if (method)
       ReflectApply(this.socket[method], this.socket, arguments_);
-
-    if (typeof cb === 'function')
-      cb();
   };
 
   const onSocket = () => {