Closed Bug 1578149 Opened 11 months ago Closed 11 months ago

Group `nsPlainTextSerializer`'s indentation members in common struct

Categories

(Core :: DOM: Serializers, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla71
Tracking Status
firefox71 --- fixed

People

(Reporter: mbrodesser, Assigned: mbrodesser)

Details

Attachments

(5 files)

No description provided.
Assignee: nobody → mbrodesser

In order to encapsulate the indentation related members.

Whitespace triming was used by only one caller, so it deserves to live
outside. Additionally, this defines the method name more precisely.

Depends on D44347

Pushed by mbrodesser@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/73bbd2dfec98
part 1) Add `nsPlainTextSerializer::Indentation`. r=hsivonen
https://hg.mozilla.org/integration/autoland/rev/e6a2035efc36
part 2) Simplify code in `nsPlainTextSerializer::Write`.  r=hsivonen
https://hg.mozilla.org/integration/autoland/rev/ebdf7c03ad58
part 3) Move whitespace triming out of `OutputQuotesAndIndent`. r=hsivonen
https://hg.mozilla.org/integration/autoland/rev/5712af8379c5
part 4) Move truncating `mIndentation.mHeader` out of `CreateQuotesAndIndent`. r=hsivonen
https://hg.mozilla.org/integration/autoland/rev/bb67417b6f51
part 5) Bundle resetting `mIndentation.mHeader` and `mCurrentLineContent`. r=hsivonen
You need to log in before you can comment on or make changes to this bug.