Open
Bug 363688
Opened 18 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, enhancement, P3)
Core
DOM: HTML Parser
Tracking
()
NEW
People
(Reporter: hsivonen, Unassigned)
References
(Depends on 2 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.)
Updated•18 years ago
|
No longer depends on: incrementalxml
Updated•18 years ago
|
Depends on: incrementalxml
Updated•15 years ago
|
Assignee: mrbkap → nobody
Reporter | ||
Comment 1•15 years ago
|
||
I guess this now means replacing nsXMLContentSink with something that emits nsHtml5TreeOperations that feed into nsHtml5TreeOpExecutor.
Comment 2•7 years ago
|
||
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)
Comment 3•7 years ago
|
||
I am also exploring alternatives to move this forward, setting NI on me as a reminder.
Flags: needinfo?(htsai)
Updated•7 years ago
|
Flags: needinfo?(overholt)
Updated•7 years ago
|
Flags: needinfo?(htsai)
Comment 4•7 years ago
|
||
Alphan will start to study this in Q4.
Updated•7 years ago
|
Assignee: nobody → alchen
Comment 5•7 years ago
|
||
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•7 years ago
|
||
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
Comment 7•4 years ago
|
||
Henri, can you please check the priority here? Thank you!
Assignee: alchen → nobody
Flags: needinfo?(hsivonen)
Reporter | ||
Comment 8•4 years ago
|
||
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
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•