[COOP] access reporting: Use infinite timeout for receive.

Some browsers do not implement COOP reporting. As a result, no reports
are received after a 2.5s timeout.

Waiting 2.5s many time can reach the global timeout. Sometime, this
flakes in between FAIL and TIMEOUT.

This patch makes the timeout to be infinite by default to get consistent
failure.

Bug: 1090273
Change-Id: I081b9c8e944fc50b391753ce0948392b88d3fd0e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2428976Reviewed-by: Pâris Meuleman <pmeuleman@chromium.org>
Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#810604}

third_party/blink/web_tests/external/wpt/html/cross-origin-opener-policy/reporting/access-reporting/access-from-coop-page-to-openee_coop-ro.https.html

@@ -63,7 +63,6 @@ let runTest = (openee_redirect, name) => promise_test(async t => {
   // 5. Check a report sent to the opener.
   let report =
     await receiveReport(opener_report_token, "access-from-coop-page-to-openee")
-  assert_not_equals(report, "timeout", "Report not received");
   assert_equals(report.type, "coop");
   assert_equals(report.url, opener_url.replace(/"/g, '%22'));
   assert_equals(report.body.disposition, "reporting");

third_party/blink/web_tests/external/wpt/html/cross-origin-opener-policy/reporting/access-reporting/access-from-coop-page-to-openee_coop-ro_cross-origin.https.html

@@ -64,7 +64,6 @@ let runTest = (openee_redirect, name) => promise_test(async t => {
   // 5. Check a report sent to the opener.
   let report =
     await receiveReport(opener_report_token, "access-from-coop-page-to-openee")
-  assert_not_equals(report, "timeout", "Report not received");
   assert_equals(report.type, "coop");
   assert_equals(report.url, opener_url.replace(/"/g, '%22'));
   assert_equals(report.body.disposition, "reporting");

third_party/blink/web_tests/external/wpt/html/cross-origin-opener-policy/reporting/access-reporting/access-from-coop-page-to-opener_coop-ro.https.html

@@ -40,7 +40,6 @@ let runTest = (openee_redirect, name) => promise_test(async t => {
   // 2. Check a report is sent to the openee.
   let report =
     await receiveReport(report_token, "access-from-coop-page-to-opener")
-  assert_not_equals(report, "timeout", "Report not received");
   assert_equals(report.type, "coop");
   assert_equals(report.url, openee_url.replace(/"/g, '%22'));
   assert_equals(report.body.disposition, "reporting");

third_party/blink/web_tests/external/wpt/html/cross-origin-opener-policy/reporting/access-reporting/access-from-coop-page-to-opener_coop-ro_cross-origin.https.html

@@ -41,7 +41,6 @@ let runTest = (openee_redirect, name) => promise_test(async t => {
   // 2. Check a report is sent to the openee.
   let report =
     await receiveReport(report_token, "access-from-coop-page-to-opener")
-  assert_not_equals(report, "timeout", "Report not received");
   assert_equals(report.type, "coop");
   assert_equals(report.url, openee_url.replace(/"/g, '%22'));
   assert_equals(report.body.disposition, "reporting");

third_party/blink/web_tests/external/wpt/html/cross-origin-opener-policy/reporting/access-reporting/access-from-coop-page-to-other_coop-ro.https.html

@@ -75,7 +75,6 @@ promise_test(async t => {
   // 6. Check a report is sent to the openee.
   let report =
     await receiveReport(report_token, "access-from-coop-page-to-other")
-  assert_not_equals(report, "timeout", "Report not received");
   assert_equals(report.type, "coop");
   assert_equals(report.url, opener_url.replace(/"/g, '%22'));
   assert_equals(report.body.disposition, "reporting");

third_party/blink/web_tests/external/wpt/html/cross-origin-opener-policy/reporting/access-reporting/access-from-coop-page-to-other_coop-ro_cross-origin.https.html

@@ -76,7 +76,6 @@ promise_test(async t => {
   // 6. Check a report is sent to the openee.
   let report =
     await receiveReport(report_token, "access-from-coop-page-to-other")
-  assert_not_equals(report, "timeout", "Report not received");
   assert_equals(report.type, "coop");
   assert_equals(report.url, opener_url.replace(/"/g, '%22'));
   assert_equals(report.body.disposition, "reporting");

third_party/blink/web_tests/external/wpt/html/cross-origin-opener-policy/reporting/access-reporting/access-to-coop-page-from-openee_coop-ro.https.html

@@ -57,7 +57,6 @@ let runTest = (openee_redirect, name) => promise_test(async t => {
   // 4. Check a report sent to the opener.
   let report =
     await receiveReport(opener_report_token, "access-to-coop-page-from-openee")
-  assert_not_equals(report, "timeout", "Report not received");
   assert_equals(report.type, "coop");
   assert_equals(report.url, opener_url.replace(/"/g, '%22'));
   assert_equals(report.body.disposition, "reporting");

third_party/blink/web_tests/external/wpt/html/cross-origin-opener-policy/reporting/access-reporting/access-to-coop-page-from-openee_coop-ro_cross-origin.https.html

@@ -58,7 +58,6 @@ let runTest = (openee_redirect, name) => promise_test(async t => {
   // 4. Check a report sent to the opener.
   let report =
     await receiveReport(opener_report_token, "access-to-coop-page-from-openee")
-  assert_not_equals(report, "timeout", "Report not received");
   assert_equals(report.type, "coop");
   assert_equals(report.url, opener_url.replace(/"/g, '%22'));
   assert_equals(report.body.disposition, "reporting");

third_party/blink/web_tests/external/wpt/html/cross-origin-opener-policy/reporting/access-reporting/access-to-coop-page-from-opener_coop-ro.https.html

@@ -47,7 +47,6 @@ let runTest = (openee_redirect, name) => promise_test(async t => {
   // 3. Check a report is sent to the openee.
   let report =
     await receiveReport(report_token, "access-to-coop-page-from-opener")
-  assert_not_equals(report, "timeout", "Report not received");
   assert_equals(report.type, "coop");
   assert_equals(report.url, openee_url.replace(/"/g, '%22'));
   assert_equals(report.body.disposition, "reporting");

third_party/blink/web_tests/external/wpt/html/cross-origin-opener-policy/reporting/access-reporting/access-to-coop-page-from-opener_coop-ro_cross-origin.https.html

@@ -48,7 +48,6 @@ let runTest = (openee_redirect, name) => promise_test(async t => {
   // 3. Check a report is sent to the openee.
   let report =
     await receiveReport(report_token, "access-to-coop-page-from-opener")
-  assert_not_equals(report, "timeout", "Report not received");
   assert_equals(report.type, "coop");
   assert_equals(report.url, openee_url.replace(/"/g, '%22'));
   assert_equals(report.body.disposition, "reporting");

third_party/blink/web_tests/external/wpt/html/cross-origin-opener-policy/reporting/access-reporting/access-to-coop-page-from-other_coop-ro.https.html

@@ -63,7 +63,6 @@ promise_test(async t => {
   // 4. Check a report sent to the openee.
   let report =
     await receiveReport(openee_report_token, "access-to-coop-page-from-other")
-  assert_not_equals(report, "timeout", "Report not received");
   assert_equals(report.type, "coop");
   assert_equals(report.url, openee_url.replace(/"/g, '%22'));
   assert_equals(report.body.disposition, "reporting");

third_party/blink/web_tests/external/wpt/html/cross-origin-opener-policy/reporting/access-reporting/access-to-coop-page-from-other_coop-ro_cross-origin.https.html

@@ -64,7 +64,6 @@ promise_test(async t => {
   // 4. Check a report sent to the openee.
   let report =
     await receiveReport(openee_report_token, "access-to-coop-page-from-other")
-  assert_not_equals(report, "timeout", "Report not received");
   assert_equals(report.type, "coop");
   assert_equals(report.url, openee_url.replace(/"/g, '%22'));
   assert_equals(report.body.disposition, "reporting");

third_party/blink/web_tests/external/wpt/html/cross-origin-opener-policy/reporting/access-reporting/property.https.html

@@ -58,7 +58,6 @@ origin.forEach(([origin_name, origin]) => {
 
       // 3. Check a reports is sent to the opener.
       let report = await receiveReport(report_token, "access-to-coop-page")
-      assert_not_equals(report, "timeout", "Report not received");
       assert_equals(report.body.property, property);
 
     }, `${origin_name} > ${op}`);

third_party/blink/web_tests/external/wpt/html/cross-origin-opener-policy/reporting/access-reporting/report-to-both_coop-ro.https.html

@@ -83,9 +83,6 @@ promise_test(async test => {
   let report_openee =
     await receiveReport(openee_report_token, "access-to-coop-page-from-opener")
 
-  assert_not_equals(report_openee, "timeout", "Report openee not received");
-  assert_not_equals(report_opener, "timeout", "Report opener not received");
-
   assert_generic_coop_report(report_openee);
   assert_generic_coop_report(report_opener);
 
@@ -109,9 +106,6 @@ promise_test(async test => {
   let report_openee =
     await receiveReport(openee_report_token, "access-from-coop-page-to-opener")
 
-  assert_not_equals(report_openee, "timeout", "Report openee not received");
-  assert_not_equals(report_opener, "timeout", "Report opener not received");
-
   assert_generic_coop_report(report_openee);
   assert_generic_coop_report(report_opener);
 

third_party/blink/web_tests/external/wpt/html/cross-origin-opener-policy/reporting/access-reporting/reporting-observer.html

@@ -186,7 +186,6 @@ promise_test(async t => {
   send(opener_iframe_token, `tryAccess(openee);`);
 
   let reports = await receive(this_window_token);
-  assert_not_equals(reports, "timeout", "No report received.");
   reports = JSON.parse(reports);
   assert_equals(reports.length, 1, "No report received.");
   assert_equals(reports[0].type, "coop-access-violation");
@@ -248,7 +247,7 @@ promise_test(async t => {
   //    from the main document aren't notified.
   send(opener_iframe_token, `tryAccess(openee);`);
 
-  let reports = await receive(this_window_token);
+  let reports = await receive(this_window_token, 2000);
   assert_equals(reports, "timeout", "Unexpected report received.");
 }, "Access from cross-site iframe")
 

third_party/blink/web_tests/external/wpt/html/cross-origin-opener-policy/reporting/navigation-reporting/reporting-redirect-with-same-origin-allow-popups.https.html

@@ -101,7 +101,6 @@ function redirect_test( popup_origin ) {
     let report = await receiveReport(
         opener_report_token,
         "navigation-to-response");
-    assert_not_equals(report, "timeout", "Report not received");
     assert_equals(report.type, "coop");
     assert_equals(report.body.disposition, "enforce");
     assert_equals(report.body.effectivePolicy, "same-origin-allow-popups");

third_party/blink/web_tests/external/wpt/html/cross-origin-opener-policy/reporting/resources/dispatcher.js

@@ -17,8 +17,8 @@ const send = function(uuid, message) {
   });
 }
 
-const receive = async function(uuid) {
-  const timeout = 2500;
+const receive = async function(uuid, maybe_timeout) {
+  const timeout = maybe_timeout || Infinity;
   const retry_delay = 100;
   for(let i = 0; i * retry_delay < timeout; ++i) {
     let response = await fetch(dispatcher_url + `?uuid=${uuid}`);