| Summary: | [IFC] Move "content has position dependent width" traversal to RenderText | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | zalan <zalan> | ||||||||||
| Component: | Layout and Rendering | Assignee: | zalan <zalan> | ||||||||||
| Status: | RESOLVED FIXED | ||||||||||||
| Severity: | Normal | CC: | bfulgham, changseok, esprehn+autocc, ews-watchlist, glenn, koivisto, kondapallykalyan, mmaxfield, pdr, simon.fraser, webkit-bug-importer, zalan | ||||||||||
| Priority: | P2 | Keywords: | InRadar | ||||||||||
| Version: | WebKit Nightly Build | ||||||||||||
| Hardware: | Unspecified | ||||||||||||
| OS: | Unspecified | ||||||||||||
| Attachments: |
|
||||||||||||
|
Description
zalan
2023-11-20 08:18:51 PST
Created attachment 468682 [details]
Patch
Comment on attachment 468682 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=468682&action=review > Source/WebCore/layout/layouttree/LayoutInlineTextBox.h:58 > +inline void InlineTextBox::updateContent(String newContent, bool canUseSimpleFontCodePath, bool canUseSimplifiedContentMeasuring, bool hasPositionDependentContentWidth) maybe we'll need an OptionSet soon (In reply to Antti Koivisto from comment #2) > Comment on attachment 468682 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=468682&action=review > > > Source/WebCore/layout/layouttree/LayoutInlineTextBox.h:58 > > +inline void InlineTextBox::updateContent(String newContent, bool canUseSimpleFontCodePath, bool canUseSimplifiedContentMeasuring, bool hasPositionDependentContentWidth) > > maybe we'll need an OptionSet soon very true! Created attachment 468688 [details]
Patch
Created attachment 468692 [details]
Patch
Created attachment 468694 [details]
[fast-cq]Patch
Committed 270993@main (d36b968cd91a): <https://commits.webkit.org/270993@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 468694 [details]. |