Closed Bug 1688329 Opened 3 years ago Closed 3 years ago

[wpt-sync] Sync PR 27296 - [Code Health] FontAccess: Consolidate WPT feature detection assertions

Categories

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

Product:
Testing
Component:
web-platform-tests
Type:
task

Tracking

(firefox87 fixed)

Status:
RESOLVED FIXED
Milestone:
87 Branch
Tracking Flags:
Tracking Status
firefox87 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

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

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

b'Olivier Yiptong <oyiptong@chromium.org>' wrote:

[Code Health] FontAccess: Consolidate WPT feature detection assertions

This change consolidates feature detection assertions in Local Font
Access WPT tests in the test helper to diminish line noise.

Change-Id: I9a95504c7ddc82c741ee5ca32d456b20f557905b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2643794
Auto-Submit: Olivier Yiptong \<oyiptong@chromium.org>
Commit-Queue: Joshua Bell \<jsbell@chromium.org>
Reviewed-by: Joshua Bell \<jsbell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#846341}

PR 27296 applied with additional changes from upstream: 99ab2e81fd22fbcb80124920df83818a70024e03

CI Results

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

Total 3 tests and 13 subtests

Status Summary

Firefox

OK : 3
FAIL: 16

Chrome

OK : 3
FAIL: 16

Safari

OK : 3
FAIL: 16

Links

Gecko CI (Treeherder)
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-chooser-selection.tentative.manual.https.html
query() with selection works: FAIL (Chrome: FAIL, Safari: FAIL)
query() with empty selection bag works: FAIL (Chrome: FAIL, Safari: FAIL)
/font-access/font_access-enumeration.tentative.https.window.html
font_access-enumeration: FAIL (Chrome: FAIL, Safari: FAIL)
font_access-enumeration 1: FAIL (Chrome: FAIL, Safari: FAIL)
font_access-enumeration 2: FAIL (Chrome: FAIL, Safari: FAIL)
font_access-enumeration 3: FAIL (Chrome: FAIL, Safari: FAIL)
font_access-enumeration 4: FAIL (Chrome: FAIL, Safari: FAIL)
font_access-enumeration 5: FAIL (Chrome: FAIL, Safari: FAIL)
font_access-enumeration 6: FAIL (Chrome: FAIL, Safari: FAIL)
font_access-enumeration 7: FAIL (Chrome: FAIL, Safari: FAIL)
font_access-enumeration 8: FAIL (Chrome: FAIL, Safari: FAIL)
font_access-enumeration 9: FAIL (Chrome: FAIL, Safari: FAIL)
font_access-enumeration 10: FAIL (Chrome: FAIL, Safari: FAIL)
font_access-enumeration 11: FAIL (Chrome: FAIL, Safari: FAIL)
font_access-enumeration 12: FAIL (Chrome: FAIL, Safari: FAIL)

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7e2c6edc71b1
[wpt PR 27296] - [Code Health] FontAccess: Consolidate WPT feature detection assertions, a=testonly
https://hg.mozilla.org/integration/autoland/rev/3fb06ba18160
[wpt PR 27296] - Update wpt metadata, a=testonly

https://hg.mozilla.org/mozilla-central/rev/7e2c6edc71b1
https://hg.mozilla.org/mozilla-central/rev/3fb06ba18160

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