tree 7d11ef0bbc54e12b92c152803e994f88781ac80b
parent e298709f319fcd50e268b211145256537928d1b6
author Koji Ishii <kojii@chromium.org> 1694792819 -0700
committer Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> 1694793971 -0700

[text-autospace] Fix for ligatures

This patch fixes `text-autospacing` not being applied after
ligatures.

To fix this:
1. `NGTextAutoSpace` is changed to use the offset to insert
   auto-spacing before, not after.
2. `ShapeResult::ApplyTextAutoSpacing` is changed to add the
   spacing to the last glyph before the offset is equal to or
   greater than the specified offset.

Note, `AddAutoSpacingToIdeographRTL` was removed because it
fails with this change, and the RTL implementation is being
re-worked out.

Bug: 1463890
Change-Id: I260539ff4bebf14e894854d033bf10b6fc52f054
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4863019
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Lingqi Chi <lingqi@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1197168}