Closed Bug 1823108 Opened 2 years ago Closed 2 years ago

[wpt-sync] Sync PR 39064 - Fix a TextIterator crash in Greek text-transform

Categories

(Core :: Layout: Text and Fonts, task, P4)

task

Tracking

()

RESOLVED FIXED
113 Branch
Tracking Status
firefox113 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 39064 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/39064
Details from upstream follow.

Xiaocheng Hu <xiaochengh@chromium.org> wrote:

Fix a TextIterator crash in Greek text-transform

CSS text-transform can change text length in either ways, resulting in
more or fewer characters in LayoutText. The former case is more well
known and has been largely worked around.

Now that we see the latter case being possible, this patch works around
it in TextIterator to avoid reaching a NOTREACHED().

Fixed: 1424324
Bug: 750990
Change-Id: I4b8ba886bcacc06e9e491badb30af98d0f22c37c

Reviewed-on: https://chromium-review.googlesource.com/4351287
WPT-Export-Revision: c15adf96155e59c9579f9026a5a0c9deba693451

Component: web-platform-tests → Layout: Text and Fonts
Product: Testing → Core
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Whiteboard: [wptsync downstream error] → [wptsync downstream]

CI Results

Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 1 tests

Status Summary

Firefox

PASS: 1

Chrome

FAIL: 1

Safari

FAIL: 1

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0e978e436f34 [wpt PR 39064] - Fix a TextIterator crash in Greek text-transform, a=testonly
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 113 Branch
You need to log in before you can comment on or make changes to this bug.