Update Sec-CH-UA tests: fix existing tests, and test brand size.

As of WICG/ua-client-hints#179,
https://wicg.github.io/ua-client-hints/#user-agent-brand now requires
that a brand in a brand list not be longer than 32 bytes.

This CL adds a new test for the userAgentData.brands interface, and
modifies the existing sec-ch-ua HTTP tests.

Also, the existing sec-ch-ua HTTP tests were updated to more closely
reflect the current state of UA-CH ("UA" is not a valid token, and
"Accept-CH-Lifetime" is not shipping).

Bug: 1263180
Change-Id: Ifa7cbf18d67271e2dd1dc1d485f432d4ca9327fa

Author: miketaylr

client-hints/resources/sec-ch-ua.py

@@ -1,8 +1,6 @@
 def main(request, response):
-    ua = request.headers.get(b'sec-ch-ua', b'')
+    ua = request.headers.get(b'Sec-CH-UA', b'')
     response.headers.set(b"Content-Type", b"text/html")
-    response.headers.set(b"Accept-CH", b"UA")
-    response.headers.set(b"Accept-CH-Lifetime", b"10")
     response.content = b'''
 <script>
   window.opener.postMessage({ header: '%s' }, "*");

client-hints/sec-ch-ua.https.html

@@ -3,14 +3,14 @@
 <script src=/resources/testharness.js></script>
 <script src=/resources/testharnessreport.js></script>
 <script>
-  var minor = "";
+  var sec_ch_ua_header = "";
   promise_test(t => {
     return new Promise((resolve, reject) => {
       var w;
       window.onmessage = e => {
         try {
-          assert_not_equals(e.data.header, "", "The `Sec-CH-UA` header is delivered.");
-          minor = e.data.header;
+          assert_not_equals(e.data.header, "", "`Sec-CH-UA` is sent.");
+          sec_ch_ua_header = e.data.header;
         } catch (ex) {
           reject(ex);
         }
@@ -19,15 +19,18 @@
       };
       w = window.open("./resources/sec-ch-ua.py");
     });
-  }, "Open HTTPS window prior to opt-in: `Sec-CH-UA` header with minor version.")
+  }, "Open HTTPS window: `Sec-CH-UA` header returned by default.");
 
   promise_test(t => {
     return new Promise((resolve, reject) => {
       var w;
       window.onmessage = e => {
         try {
-          assert_not_equals(e.data.header, "", "The `Sec-CH-UA` header is delivered.");
-          assert_equals(e.data.header, minor, "The `Sec-CH-UA` header did not change after the opt-in.");
+          assert_not_equals(e.data.header, "",
+            "The `Sec-CH-UA` header is delivered.");
+          assert_equals(e.data.header,
+            sec_ch_ua_header,
+            "The `Sec-CH-UA` header did not change between requests.");
         } catch (ex) {
           reject(ex);
         }
@@ -36,6 +39,30 @@
       };
       w = window.open("./resources/sec-ch-ua.py");
     });
-  }, "Open HTTPS window post-opt-in: `Sec-CH-UA` header with minor version.")
+  }, "Open HTTPS window: `Sec-CH-UA` header is consistent across versions.");
+
+  promise_test(t => {
+    return new Promise((resolve, reject) => {
+      var w;
+      window.onmessage = e => {
+        try {
+          let header = e.data.header;
+          assert_true(header.split(", ").every((brand) => {
+            let brandEnd = brand.indexOf(";v=");
+            assert_true(brandEnd !== -1,
+              "A well-formed Sec-CH-UA header must have version (v=) params");
+            /* 32 + 2, becuase of the extra quotes padding the brand,
+               e.g. '"lol";v=22"' */
+            return brandEnd < 34;
+          }));
+        } catch (ex) {
+          reject(ex);
+        }
+        w.close();
+        resolve();
+      };
+      w = window.open("./resources/sec-ch-ua.py");
+    });
+  }, "Open HTTPS window: No brand in `Sec-CH-UA` header is > than 32 chars.");
 </script>
 </head>

ua-client-hints/useragentdata.https.any.js

@@ -0,0 +1,7 @@
+// META: title=tests for navigator.userAgentData
+
+test(t => {
+  const brands = navigator.userAgentData.brands;
+  assert_true(brands.every(brand => brand.brand.length < 32),
+    "No brand should be longer than 32 characters.");
+});