Wrap event handlers in Client Hints tests in t.step_func (web-platf…

…orm-tests#17164)

Without this, any failed assert will cause a harness error instead of
failing the test.

Note that some problematic use like
`win.addEventListener('load', acceptChLifetimeLoaded, false)` remains,
but fixing these would require introducing new tests.

client-hints/accept_ch.tentative.https.html

@@ -38,15 +38,15 @@
 }, "Precondition: Test that the browser does not have client hints preferences cached");
 
 async_test(t => {
-  window.addEventListener('message', function(e) {
+  window.addEventListener('message', t.step_func(function(e) {
     if(!e.source.location.pathname.includes("do_not_expect_client_hints_headers.html")) {
       return;
     }
     if(typeof e.data != "string")
       return;
     assert_equals(e.data, "PASS");
     t.done();
-  })
+  }));
 }, "Loading of resources/do_not_expect_client_hints_headers.html did not finish.");
 
 function acceptChLoaded() {

client-hints/accept_ch_lifetime.tentative.https.html

@@ -38,15 +38,15 @@
 }, "Precondition: Test that the browser does not have client hints preferences cached");
 
 async_test(t => {
-  window.addEventListener('message', function(e) {
+  window.addEventListener('message', t.step_func(function(e) {
     if(!e.source.location.pathname.includes("expect_client_hints_headers.html")) {
       return;
     }
     if(typeof e.data != "string")
       return;
     assert_equals(e.data, "PASS");
     t.done();
-})
+  }));
 }, "Loading of resources/expect_client_hints_headers.html did not finish.");
 
 

client-hints/accept_ch_lifetime_cross_origin_iframe.tentative.sub.https.html

@@ -40,15 +40,15 @@
 }, "Precondition: Test that the browser does not have client hints preferences cached");
 
 async_test(t => {
-  window.addEventListener('message', function(e) {
+  window.addEventListener('message', t.step_func(function(e) {
     if(!e.source.location.pathname.includes("do_not_expect_client_hints_headers.html")) {
       return;
     }
     if(typeof e.data != "string")
       return;
     assert_equals(e.data, "PASS");
     t.done();
-  })
+  }));
 }, "Loading of resources/do_not_expect_client_hints_headers.html did not finish.");
 
 function acceptChLifetimeLoaded() {

client-hints/accept_ch_lifetime_subresource.tentative.https.html

@@ -56,15 +56,15 @@
 }, "Test receiving Accept-CH-Lifetime header");
 
 async_test(t => {
-  window.addEventListener('message', function(e) {
+  window.addEventListener('message', t.step_func(function(e) {
     if(!e.source.location.pathname.includes("do_not_expect_client_hints_headers.html")) {
       return;
     }
     if(typeof e.data != "string")
       return;
     assert_equals(e.data, "PASS");
     t.done();
-  })
+  }));
 }, "Loading of resources/do_not_expect_client_hints_headers.html did not finish.");
 
 </script>

client-hints/http_equiv_accept_ch_lifetime.tentative.https.html

@@ -38,15 +38,15 @@
 }, "Precondition: Test that the browser does not have client hints preferences cached");
 
 async_test(t => {
-  window.addEventListener('message', function(e) {
+  window.addEventListener('message', t.step_func(function(e) {
     if(!e.source.location.pathname.includes("expect_client_hints_headers.html")) {
       return;
     }
     if(typeof e.data != "string")
       return;
     assert_equals(e.data, "PASS");
     t.done();
-})
+  }));
 }, "Loading of resources/expect_client_hints_headers.html did not finish.");
 
 function acceptChLifetimeLoaded() {

client-hints/http_equiv_accept_ch_lifetime_cross_origin_iframe.tentative.sub.https.html

@@ -40,15 +40,15 @@
 }, "Precondition: Test that the browser does not have client hints preferences cached");
 
 async_test(t => {
-  window.addEventListener('message', function(e) {
+  window.addEventListener('message', t.step_func(function(e) {
     if(!e.source.location.pathname.includes("do_not_expect_client_hints_headers.html")) {
       return;
     }
     if(typeof e.data != "string")
       return;
     assert_equals(e.data, "PASS");
     t.done();
-  })
+  }));
 }, "Loading of resources/do_not_expect_client_hints_headers.html did not finish.");
 
 function acceptChLifetimeLoaded() {

client-hints/http_equiv_accept_ch_lifetime_subresource.tentative.https.html

@@ -58,15 +58,15 @@
 }, "Test receiving Accept-CH-Lifetime header");
 
 async_test(t => {
-  window.addEventListener('message', function(e) {
+  window.addEventListener('message', t.step_func(function(e) {
     if(!e.source.location.pathname.includes("do_not_expect_client_hints_headers.html")) {
       return;
     }
     if(typeof e.data != "string")
       return;
     assert_equals(e.data, "PASS");
     t.done();
-  })
+  }));
 }, "Loading of resources/do_not_expect_client_hints_headers.html did not finish.");