COOP: change handling of redirects

This CL changes the handling of redirects in COOP to match whatwg/html#5739. Instead of comparing a response to the current document, we will compare it to the last entry in the redirect chain. Change-Id: I73a3926a8827733974d7c08b7f3553da06435ac3 Bug: 1108292 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2339988 Commit-Queue: Camille Lamy <[email protected]> Reviewed-by: Arthur Sonzogni <[email protected]> Reviewed-by: Pâris Meuleman <[email protected]> Cr-Commit-Position: refs/heads/master@{#798083} GitOrigin-RevId: f338d7dc22470768bb78f1766e8816b15de30884
mjfroman · Aug 14, 2020 · 6cc5850 · 6cc5850
1 parent f36b979
commit 6cc5850
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 12 deletions.
diff --git a/...tml/cross-origin-opener-policy/popup-redirect-same-origin-allow-popups.https-expected.txt b/...tml/cross-origin-opener-policy/popup-redirect-same-origin-allow-popups.https-expected.txt
diff --git a/.../navigation-reporting/reporting-redirect-with-same-origin-allow-popups.https-expected.txt b/.../navigation-reporting/reporting-redirect-with-same-origin-allow-popups.https-expected.txt
diff --git a/...eporting/navigation-reporting/reporting-redirect-with-same-origin-allow-popups.https.html b/...eporting/navigation-reporting/reporting-redirect-with-same-origin-allow-popups.https.html
@@ -103,10 +103,8 @@
         "navigation-to-document");
     assert_not_equals(report, "timeout", "Report not received");
     assert_equals(report.type, "coop");
-    assert_equals(report.url, openee_redirect_url.replace(/"/g, '%22'));
     assert_equals(report.body["disposition"], "enforce");
     assert_equals(report.body["effective-policy"], "same-origin-allow-popups");
-    assert_equals(report.body["document-uri"], openee_url);
   }, `${popup_origin.name} openee redirected to same-origin with same-origin-allow-popups`);
 }