Re: [whatwg/streams] ReadableStream constructor: standard text and reference implementation doesn't match (#917)

I'm happy with the difference in handling of autoAllocateChunkSize and size(), because we error on size() specifically to avoid some confusing behaviour. Including autoAllocateChunkSize where it isn't needed is much less confusing: either you forgot `type: 'bytes'`, which case you'll find out when your access to `controller.byobRequest` fails, or you just included autoAllocateChunkSize where it was meaningless, in which case it behaves just like all other meaningless properties: it does nothing.

I think the payoff for making lookup / validation order more logical is pretty small: only a tiny proportion of developers will ever notice. I contemplated abandoning even the minor changes I've made, but since I'd decided we needed the tests I wanted to be able to justify the expectations I was writing.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/streams/issues/917#issuecomment-383873795