Closed Bug 1552752 Opened 5 months ago Closed 5 months ago

Split `nsDocumentEncoder::FlushText` out of `nsDocumentEncoder`

Categories

(Core :: Serializers, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla69
Tracking Status
firefox69 --- fixed

People

(Reporter: mbrodesser, Assigned: mbrodesser)

Details

Attachments

(2 files)

In order to abstract streaming details away from nsDocumentEncoder.

Assignee: nobody → mbrodesser

Abstracts the streaming details away. Reduces complexity of
nsDocumentEncoder.

Keywords: checkin-needed

Requesting checkin. A corresponding try run has only unrelated failures.

Pushed by malexandru@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/acf1896be423
Part 1) Split off text streaming in nsDocumentEncoder to separate class. r=hsivonen

Keywords: checkin-needed

@malexandru: just to be sure, can you please push "Part 2" too?

Flags: needinfo?(malexandru)
Pushed by malexandru@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0e382475292d
Part 2) Rename `ConvertAndWrite` to `EncodeAndWrite` and slighlty simplify method. r=hsivonen

Done :D

Flags: needinfo?(malexandru)

Thanks, Mihai.

Status: NEW → RESOLVED
Closed: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla69
You need to log in before you can comment on or make changes to this bug.