tree 5fbe7cad79a9d9c86327e20bc5e6f5f39cad9be9
parent 79eaf42c932d8f08f1784562ba4d975e4abb272d
author Mike West <mkwst@chromium.org> 1663079854 -0700
committer Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> 1663080729 -0700

Revert two PNA CLs that seem to be causing crashes.

This reverts commit 4f617a2060db8828b6caa16cdd57be12d4ec3af8
("[Private Network Access] Support service worker script update") and
commit ee580dabdb282695447ad86db14c7a0caaecd79d ("Fix success test
cases in service-worker-update.https.window.js").

The former is suspected to be responsible for crashes in https://crbug.com/1362131. The latter was necessary to cleanly revert
the suspected patch.

Original change's description:
> [Private Network Access] Support service worker script update
>
> This CL passes a proper client security state based on the
> ServiceWorkerRegisterJob's policy container polices to GetLoaderFactoryForUpdateCheck which eventually gets to [1] where we
> would use it to create a loader factory just like we did for new script
> fetches.  It also initialises policy container policies for the
> ServiceWorkerRegisterJob constructor used by updates.
>
> Now the PNA check will block the public->private/local fetches in wpt
> tests.
>
> [1] https://source.chromium.org/chromium/chromium/src/+/main:content/browser/service_worker/service_worker_context_wrapper.cc;l=1651;drc=13dced01cce80364299ad7663479ad65069db73d
> [2] https://source.chromium.org/chromium/chromium/src/+/main:content/browser/service_worker/service_worker_register_job.h;l=64-65;drc=13dced01cce80364299ad7663479ad65069db73d?q=register_job.h&ss=chromium%2Fchromium%2Fsrc
>
> Bug: 1239551
> Change-Id: I1871153d83ae97e4aba46437e6708d748865052e
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3865859
> Auto-Submit: Jonathan Hao <phao@chromium.org>
> Commit-Queue: Camille Lamy <clamy@chromium.org>
> Reviewed-by: Camille Lamy <clamy@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#1044019}

Bug: 1239551, 1362131
Change-Id: I0e57be7151fa035c191f635c586fac44d7500cb0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3890988
Commit-Queue: Mike West <mkwst@chromium.org>
Owners-Override: Mike West <mkwst@chromium.org>
Reviewed-by: Arthur Hemery <ahemery@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1046349}