1599472 - Stream the output of inflate in nsHTTPCompressConv::OnDataAvailable out to another thread to be converted to UTF16 when possible

Status: NEW

(Core :: DOM: Networking, enhancement, P3)

Description

[Alex Thayer [:alexical] (she/her)]

See this profile of fetching a large json payload: https://perfht.ml/2rwfKSF (observed on http://queze.net/bhr/test/)

We spend almost 400ms in StreamTrans #131 in MOZ_Z_inflate, and then almost 400ms in the main thread in encoding_rs::utf_8::convert_utf8_to_utf16_up_to_invalid. It seems like these could be run concurrently, with the output of the first streaming into the latter.

This seems like a small problem for small payloads, but I suspect it might strongly affect web developers who build their own tools to, say, visualize data from large json payloads. (It should be noted that the page in question performs significantly better in Chrome)

Comment 1

[Alex Thayer [:alexical] (she/her)]

I think DOM: Networking might actually be a more appropriate fit for this. Still not entirely sure.

Comment 2

[Honza Bambas (:mayhemer)]

Could be related to bug 1505493.