Closed Bug 1660522 Opened 4 years ago Closed 4 years ago

[wpt-sync] Sync PR 25187 - Reland "FontAccess: Replace web_tests with WPT tests"

Categories

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

task

Tracking

(firefox82 fixed)

RESOLVED FIXED
82 Branch
Tracking Status
firefox82 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Olivier Yiptong <oyiptong@chromium.org> wrote:

Reland "FontAccess: Replace web_tests with WPT tests"

This is a reland of Ice3b2bd0589a8aff83a0f50f7bcd2d2ff6938989

Original change's description:

Revert "FontAccess: Replace web_tests with WPT tests"

This reverts commit cfb42f4aed1af85a1ae06a55c698e106213ac479.

Reason for revert: The upstream CL of this CL (https://crrev.com/c/2364009) is likely the cause of build breakage for Win10 Tests x64 build starting from: https://ci.chromium.org/p/chromium/builders/ci/Win10%20Tests%20x64/48700

Original change's description:

FontAccess: Replace web_tests with WPT tests

Replaces web_tests with WPT tests to ensure cross-browser compatibility
in the future.

Bug: 535764
Change-Id: Ice3b2bd0589a8aff83a0f50f7bcd2d2ff6938989
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2365332
Commit-Queue: Olivier Yiptong \<oyiptong@chromium.org>
Reviewed-by: Joshua Bell \<jsbell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#800345}

TBR=jsbell@chromium.org,oyiptong@chromium.org

Change-Id: I6f85c0cb68f8c30ab29a618f3365d2473c9a8b99
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 535764
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2367585
Reviewed-by: Maggie Cai \<mxcai@chromium.org>
Commit-Queue: Maggie Cai \<mxcai@chromium.org>
Cr-Commit-Position: refs/heads/master@{#800478}

TBR=jsbell@chromium.org,mxcai@chromium.org,oyiptong@chromium.org

Not skipping CQ checks because this is a reland.

Bug: 535764
Change-Id: I360c6c2fba2ec5bb665430b670df77ed645aa794
Reviewed-on: https://chromium-review.googlesource.com/2368444
WPT-Export-Revision: 4414bb7e2d779652087c793665f99c71fc76e491

Whiteboard: [wptsync downstream] → [wptsync downstream error]
Whiteboard: [wptsync downstream error] → [wptsync downstream]

CI Results

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

Total 2 tests and 2 subtests

Status Summary

Firefox

OK : 2
FAIL: 3

Chrome

OK : 2
FAIL: 3

Safari

OK : 2
FAIL: 3

Links

GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

/font-access/font_access-blob.tentative.https.window.html
font_access-blob: FAIL (Chrome: FAIL, Safari: FAIL)
/font-access/font_access-enumeration.tentative.https.window.html
font_access-enumeration: FAIL (Chrome: FAIL, Safari: FAIL)
query(): fails if there is no user activation: FAIL (Chrome: FAIL, Safari: FAIL)

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/693f3b65a61e
[wpt PR 25187] - Reland "FontAccess: Replace web_tests with WPT tests", a=testonly
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 82 Branch
You need to log in before you can comment on or make changes to this bug.