Open Bug 363688 Opened 15 years ago Updated 9 months ago

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


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





(Reporter: hsivonen, Unassigned)


(Depends on 3 open bugs, Blocks 1 open bug)


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.)
No longer depends on: incrementalxml
Depends on: incrementalxml
Assignee: mrbkap → nobody
I guess this now means replacing nsXMLContentSink with something that emits nsHtml5TreeOperations that feed into nsHtml5TreeOpExecutor.
Blocks: 698442
Depends on: 717561
Depends on: 737778
Depends on: 737845
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....
Flags: needinfo?(overholt)
I am also exploring alternatives to move this forward, setting NI on me as a reminder.
Flags: needinfo?(htsai)
Flags: needinfo?(overholt)
Flags: needinfo?(htsai)
Alphan will start to study this in Q4.
Assignee: nobody → 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.

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.  a test served as text/xml same, served as text/html
Rename this bug to match the current goal.
Summary: Unify incrementalism management code between the HTML and XML content sinks → Make the XML Web content load path have the same structure as the HTML loads path

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

Assignee: alchen → nobody
Flags: needinfo?(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.

Type: defect → enhancement
Flags: needinfo?(hsivonen)
Priority: -- → P3
You need to log in before you can comment on or make changes to this bug.