Closed Bug 595844 Opened 10 years ago Closed 9 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)

defect

Tracking

()

RESOLVED FIXED
mozilla2.0b7
Tracking Status
blocking2.0 --- betaN+

People

(Reporter: hsivonen, Assigned: hsivonen)

References

()

Details

Attachments

(1 file)

<svg><foreignObject><div><![CDATA[foo]]> should create a bogus comment but instead creates a text node "foo".
Attached patch FixSplinter Review
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: nobody → hsivonen
Status: NEW → ASSIGNED
Attachment #474684 - Flags: review?(jonas)
Depends on: 559023
Priority: -- → P2
Nominating as a blocker, because failing to land this fix would meaning shipping a spec violation that sites might start depending on.
blocking2.0: --- → ?
blocking2.0: ? → betaN+
Whiteboard: [ready to land if bug 595797 gets approval]
http://hg.mozilla.org/mozilla-central/rev/01f0423319a7
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b8
Whiteboard: [ready to land if bug 595797 gets approval]
Target Milestone: mozilla2.0b8 → mozilla2.0b7
You need to log in before you can comment on or make changes to this bug.