serenity icon indicating copy to clipboard operation
serenity copied to clipboard

Published 20 hours ago • verified publisher icon SerenityOS

Draft: LibWeb: Implement QueuingStrategy for WriteableStream and ReadableStream

Open shannonbooth opened this issue 1 year ago • 1 comments

Draft on top of https://github.com/SerenityOS/serenity/pull/19455 as otherwise would have conflicts in LibWeb's Forward.h

These set of changes aim to implement the optional QueuingStrategy strategy = {} argument in the constructor of both ReadableStream and WritableStream - and the AO's required to implement that.

shannonbooth avatar Jun 18 '23 10:06 shannonbooth

While implementing ByteLengthQueuingStrategy size algorithm found a bug in my ExtractSizeAlgorithm AO implementation by mistakenly passing through chunk as this value rather than as an argument. Everything works good after that though!

shannonbooth avatar Jun 20 '23 11:06 shannonbooth

Pushed rebase after merge of https://github.com/SerenityOS/serenity/pull/19455

shannonbooth avatar Jun 21 '23 18:06 shannonbooth