RESOLVED FIXED 233850
[LFC][IFC] Compute isFirstBox/isLastBox for text content 
https://bugs.webkit.org/show_bug.cgi?id=233850
Summary [LFC][IFC] Compute isFirstBox/isLastBox for text content
zalan
Reported 2021-12-03 19:12:39 PST
Iterators use these bits to figure out renderer boundaries.
Attachments
Patch (11.62 KB, patch)
2021-12-03 19:27 PST, zalan 		no flags
DetailsFormatted DiffDiff
Patch (12.82 KB, patch)
2021-12-03 21:09 PST, zalan 		no flags
DetailsFormatted DiffDiff
Patch (12.90 KB, patch)
2021-12-03 22:00 PST, zalan 		no flags
DetailsFormatted DiffDiff
Patch (6.53 KB, patch)
2021-12-04 06:46 PST, zalan 		no flags
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
zalan
Comment 1 2021-12-03 19:27:37 PST
Created attachment 445937 [details] Patch
zalan
Comment 2 2021-12-03 21:09:56 PST
Created attachment 445955 [details] Patch
zalan
Comment 3 2021-12-03 22:00:21 PST
Created attachment 445956 [details] Patch
Antti Koivisto
Comment 4 2021-12-04 05:37:09 PST
Comment on attachment 445956 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=445956&action=review > Source/WebCore/layout/formattingContexts/inline/InlineDisplayContentBuilder.cpp:394 > + enum class LineHasFirstLastRunForTextBox { No, FirstOnly, LastOnly, Both }; Don't you have a few types for this already like IsFirstLastBox and OptionSet<InlineDisplay::Box::PositionWithinInlineLevelBox>?
zalan
Comment 5 2021-12-04 06:46:43 PST
Created attachment 445970 [details] Patch
EWS
Comment 6 2021-12-04 08:51:46 PST
Committed r286534 (244866@main): <https://commits.webkit.org/244866@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 445970 [details].
Radar WebKit Bug Importer
Comment 7 2021-12-04 08:52:24 PST
<rdar://problem/86053206>
Note You need to log in before you can comment on or make changes to this bug.