1569197 - [BinAST] Deactivate Brotli decompression

Status: RESOLVED FIXED

(Core :: JavaScript Engine, task, P2)

Description

[David Teller [:Yoric] - still alive but not very active]

In the first release of context-0.x, we'll let Necko handle brotli decompression, so we need to make sure that we don't use our brotli decompression code just yet.

Comment 1

[David Teller [:Yoric] - still alive but not very active]

Attachment: Bug 1569197 - Stop decompressing content-0.1 data's brotli layer manually;r?arai

The latest version of content-0.1 does NOT embed brotli content. Rather, the protocol expects the data to be brotli-(de)compressed at the http layer, with the VM only taking care of Huffman-level compression.

This patch removes the code that we had installed to handle brotli decompression. Templatized methods that take as type argument Compress::{Yes, No} remain templatized, as we expect that we're going to need Huffman decompression
soon, but for the time being, only the uncompressed implementation remains.

Comment 2

[David Teller [:Yoric] - still alive but not very active]

Attachment: Bug 1569197 - Backed out changeset 2a053de2828f;r?arai

Since we do not use Brotli as part of SpiderMonkey anymore, let's avoid linking it.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e2fa438ebdc7
Backed out changeset 2a053de2828f;r=arai

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ce6819edd27e
Stop decompressing content-0.1 data's brotli layer manually. r=arai

Comment 5

[Stefan Hindli [:stefan_hindli]]

https://hg.mozilla.org/mozilla-central/rev/e2fa438ebdc7
https://hg.mozilla.org/mozilla-central/rev/ce6819edd27e