Closed
Bug 595844
Opened 15 years ago
Closed 15 years ago
Should check the namespace of the current element on the tree builder stack when deciding whether to support CDATA sections in text/html
Categories
(Core :: DOM: HTML Parser, defect, P2)
Core
DOM: HTML Parser
Tracking
()
RESOLVED
FIXED
mozilla2.0b7
| Tracking | Status | |
|---|---|---|
| blocking2.0 | --- | betaN+ |
People
(Reporter: hsivonen, Assigned: hsivonen)
References
()
Details
Attachments
(1 file)
|
5.98 KB,
patch
|
sicking
:
review+
|
Details | Diff | Splinter Review |
<svg><foreignObject><div><![CDATA[foo]]> should create a bogus comment but instead creates a text node "foo".
|
Assignee | |
Comment 1•15 years ago
|
||
FWIW, I think the inForeign member of the tree builder has become pointless and the namespace of the current node should always be checked instead. However, in case I'm misunderstanding something, I have not made that change.
The test case will land as part of bug 559023.
|
|
Assignee | |
Updated•15 years ago
|
Priority: -- → P2
Attachment #474684 -
Flags: review?(jonas) → review+
rs=me
|
|
Assignee | |
Comment 3•15 years ago
|
||
Nominating as a blocker, because failing to land this fix would meaning shipping a spec violation that sites might start depending on.
blocking2.0: --- → ?
|
||
Updated•15 years ago
|
blocking2.0: ? → betaN+
|
|
Assignee | |
Updated•15 years ago
|
Whiteboard: [ready to land if bug 595797 gets approval]
|
|
Assignee | |
Comment 4•15 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b8
|
|
Assignee | |
Updated•15 years ago
|
Whiteboard: [ready to land if bug 595797 gets approval]
|
||
Updated•15 years ago
|
Target Milestone: mozilla2.0b8 → mozilla2.0b7
You need to log in
before you can comment on or make changes to this bug.
Description
•