Bug 1659787 [wpt PR 25072] - COOP: add reporting to redirects, a=test…

…only Automatic update from web-platform-tests COOP: add reporting to redirects This CL allows reporting browsing context group switches triggered by redirects and updates the reports sent in this case to the latest version of the spec PR (whatwg/html#5518). Since the status of COOP was becoming hard to track, I moved it to its own class for better encapsulation. Bug: 1059303 Change-Id: Ifafb23073301bd05cd9ce83fdb0b748c28e8a51f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2352880 Commit-Queue: Camille Lamy <[email protected]> Reviewed-by: Arthur Sonzogni <[email protected]> Reviewed-by: Mike West <[email protected]> Reviewed-by: Kinuko Yasuda <[email protected]> Cr-Commit-Position: refs/heads/master@{#799863} -- wpt-commits: 677c57c0e8816b0892cc3ae1c2772189b1bdcf65 wpt-pr: 25072
sidvishnoi · Sep 2, 2020 · a47406e · a47406e
1 parent 2148144
commit a47406e
Show file tree

Hide file tree

Showing 12 changed files with 120 additions and 136 deletions.
diff --git a/...s-origin-opener-policy/reporting/navigation-reporting/report-only-four-reports.https.html b/...s-origin-opener-policy/reporting/navigation-reporting/report-only-four-reports.https.html
@@ -28,10 +28,9 @@
         "report": {
           "body": {
             "disposition": "enforce",
-            "documentURI": `${location.href}`,
             "effectivePolicy": "same-origin-allow-popups",
-            "navigationURI": /uuid=EXECUTOR_UUID$/, // next document URL
-            "type": "navigation-from-document"
+            "nextResponseURL": /uuid=EXECUTOR_UUID$/, // next document URL
+            "type": "navigation-from-response"
           },
           "url": `${location.href}`,
           "type": "coop"
@@ -42,10 +41,9 @@
         "report": {
           "body": {
             "disposition": "reporting",
-            "documentURI": `${location.href}`,
             "effectivePolicy": "same-origin-plus-coep",
-            "navigationURI": /uuid=EXECUTOR_UUID$/, // next document URL
-            "type": "navigation-from-document"
+            "nextResponseURL": /uuid=EXECUTOR_UUID$/, // next document URL
+            "type": "navigation-from-response"
           },
           "url": `${location.href}`,
           "type": "coop"
@@ -56,10 +54,10 @@
         "report": {
           "body": {
             "disposition": "enforce",
-            "documentURI": /uuid=EXECUTOR_UUID$/,
             "effectivePolicy": "same-origin-allow-popups",
-            "navigationURI": `${location.origin}/`, // referrer
-            "type": "navigation-to-document"
+            "previousResponseURL": "",
+            "referrer": `${location.origin}/`, // referrer
+            "type": "navigation-to-response"
           },
           "url": /uuid=EXECUTOR_UUID$/,
           "type": "coop"
@@ -70,10 +68,10 @@
         "report": {
           "body": {
             "disposition": "reporting",
-            "documentURI": /uuid=EXECUTOR_UUID$/,
             "effectivePolicy": "same-origin-plus-coep",
-            "navigationURI": `${location.origin}/`, // referrer
-            "type": "navigation-to-document"
+            "previousResponseURL": "",
+            "referrer": `${location.origin}/`, // referrer
+            "type": "navigation-to-response"
           },
           "url": /uuid=EXECUTOR_UUID$/,
           "type": "coop"

diff --git a/...igin-opener-policy/reporting/navigation-reporting/report-only-from-unsafe-none.https.html b/...igin-opener-policy/reporting/navigation-reporting/report-only-from-unsafe-none.https.html
@@ -28,10 +28,10 @@
         "report": {
           "body": {
             "disposition": "reporting",
-            "documentURI": /uuid=EXECUTOR_UUID$/,
             "effectivePolicy": "same-origin",
-            "navigationURI": `${location.href}`, // previous documnent url
-            "type": "navigation-to-document"
+            "previousResponseURL": `${location.href}`, // previous documnent url
+            "referrer": `${location.origin}/`, // referrer (origin, as dictated by the referrer policy)
+            "type": "navigation-to-response"
           },
           "url": /uuid=EXECUTOR_UUID$/,
           "type": "coop"
@@ -53,10 +53,10 @@
         "report": {
           "body": {
             "disposition": "reporting",
-            "documentURI": /uuid=EXECUTOR_UUID$/,
             "effectivePolicy": "same-origin",
-            "navigationURI": `${location.origin}/`, // referrer (origin, as dictated by the referrer policy)
-            "type": "navigation-to-document"
+            "previousResponseURL": "",
+            "referrer": `${location.origin}/`, // referrer (origin, as dictated by the referrer policy)
+            "type": "navigation-to-response"
           },
           "url": /uuid=EXECUTOR_UUID$/,
           "type": "coop"

diff --git a/...opener-policy/reporting/navigation-reporting/report-only-same-origin-report-to.https.html b/...opener-policy/reporting/navigation-reporting/report-only-same-origin-report-to.https.html
@@ -29,10 +29,9 @@
         "report": {
           "body": {
             "disposition": "reporting",
-            "documentURI": `${location.href}`,
             "effectivePolicy": "same-origin",
-            "navigationURI": /uuid=EXECUTOR_UUID$/, // next document URL
-            "type": "navigation-from-document"
+            "nextResponseURL": /uuid=EXECUTOR_UUID$/, // next document URL
+            "type": "navigation-from-response"
           },
           "url": `${location.href}`,
           "type": "coop"
@@ -55,10 +54,9 @@
         "report": {
           "body": {
             "disposition": "reporting",
-            "documentURI": `${location.href}`,
             "effectivePolicy": "same-origin",
-            "navigationURI": /uuid=EXECUTOR_UUID$/, // next document URL
-            "type": "navigation-from-document"
+            "nextResponseURL": /uuid=EXECUTOR_UUID$/, // next document URL
+            "type": "navigation-from-response"
           },
           "url": `${location.href}`,
           "type": "coop"
@@ -69,10 +67,10 @@
         "report": {
           "body": {
             "disposition": "reporting",
-            "documentURI": /uuid=EXECUTOR_UUID$/,
             "effectivePolicy": "same-origin",
-            "navigationURI": `${location.origin}/`, // referrer
-            "type": "navigation-to-document"
+            "previousResponseURL": "",
+            "referrer": `${location.origin}/`, // referrer
+            "type": "navigation-to-response"
           },
           "url": /uuid=EXECUTOR_UUID$/,
           "type": "coop"

diff --git a/...ss-origin-opener-policy/reporting/navigation-reporting/report-only-same-origin.https.html b/...ss-origin-opener-policy/reporting/navigation-reporting/report-only-same-origin.https.html
@@ -29,10 +29,10 @@
         "report": {
           "body": {
             "disposition": "reporting",
-            "documentURI": /uuid=EXECUTOR_UUID$/,
             "effectivePolicy": "same-origin-plus-coep",
-            "navigationURI": `${location.href}`,
-            "type": "navigation-to-document"
+            "previousResponseURL": `${location.href}`,
+            "referrer": `${location.origin}/`,
+            "type": "navigation-to-response"
           },
           "url": /uuid=EXECUTOR_UUID$/,
           "type": "coop"
@@ -55,10 +55,10 @@
         "report": {
           "body": {
             "disposition": "reporting",
-            "documentURI": /uuid=EXECUTOR_UUID$/,
             "effectivePolicy": "same-origin-plus-coep",
-            "navigationURI": `${location.href}`,
-            "type": "navigation-to-document"
+            "previousResponseURL": `${location.href}`,
+            "referrer": `${location.origin}/`,
+            "type": "navigation-to-response"
           },
           "url": /uuid=EXECUTOR_UUID$/,
           "type": "coop"

diff --git a/...in-opener-policy/reporting/navigation-reporting/reporting-coop-navigated-popup.https.html b/...in-opener-policy/reporting/navigation-reporting/reporting-coop-navigated-popup.https.html
@@ -54,10 +54,9 @@
       "report": {
         "body": {
           "disposition": "enforce",
-          "documentURI": RegExp(`uuid=${noCoopToken}$`),
           "effectivePolicy": "unsafe-none",
-          "navigationURI": RegExp(`uuid=${coopToken}$`),
-          "type": "navigation-from-document"
+          "nextResponseURL": RegExp(`uuid=${coopToken}$`),
+          "type": "navigation-from-response"
         },
         "url": RegExp(`uuid=${noCoopToken}$`),
         "type": "coop"
@@ -68,10 +67,10 @@
       "report": {
         "body": {
           "disposition": "enforce",
-          "documentURI": RegExp(`uuid=${coopToken}$`),
           "effectivePolicy": "same-origin",
-          "navigationURI": RegExp(`uuid=${noCoopToken}$`),
-          "type": "navigation-to-document"
+          "previousResponseURL": RegExp(`uuid=${noCoopToken}$`),
+          "referrer": RegExp(`uuid=${noCoopToken}$`),
+          "type": "navigation-to-response"
         },
         "url": RegExp(`uuid=${coopToken}$`),
         "type": "coop"

diff --git a/...orting/navigation-reporting/reporting-popup-same-origin-allow-popups-report-to.https.html b/...orting/navigation-reporting/reporting-popup-same-origin-allow-popups-report-to.https.html
@@ -28,10 +28,9 @@
         "report": {
           "body": {
             "disposition": "enforce",
-            "documentURI": `${location.href}`,
             "effectivePolicy": "same-origin-allow-popups",
-            "navigationURI": /uuid=EXECUTOR_UUID$/, // next document URL
-            "type": "navigation-from-document"
+            "nextResponseURL": /uuid=EXECUTOR_UUID$/, // next document URL
+            "type": "navigation-from-response"
           },
           "url": `${location.href}`,
           "type": "coop"
@@ -42,10 +41,10 @@
         "report": {
           "body": {
             "disposition": "enforce",
-            "documentURI": /uuid=EXECUTOR_UUID$/,
             "effectivePolicy": "same-origin",
-            "navigationURI": `${location.href}`, // previous documnent url
-            "type": "navigation-to-document"
+            "previousResponseURL": `${location.href}`, // previous documnent url
+            "referrer": `${location.origin}/`, // referrer (origin, as dictated by the referrer policy)
+            "type": "navigation-to-response"
           },
           "url": /uuid=EXECUTOR_UUID$/,
           "type": "coop"
@@ -69,10 +68,9 @@
         "report": {
           "body": {
             "disposition": "enforce",
-            "documentURI": `${location.href}`,
             "effectivePolicy": "same-origin-allow-popups",
-            "navigationURI": /uuid=EXECUTOR_UUID$/, // initial navigation URL
-            "type": "navigation-from-document"
+            "nextResponseURL": /uuid=EXECUTOR_UUID$/, // next document URL
+            "type": "navigation-from-response"
           },
           "url": `${location.href}`,
           "type": "coop"
@@ -83,10 +81,10 @@
         "report": {
           "body": {
             "disposition": "enforce",
-            "documentURI": /uuid=EXECUTOR_UUID$/,
             "effectivePolicy": "same-origin-allow-popups",
-            "navigationURI": `${location.origin}/`, // referrer (origin, as dictated by the referrer policy)
-            "type": "navigation-to-document"
+            "previousResponseURL": ``,
+            "referrer": `${location.origin}/`, // referrer (origin, as dictated by the referrer policy)
+            "type": "navigation-to-response"
           },
           "url": /uuid=EXECUTOR_UUID$/,
           "type": "coop"
@@ -110,10 +108,9 @@
         "report": {
           "body": {
             "disposition": "enforce",
-            "documentURI": `${location.href}`,
             "effectivePolicy": "same-origin-allow-popups",
-            "navigationURI": /uuid=EXECUTOR_UUID$/, // initial navigation URL
-            "type": "navigation-from-document"
+            "nextResponseURL": /uuid=EXECUTOR_UUID$/, // initial navigation URL
+            "type": "navigation-from-response"
           },
           "url": `${location.href}`,
           "type": "coop"

diff --git a/...licy/reporting/navigation-reporting/reporting-popup-same-origin-coep-report-to.https.html b/...licy/reporting/navigation-reporting/reporting-popup-same-origin-coep-report-to.https.html
@@ -38,10 +38,9 @@
         "report": {
           "body": {
             "disposition": "enforce",
-            "documentURI": `${location.href}`,
             "effectivePolicy": "same-origin-plus-coep",
-            "navigationURI": /uuid=EXECUTOR_UUID$/, // next destination url
-            "type": "navigation-from-document"
+            "nextResponseURL": /uuid=EXECUTOR_UUID$/, // next destination url
+            "type": "navigation-from-response"
           },
           "url": `${location.href}`,
           "type": "coop"
@@ -52,10 +51,10 @@
         "report": {
           "body": {
             "disposition": "enforce",
-            "documentURI": /uuid=EXECUTOR_UUID$/,
             "effectivePolicy": "same-origin",
-            "navigationURI": `${location.href}`, // previous document url
-            "type": "navigation-to-document"
+            "previousResponseURL": `${location.href}`, // previous document url
+            "referrer": `${location.origin}/`, // referrer (origin, as dictated by the referrer policy)
+            "type": "navigation-to-response"
           },
           "url": /uuid=EXECUTOR_UUID$/,
           "type": "coop"
@@ -80,10 +79,9 @@
         "report": {
           "body": {
             "disposition": "enforce",
-            "documentURI": `${location.href}`,
             "effectivePolicy": "same-origin-plus-coep",
-            "navigationURI": /uuid=EXECUTOR_UUID$/, // initial navigation url
-            "type": "navigation-from-document"
+            "nextResponseURL": /uuid=EXECUTOR_UUID$/, // initial navigation url
+            "type": "navigation-from-response"
           },
           "url": `${location.href}`,
           "type": "coop"
@@ -94,10 +92,10 @@
         "report": {
           "body": {
             "disposition": "enforce",
-            "documentURI": /uuid=EXECUTOR_UUID$/,
             "effectivePolicy": "same-origin-plus-coep",
-            "navigationURI": `${location.origin}/`, // referrer (origin, as dictated by the referrer policy)
-            "type": "navigation-to-document"
+            "previousResponseURL": ``,
+            "referrer": `${location.origin}/`, // referrer (origin, as dictated by the referrer policy)
+            "type": "navigation-to-response"
           },
           "url": /uuid=EXECUTOR_UUID$/,
           "type": "coop"
@@ -119,10 +117,9 @@
         "report": {
           "body": {
             "disposition": "enforce",
-            "documentURI": `${location.href}`,
             "effectivePolicy": "same-origin-plus-coep",
-            "navigationURI": /uuid=EXECUTOR_UUID$/, // next destination url
-            "type": "navigation-from-document"
+            "nextResponseURL": /uuid=EXECUTOR_UUID$/, // initial navigation url
+            "type": "navigation-from-response"
           },
           "url": `${location.href}`,
           "type": "coop"
@@ -144,10 +141,9 @@
         "report": {
           "body": {
             "disposition": "enforce",
-            "documentURI": `${location.href}`,
             "effectivePolicy": "same-origin-plus-coep",
-            "navigationURI": /uuid=EXECUTOR_UUID$/, // next destination url
-            "type": "navigation-from-document"
+            "nextResponseURL": /uuid=EXECUTOR_UUID$/, // initial navigation url
+            "type": "navigation-from-response"
           },
           "url": `${location.href}`,
           "type": "coop"
@@ -158,10 +154,10 @@
         "report": {
           "body": {
             "disposition": "reporting",
-            "documentURI": /uuid=EXECUTOR_UUID$/,
             "effectivePolicy": "same-origin-plus-coep",
-            "navigationURI": `${location.origin}/`, // referrer (origin, as dictated by the referrer policy)
-            "type": "navigation-to-document"
+            "previousResponseURL": ``,
+            "referrer": `${location.origin}/`, // referrer (origin, as dictated by the referrer policy)
+            "type": "navigation-to-response"
           },
           "url": /uuid=EXECUTOR_UUID$/,
           "type": "coop"