http: use standard args order in IncomingMEssage onError

PR-URL: #33035 Refs: #30625 Reviewed-By: Robert Nagy <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Rich Trott <[email protected]>

Author: dnlup

lib/_http_incoming.js

@@ -169,7 +169,7 @@ IncomingMessage.prototype._destroy = function _destroy(err, cb){
 this.emit('aborted');
 }
 
-// If aborted and the underlying socket not already destroyed,
+// If aborted and the underlying socket is not already destroyed,
 // destroy it.
 // We have to check if the socket is already destroyed because finished
 // does not call the callback when this methdod is invoked from `_http_client`
@@ -178,10 +178,10 @@ IncomingMessage.prototype._destroy = function _destroy(err, cb){
 this.socket.destroy(err);
 constcleanup=finished(this.socket,(e)=>{
 cleanup();
-onError(this,cb,e||err);
+onError(this,e||err,cb);
 });
 }else{
-onError(this,cb,err);
+onError(this,err,cb);
 }
 };
 
@@ -361,7 +361,7 @@ IncomingMessage.prototype._dump = function _dump(){
 }
 };
 
-functiononError(instance,cb,error){
+functiononError(instance,error,cb){
 // This is to keep backward compatible behavior.
 // An error is emitted only if there are listeners attached to
 // the event.