651049 - Write a new off-the-main-thread IO driver for expat

Status: NEW

(Core :: XML, defect)

Description

[Henri Sivonen (:hsivonen)]

In order to get rid of the badness that is nsScanner and in order to move the parsing of XML Web content off the main thread, there should be a new off-the-main-thread IO driver for expat. This IO driver should deal with receiving bytes from a network stream, handling character encoding conversion and feeding the decoded UTF-16 data to expat. The design should be similar to the nsHtml5Parser/nsHtml5StreamParser pair.

Comment 1

[Mike Conley (:mconley) (:⚙️)]

How likely is this to happen in the short-term? We ask, because it seems like a blocker for bug 1614137, which we're attempting to set a priority on during triage.

Comment 2

[Henri Sivonen (:hsivonen)]

Fission focus stuff takes up my time in the short term. Alphan, what's your current thinking on the off-the-main-thread XML project?

Comment 3

[Mike Conca [:mconca]]

(In reply to Mike Conley (:mconley) (:⚙️) from comment #1)

How likely is this to happen in the short-term? We ask, because it seems like a blocker for bug 1614137, which we're attempting to set a priority on during triage.

I saw this is blocking bug 1614137, which was reported on an Android device. Is this specific to Fenix (mobile)?

Comment 4

[Mike Conley (:mconley) (:⚙️)]

(In reply to Mike Conca [:mconca] from comment #3)

I saw this is blocking bug 1614137, which was reported on an Android device. Is this specific to Fenix (mobile)?

No, this also affects Desktop.

Comment 5

[Alphan Chen [:alchen]]

(In reply to Henri Sivonen (:hsivonen) from comment #2)

Fission focus stuff takes up my time in the short term. Alphan, what's your current thinking on the off-the-main-thread XML project?

I should have time after landing the session store rewriting.
WIll discuss with Hsinyi soon.

Comment 6

[Olli Pettay [:smaug][[e-mail]]]

This affects pageload, possibly quite a lot if the page uses SVG heavily. So p1 (pageload bugs are p1 by default).
But I do expect fixing this to take some time.

Comment 7

[Henri Sivonen (:hsivonen)]

It would be preferable, though not strictly required, to fix bug 1733043 before this one.