Add external entity inclusions to internalSubset

RESOLVED FIXED in mozilla1.4alpha

Status

()

RESOLVED FIXED
16 years ago
16 years ago

People

(Reporter: hjtoi-bugzilla, Assigned: hjtoi-bugzilla)

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.
Attachment #113229 - Flags: superreview?(peterv)
Attachment #113229 - Flags: review?(harishd)
Status: NEW → ASSIGNED
Whiteboard: [fixinhand]
Target Milestone: --- → mozilla1.4alpha

Comment 2

16 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: 16 years ago
Resolution: --- → FIXED
Attachment #113229 - Attachment is obsolete: true
Attachment #113229 - Flags: superreview?(peterv)
You need to log in before you can comment on or make changes to this bug.