1731321 - [wpt-sync] Sync PR 30781 - [Re-Factor Client Hints] (5) Remove (unlaunched) `lang` client hint

Status: RESOLVED FIXED

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

Description

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

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

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

b'Ari Chivukula <[e-mail]>' wrote:

[Re-Factor Client Hints] (5) Remove (unlaunched) lang client hint

Now that we've cleaned up the array indexing that was blocking this,
let's move forward with removing this unused client hint.

This is a part of a series of CLs to re-factor the client hints code:
(1) Label hints with legacy names
(2) Fix the naming of (unlaunched) lang client hint
(3) Consolidate client hint name lists
(4) Migrate from array to map indexing for client hint names
(5) Remove (unlaunched) lang client hint

Bug: 1227043
Change-Id: I57404516cd1f39f738f0fbd7c202de7adf063d98
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3160286
Auto-Submit: Ari Chivukula \<[e-mail]>
Commit-Queue: Matt Falkenhagen \<[e-mail]>
Reviewed-by: Matt Falkenhagen \<[e-mail]>
Reviewed-by: David Dorwin \<[e-mail]>
Reviewed-by: Yoav Weiss \<[e-mail]>
Reviewed-by: Dominick Ng \<[e-mail]>
Cr-Commit-Position: refs/heads/main@{#921893}

Comment 1

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

PR 30781 applied with additional changes from upstream: eb90781d3ec0766b56286dbb19ecb31dc2c8369d

Comment 2

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=aa706b2acb280e325e63e4e42702ddf0e997fa8c

Comment 3

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

CI Results

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

Total 10 tests and 1 subtests

Status Summary

Firefox

OK : 10
PASS: 8
FAIL: 5

Chrome

OK : 10
PASS: 9
FAIL: 4

Safari

OK : 10
PASS: 7
FAIL: 6

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

/client-hints/accept-ch-answers.sub.https.html
Accept-CH header test: FAIL (Chrome: PASS, Safari: FAIL)
/client-hints/accept-ch-feature-policy.sub.https.html
Accept-CH header test: FAIL (Chrome: PASS, Safari: FAIL)
Cross-Origin Accept-CH header test: FAIL (Chrome: PASS, Safari: FAIL)
/client-hints/accept-ch-no-feature-policy.sub.https.html
Accept-CH header test: FAIL (Chrome: PASS, Safari: FAIL)
/client-hints/http-equiv-accept-ch-merge.https.html
Accept-CH header test: FAIL (Chrome: PASS, Safari: FAIL)

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/fc751892e985
[wpt PR 30781] - [Re-Factor Client Hints] (5) Remove (unlaunched) `lang` client hint, a=testonly

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/36ee490d1bdd
[wpt PR 30781] - [Re-Factor Client Hints] (5) Remove (unlaunched) `lang` client hint, a=testonly

Comment 6

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/36ee490d1bdd