Add external entity inclusions to internalSubset

RESOLVED FIXED in mozilla1.4alpha

Status

()

Core
XML
RESOLVED FIXED
15 years ago
15 years ago

People

(Reporter: Heikki Toivonen (remove -bugzilla when emailing directly), Assigned: Heikki Toivonen (remove -bugzilla when emailing directly))

Tracking

Trunk
mozilla1.4alpha
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [fixinhand])

Attachments

(1 attachment, 1 obsolete attachment)

jst convinced me that we should add the %foo; part into the internal subset as
well. If save as complete wants to tweak that, then it's their business.

We need to change Expat a little so that when it calls
nsExpatDriver::HandleExternalEntityRef() we get the name, and then in that same
method (?) append to mDoctypeText.
Created attachment 113229 [details] [diff] [review]
Possible fix
Attachment #113229 - Flags: superreview?(peterv)
Attachment #113229 - Flags: review?(harishd)
Status: NEW → ASSIGNED
Whiteboard: [fixinhand]
Target Milestone: --- → mozilla1.4alpha

Comment 2

15 years ago
Comment on attachment 113229 [details] [diff] [review]
Possible fix

r=harishd
Attachment #113229 - Flags: review?(harishd) → review+
Was checked in with the fix to bug 191327.
Status: ASSIGNED → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → FIXED
Attachment #113229 - Attachment is obsolete: true
Created attachment 116591 [details]
testcase
Attachment #113229 - Flags: superreview?(peterv)
You need to log in before you can comment on or make changes to this bug.