Open Bug 363688 Opened 13 years ago Updated 1 year ago

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

Categories

(Core :: DOM: HTML Parser, defect)

defect
Not set

Tracking

()

People

(Reporter: hsivonen, Assigned: alchen)

References

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

Details

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.
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
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
You need to log in before you can comment on or make changes to this bug.