Closed Bug 143193 Opened 22 years ago Closed 13 years ago

nsHTMLContentSink::AddDoctypeDecl needs serious iterator love

Categories

(Core :: DOM: HTML Parser, defect)

Product:
Core
Component:
DOM: HTML Parser
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED WONTFIX

People

(Reporter: bzbarsky, Unassigned)

References

(Depends on 1 open bug)

Details

nsHTMLContentSink::AddDoctypeDecl uses Mid() _way_ too much.  Iterators are our
friends.  We should use them.
Priority: -- → P3
Target Milestone: --- → mozilla1.2beta
QA Contact: desale → stummala
Target Milestone: mozilla1.2beta → mozilla1.3beta
Perhaps all this code can even be shared with the parser (which needs to parse
shit for quirs-vs-strict anyway)?
Target Milestone: mozilla1.3beta → mozilla1.5alpha
Priority: P3 → P1
QA Contact: stummala → general
Target Milestone: mozilla1.5alpha → ---
Depends on: 563890
Assignee: bzbarsky → nobody
Priority: P1 → --
Component: DOM: Abstract Schemas → HTML: Parser
QA Contact: general → parser
Marking this WONTFIX, since this code  is in the process of getting replaced.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.