Bug 1706312 [wpt PR 28595] - Consider text direction when building the GlyphData structure, a=testonly

Automatic update from web-platform-tests Consider text direction when building the GlyphData structure When there is a list item we need to insert a space between the marker symbol and the element being listed. We use the CreateForSpace function in this case. Depending on the length, we create a space glyph of the required font size, plus length - 1 0 width space glyphs. The character index is added in a for loop as an ascending integer sequence. However, this approach is not correct when the text is RTL, where the index sequence should be descending to match how the text is rendered visually. Bug: 1188728, 1160582 Change-Id: I50c30ae142e73ffde3c3302b8e4baa4ab334a258 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2825447 Commit-Queue: Koji Ishii <kojii@chromium.org> Reviewed-by: Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#874326} -- wpt-commits: c99958cd1ebbd17045842d57706545362616020e wpt-pr: 28595
2021-04-24 09:12:04 +00:00 · 2021-04-24 09:12:04 +00:00 · 5c869784b1
--- a/testing/web-platform/tests/css/css-text/crashtests/rendering-table-caption-with-list-item-and-svg-crash.html
+++ b/testing/web-platform/tests/css/css-text/crashtests/rendering-table-caption-with-list-item-and-svg-crash.html
@ -0,0 +1,7 @@
+<!doctype html>
+<title>CSS Text Test: Crash rendering a table caption with list item and svg</title>
+<link rel="author" title="Javier Fernandez" href="mailto:jfernandez@igalia.com" />
+<link rel="help" href="https://crbug.com/1197974">
+<meta name="assert" content="The test doesn't crash in debug"/>
+<link rel="stylesheet" type="text/css" href="/fonts/ahem.css" />
+<table dir="rtl"><caption><li><svg></caption></table>