Bug 1723199 [wpt PR 29851] - Reporting: Allow document token to change, a=testonly
commit153a83aa3221dc0a30f2dfc44709c3a6d85e3abe
authorIan Clelland <iclelland@chromium.org>
Wed, 1 Sep 2021 14:03:48 +0000 (1 14:03 +0000)
committermoz-wptsync-bot <wptsync@mozilla.com>
Thu, 2 Sep 2021 10:30:10 +0000 (2 10:30 +0000)
treeb30232bab10d29eefeedf8e5830917560f345434
parent48abb31a0b79f434f9d6e38bdfbe4a13c770b046
Bug 1723199 [wpt PR 29851] - Reporting: Allow document token to change, a=testonly

Automatic update from web-platform-tests
Reporting: Allow document token to change

Since the RenderFrameHostImpl is reused for same-origin navigations,
the reporting source token must be able to change when a new document
is loaded into the frame. This is important for reporting isolation when
reports are still queued for the old document.

WPT is added to catch the situation where not changing this token would
cause previously queued reports to be sent to the new document's
configured endpoints.

This is part of a series of CLs implementing V1 reporting isolation.
A summary of the architectural changes introduced can be found at
https://docs.google.com/document/d/1RmEz17pGSUQITPoRKV4s3IBgbyHjv-HLZqtVYYZ4lMg/edit

CLs in this series:
https://crrev.com/c/2878175 Update NEL browser tests to require Report-To
https://crrev.com/c/2878376 Separate parsing from processing Reporting-Endpoints
https://crrev.com/c/2889833 Add reporting source to worker objects
https://crrev.com/c/2889975 Add reporting source to COOP/COEP reporters
https://crrev.com/c/3039579 Add source token to reports
https://crrev.com/c/3041880 Add source token to IPC calls
https://crrev.com/c/3042463 Isolate v1 reports from each other
https://crrev.com/c/3042465 (This CL) Allow document source token to change
https://crrev.com/c/3042521 Clean up when docs/workers are destroyed

Bug: 1062359
Change-Id: I908e49b7b773c57335f8c527843643c896082ce4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3042465
Commit-Queue: Ian Clelland <iclelland@chromium.org>
Reviewed-by: Matt Menke <mmenke@chromium.org>
Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Reviewed-by: Lily Chen <chlily@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Cr-Commit-Position: refs/heads/main@{#914491}

--

wpt-commits: b5c4ae1193387f99dc2bd7e723ed8561e56a137f
wpt-pr: 29851
testing/web-platform/tests/lint.ignore
testing/web-platform/tests/reporting/reporting-isolated-across-navigations.https.sub.html [new file with mode: 0644]
testing/web-platform/tests/reporting/resources/first-csp-report.https.sub.html [new file with mode: 0644]
testing/web-platform/tests/reporting/resources/first-csp-report.https.sub.html.sub.headers [new file with mode: 0644]
testing/web-platform/tests/reporting/resources/second-csp-report.https.sub.html [new file with mode: 0644]
testing/web-platform/tests/reporting/resources/second-csp-report.https.sub.html.sub.headers [new file with mode: 0644]