[Webkit-unassigned] [Bug 181315] ::first-letter incorrectly selects grapheme pairs
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon Jan 8 12:32:06 PST 2018
https://bugs.webkit.org/show_bug.cgi?id=181315
--- Comment #26 from Chris Nardi <christopherncarmel at hotmail.com> ---
Comment on attachment 330650
--> https://bugs.webkit.org/attachment.cgi?id=330650
Patch
View in context: https://bugs.webkit.org/attachment.cgi?id=330650&action=review
>> Source/WebCore/rendering/updating/RenderTreeBuilderFirstLetter.cpp:240
>> + scanLength += numCharacters - 1;
>
> The is a strange way to write the loop. I would instead suggest either:
>
> 1) Removing the ++scanLength from the for loop above and instead writing this here:
>
> scanLength += numCharacters;
>
> or
>
> 2) Defining numCharacters before the loop so it can be used in the for loop and then writing:
>
> for (unsigned scanLength = length; scanLength < oldText.length(); scanLength += numCharacters)
>
> Also seems that numCharacters and numCharacterInGraphemeClusters are both not great names. Since these are "code units", not "characters". Should be numCodeUnitsInGraphemeClusters and numCodeUnits. Someone should come back and rename.
I implemented your second suggestion. I did not change the function name or the variable name; I figured that would be better with another patch.
I also added my proposed WPT testcase as I realized that it could just be reviewed here instead of upstream.
--
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20180108/f41f2e0a/attachment.html>
More information about the webkit-unassigned
mailing list