Closed Bug 553540 Opened 10 years ago Closed 10 years ago

need new flag to nsPlainTextSerializer to support delsp=yes (RFC 3676)

Categories

(Core :: DOM: Serializers, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla1.9.3a5

People

(Reporter: m_kato, Assigned: m_kato)

References

()

Details

Attachments

(1 file)

To implement bug 26734, need new flag (OutputFormatDelSp?) to nsPlainTextSerializer.
Attached patch patch v1Splinter Review
Attachment #436848 - Flags: review?(Olli.Pettay)
Attachment #436848 - Flags: review?(Olli.Pettay) → review?(laurent)
Comment on attachment 436848 [details] [diff] [review]
patch v1

Laurent, could you review this? I could sr, but would be great if you could look at this too.
Comment on attachment 436848 [details] [diff] [review]
patch v1

Although I'm not an expert yet of nsPlainTextSerializer and of the RFC 3676 (that I just discovered and I hope I understood well it), this patch seems ok for me.
Attachment #436848 - Flags: review?(laurent) → review+
Attachment #436848 - Flags: superreview?(jst)
Comment on attachment 436848 [details] [diff] [review]
patch v1

Looks good. And sorry for this review taking so long.
Attachment #436848 - Flags: superreview?(jst) → superreview+
http://hg.mozilla.org/mozilla-central/rev/42c54fb3a4ab
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a5
You need to log in before you can comment on or make changes to this bug.