Re: [w3ctag/design-reviews] CompressStream (#410)

Overall this looks good to us. A few relatively minor points:

1) Have you considered using different classes for each compression algorithm rather than a constructor argument? e.g. `let s = new GzipCompressionStream();` this might make it easier to pollyfill new algorithms.

2) If the compression algorithm remains an argument, please ensure that the value uses (and references in the spec) the HTTP Content Coding tokens: https://www.iana.org/assignments/http-parameters/http-parameters.xhtml#content-coding

3) It might be useful to have a property to access the compression algorithm of the stream (and any future options).

4) It may be worth considering adding support for Brotli in V1.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/w3ctag/design-reviews/issues/410#issuecomment-530173723