363688 - Make the XML Web content load path have the same structure as the HTML loads path

Status: NEW

(Core :: DOM: HTML Parser, enhancement, P3)

Description

[Henri Sivonen (:hsivonen)]

Filed in response to bug 18333 comment 60.

Code from HTML sink contexts was forked to the XML content sink as part of the fix for bug 18333. Ideally, this code should be unforked and moved to a superclass. However, that probably cannot happen with sink contexts in place on the HTML side.

(It seems to me that rewriting the HTML parser and content sink to implement the HTML5 parsing algorithm would give a good opportunity to address this issue.)

Comment 1

[Henri Sivonen (:hsivonen)]

I guess this now means replacing nsXMLContentSink with something that emits nsHtml5TreeOperations that feed into nsHtml5TreeOpExecutor.

Comment 2

[Boris Zbarsky [:bzbarsky]]

Andrew, this is actually a good tech debt item to deal with for DOM; there's a bunch of code that can go away if we do this....

Comment 3

[Hsin-Yi Tsai (she/her) [:hsinyi]]

I am also exploring alternatives to move this forward, setting NI on me as a reminder.

Comment 4

[Hsin-Yi Tsai (she/her) [:hsinyi]]

Alphan will start to study this in Q4.

Comment 5

[Alphan Chen [:alchen]]

After discussing with Henri, this bug should be XML rewrite.
The bug will make the XML Web content load path have the same structure as the HTML loads path.
nsXMLContentSink would be replaced by mozilla::parser::xml::TreeOpGenerator(new added) and nsHtml5TreeOperations/nsHtml5TreeOpExecutor(eventually to be named mozilla::parser::TreeOperations/TreeOpExecutor).

The detailed work is described in the following link.
https://wiki.mozilla.org/Platform/XML_Rewrite

At first, I will try to resolve bug 717561, bug 737778 and bug 737845.

One more thing, the incrementalism is not worked currently on the two test pages.
http://www.damowmow.com/mozilla/bugs/18333/xml  a test served as text/xml
http://www.damowmow.com/mozilla/bugs/18333/html same, served as text/html

Comment 6

[Alphan Chen [:alchen]]

Rename this bug to match the current goal.

Comment 7

[Jens Stutte [:jstutte]]

Henri, can you please check the priority here? Thank you!

Comment 8

[Henri Sivonen (:hsivonen)]

We want this, but in order to get this done, someone needs to be able to dedicate the time that's needed to get this done.