Fixes `node:http` and `node:stream` so `ytdl-core` works. (#3452)

* fix crash in readablestate

* make node:https request+get actually use https

* use a native readablestream in IncomingMessage

* tweaks

* fix abort crash

* emit close by default

* remove abort. this isnt a real function

* add validate functions, fixup some other requested changes. not done yet

* Update WebCoreJSBuiltins.cpp

* Update JSReadableState.cpp

* Add some missing exports

---------

Co-authored-by: Jarred Sumner <709451+Jarred-Sumner@users.noreply.github.com>

1 files changed, 2 insertions, 2 deletions

diff --git a/src/js/builtins/ReadableStreamDefaultReader.ts b/src/js/builtins/ReadableStreamDefaultReader.ts
index ecd553ed5..70c6df8c3 100644
--- a/src/js/builtins/ReadableStreamDefaultReader.ts
+++ b/src/js/builtins/ReadableStreamDefaultReader.ts
@@ -43,7 +43,7 @@ export function cancel(this, reason) {
   return $readableStreamReaderGenericCancel(this, reason);
 }
 
-export function readMany(this) {
+export function readMany(this: ReadableStreamDefaultReader): ReadableStreamDefaultReadManyResult<any> {
   if (!$isReadableStreamDefaultReader(this))
     throw new TypeError("ReadableStreamDefaultReader.readMany() should not be called directly");
 
@@ -75,7 +75,7 @@ export function readMany(this) {
   var length = values.length;
 
   if (length > 0) {
-    var outValues = $newArrayWithSize(length);
+    var outValues = $newArrayWithSize<T>(length);
     if ($isReadableByteStreamController(controller)) {
       {
         const buf = values[0];