Closed Bug 1733876 Opened 4 months ago Closed 4 months ago

Get rid of LineBreaker::Prev() in nsPlainTextSerializer

Categories

(Core :: DOM: Serializers, task)

task

Tracking

()

RESOLVED FIXED
95 Branch
Tracking Status
firefox95 --- fixed

People

(Reporter: TYLin, Assigned: TYLin)

References

Details

Attachments

(5 files)

This fixed part of Bug 1733009.

Blocks: 1733009

With this edge case explicitly considered, it will be easier for the
following patches to simplify logic in FindWrapIndexForContent().

Depends on D127381

This patch only moves logic, and shouldn't change the behavior.

Depends on D127382

This is required by the next part.

Depends on D127383

The purpose of this patch is to remove LineBreaker::Prev(), and audit
the deprecated LineBreaker::DeprecatedNext() (added in bug 1733872).

Add blockquotecite gtest to make sure short line wrapping with
blockquote citation still works. The testcase's behavior is not changed.

Depends on D127384

Pushed by aethanyc@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/dd5f9ce8a831
Part 1 - Remove unused kNoFlags in nsPlainTextSerializer.cpp. r=mbrodesser
https://hg.mozilla.org/integration/autoland/rev/2a6422635d37
Part 2 - Run line wrapping algorithm only if content of current line is not empty. r=m_kato,mbrodesser
https://hg.mozilla.org/integration/autoland/rev/b06880832da2
Part 3 - Consolidate logic dealing with and without aLineBreaker in FindWrapIndexForContent(). r=m_kato
https://hg.mozilla.org/integration/autoland/rev/6f0a6fcd5f5e
Part 4 - Allow GetUnicharStringWidth() to support mozilla::Span substring argument. r=m_kato
https://hg.mozilla.org/integration/autoland/rev/f7f60a86bf47
Part 5 - Simply line wrapping logic involving line breaker in FindWrapIndexForContent(). r=m_kato
Backout by nerli@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/eb78120056f6
Backed out 5 changesets for causing build windows bustages CLOSED TREE
Flags: needinfo?(aethanyc)
Pushed by aethanyc@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/5c7ea057db25
Part 1 - Remove unused kNoFlags in nsPlainTextSerializer.cpp. r=mbrodesser
https://hg.mozilla.org/integration/autoland/rev/dd3ea0d8556f
Part 2 - Run line wrapping algorithm only if content of current line is not empty. r=m_kato,mbrodesser
https://hg.mozilla.org/integration/autoland/rev/9fc206a0b925
Part 3 - Consolidate logic dealing with and without aLineBreaker in FindWrapIndexForContent(). r=m_kato
https://hg.mozilla.org/integration/autoland/rev/d8baa8f2e4c6
Part 4 - Allow GetUnicharStringWidth() to support mozilla::Span substring argument. r=m_kato
https://hg.mozilla.org/integration/autoland/rev/7f1e13c8839d
Part 5 - Simply line wrapping logic involving line breaker in FindWrapIndexForContent(). r=m_kato
Regressions: 1734792
You need to log in before you can comment on or make changes to this bug.