Closed Bug 1984151 Opened 7 months ago Closed 7 months ago

[wpt-sync] Sync PR 54431 - Revert "blink/css: Implement text-transform: full-width"

Categories

(Testing :: web-platform-tests, task, P4)

task

Tracking

(firefox144 fixed)

RESOLVED FIXED
144 Branch
Tracking Status
firefox144 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Brad Triebwasser <btriebw@chromium.org> wrote:

Revert "blink/css: Implement text-transform: full-width"

This reverts commit 10bca80bad89c58b0187e1059b2a4cecc9759f51.

Reason for revert: Causing WPT failures on Windows 10 bots (https://ci.chromium.org/ui/p/chromium/builders/ci/WebKit%20Win10/160052/overview)

Bug: 41297237
Original change's description:

blink/css: Implement text-transform: full-width

Adds support for CSS text-transform: full-width specified in CSS Text
Level 3.

https://www.w3.org/TR/css-text-3/#text-transform-property

This value transforms all narrow characters that have a full-width
mapping in Unicode to their wide equivalents. This includes ASCII,
Japanese Katakana, Korean Hangul, and several special symbols.

The concrete list of character conversions follows the one used by
Firefox, which was compiled directly from the Unicode database. See:

https://bugzilla.mozilla.org/show_bug.cgi?id=774560

This CL includes a new WPT test:

/css/css-text/text-transform/text-transform-fullwidth-010.html

Bug: 41297237
Change-Id: I280d4c5df9d221167bfb51c5988f485a03bdfad4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6790585
Commit-Queue: Felipe Erias \<felipeerias@igalia.com>
Reviewed-by: Kent Tamura \<tkent@chromium.org>
Reviewed-by: Koji Ishii \<kojii@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1503902}

Bug: 41297237
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Change-Id: I98650c492244d7cf44382aa617d0755f21d3a9e8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6867371
Bot-Commit: Rubber Stamper \<rubber-stamper@appspot.gserviceaccount.com>
Commit-Queue: Brad Triebwasser \<btriebw@chromium.org>
Owners-Override: Brad Triebwasser \<btriebw@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1504029}

PR 54431 applied with additional changes from upstream: 0d8ca16bee91e5b73875063350ae4e1f7a1051f9
The PR was not expected to affect any tests, but the try push wasn't a success. Check the try results for infrastructure issues
Test result changes from PR not available.
Pushed by wptsync@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/5e0b6e1cb371 https://hg.mozilla.org/integration/autoland/rev/f882e3da6f1e [wpt PR 54431] - Revert "blink/css: Implement text-transform: full-width", a=testonly
Status: NEW → RESOLVED
Closed: 7 months ago
Resolution: --- → FIXED
Target Milestone: --- → 144 Branch
You need to log in before you can comment on or make changes to this bug.