Closed Bug 1576493 Opened 7 years ago Closed 7 years ago

[wpt-sync] Sync PR 18660 - [css-writing-modes] Remove text-orientation-script-001*

Categories

(Core :: CSS Parsing and Computation, task, P4)

Product:
Core
Component:
CSS Parsing and Computation
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla71
Tracking Flags:
Tracking Status
firefox71 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

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

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

Koji Ishii <kojii@chromium.org> wrote:

[css-writing-modes] Remove text-orientation-script-001*

This patch removes text-orientation-script-001* tests.

At the point this set of tests were developed, UA had to
implement computing the Vertical_Orientation property as
defiend in UTR#50.

Since then, UTR#50 turned to UAX#50 and ICU63 added this
property that the benefit of testing all code points became
less important.

This set of tests have following downsides:

  1. Specific to a version of UAX#50 that the tests and all UA
    must be in sync.
  2. Takes very long time, often timeouts in Edge and WebKit.
  3. Is sensitive to the undefined behavior of
    getBoundingClientRect.
  4. It's not easy to maintain on every revision update.

Given the benefit went down, and we have tests for specific
cases (other text-orientation-* tests), this set of tests
has done its role hence removing.

Component: web-platform-tests → CSS Parsing and Computation
Product: Testing → Core
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/0d7b80677817 [wpt PR 18660] - [css-writing-modes] Remove text-orientation-script-001*, a=testonly
Result changes from PR not available.

https://hg.mozilla.org/mozilla-central/rev/0d7b80677817

Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox71: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla71
You need to log in before you can comment on or make changes to this bug.