[LFC][Integration] Add some useful functions to iterators

[Antti Koivisto]

For future use

[Antti Koivisto]

Created attachment 438462 [details]
Patch

[zalan]

Comment on attachment 438462 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=438462&action=review

> Source/WebCore/layout/integration/LayoutIntegrationRunIteratorModernPath.h:138
> +    enum class HyphenMode { Include, Ignore };
> +    TextRun createTextRun(HyphenMode hyphenMode = HyphenMode::Include) const

How does the caller decide whether it needs hyphen or not (as this decision is mostly driven by line breaking and the run already includes it In ::renderedContent())

[Antti Koivisto]

> How does the caller decide whether it needs hyphen or not (as this decision
> is mostly driven by line breaking and the run already includes it In
> ::renderedContent())

You are right, kept the argument version internal and only exposed a version without any.

[Antti Koivisto]

Created attachment 438473 [details]
Patch

[EWS]

Committed r282661 (241802@main): <https://commits.webkit.org/241802@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 438473 [details].

[Radar WebKit Bug Importer]

<rdar://problem/83241151>