Skip to content

Commit 1691e2c

Browse files
[Re-Factor Client Hints] (5) Remove (unlaunched) lang client hint (#30781)
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 <[email protected]> Commit-Queue: Matt Falkenhagen <[email protected]> Reviewed-by: Matt Falkenhagen <[email protected]> Reviewed-by: David Dorwin <[email protected]> Reviewed-by: Yoav Weiss <[email protected]> Reviewed-by: Dominick Ng <[email protected]> Cr-Commit-Position: refs/heads/main@{#921893} Co-authored-by: Ari Chivukula <[email protected]>
1 parent 26847f2 commit 1691e2c

File tree

4 files changed

+1
-6
lines changed

4 files changed

+1
-6
lines changed

client-hints/accept-ch-cache-revalidation.https.html

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@
2323
"rtt",
2424
"downlink",
2525
"ect",
26-
"lang",
2726
"sec-ch-ua",
2827
"sec-ch-ua-arch",
2928
"sec-ch-ua-platform",

client-hints/http-equiv-accept-ch-merge.https.html

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
<html>
22
<head>
33
<meta http-equiv="Accept-CH" content="viewport-width, rtt">
4-
<meta http-equiv="Accept-CH" content="downlink, ect, lang, sec-ch-prefers-color-scheme">
4+
<meta http-equiv="Accept-CH" content="downlink, ect, sec-ch-prefers-color-scheme">
55
<body>
66
<script src="/resources/testharness.js"></script>
77
<script src="/resources/testharnessreport.js"></script>
@@ -19,7 +19,6 @@
1919
// Verify that the browser includes client hints in the headers.
2020
assert_true(r.headers.has("device-memory-received"), "device-memory-received");
2121
assert_true(r.headers.has("dpr-received"), "dpr-received");
22-
assert_true(r.headers.has("lang-received"), "lang-received");
2322
assert_true(r.headers.has("viewport-width-received"), "viewport-width-received");
2423

2524
assert_true(r.headers.has("rtt-received"), "rtt-received");

client-hints/resources/echo-client-hints-received.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,6 @@ def main(request, response):
2222
response.headers.set(b"downlink-received", request.headers.get(b"downlink"))
2323
if b"ect" in request.headers:
2424
response.headers.set(b"ect-received", request.headers.get(b"ect"))
25-
if b"Lang" in request.headers:
26-
response.headers.set(b"lang-received", request.headers.get(b"Lang"))
2725
if b"sec-ch-ua-mobile" in request.headers:
2826
response.headers.set(b"mobile-received", request.headers.get(b"sec-ch-ua-mobile"))
2927
if b"sec-ch-prefers-color-scheme" in request.headers:

client-hints/resources/stale-echo-client-hints.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@ def main(request, response):
1616
b"rtt",
1717
b"downlink",
1818
b"ect",
19-
b"lang",
2019
b"sec-ch-ua",
2120
b"sec-ch-ua-arch",
2221
b"sec-ch-ua-platform",

0 commit comments

Comments
 (0)