stream: use ByteLengthQueuingStrategy when not in object mode (fixes #…

…46347)

lib/internal/webstreams/adapters.js

@@ -25,6 +25,7 @@ const {
 
 const {
  CountQueuingStrategy,
+ ByteLengthQueuingStrategy,
 } = require('internal/webstreams/queuingstrategies');
 
 const {
@@ -417,11 +418,7 @@ function newReadableStreamFromStreamReadable(streamReadable, options = kEmptyObj
  return new CountQueuingStrategy({ highWaterMark });
  }
 
- // When not running in objectMode explicitly, we just fall
- // back to a minimal strategy that just specifies the highWaterMark
- // and no size algorithm. Using a ByteLengthQueuingStrategy here
- // is unnecessary.
- return { highWaterMark };
+ return new ByteLengthQueuingStrategy({ highWaterMark });
  };
 
  const strategy = evaluateStrategyOrFallback(options?.strategy);
@@ -457,12 +454,6 @@ function newReadableStreamFromStreamReadable(streamReadable, options = kEmptyObj
  streamReadable.on('data', onData);
 
  return new ReadableStream({
- // CGQAQ: we know that `Readable.readableHighWaterMark
- // size is always in bytes, so we can use `bytes` here
- // to make the `ReadableStream` a byte stream.
- // Ref: https://streams.spec.whatwg.org/#high-water-mark:~:text=Concretely%2C%20a%20queuing,floating%2Dpoint%20units.
- type: 'bytes',
-
  start(c) { controller = c; },
 
  pull() { streamReadable.resume(); },