tree d94997a1eb1c09978ab53b5db26fc6d9a65a2f27
parent 4725e60f51401bfd03d1ba0021144847080c85b0
author Elly FJ <ellyjones@chromium.org> 1711015668 -0700
committer Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> 1711021559 -0700

Revert "[Private Network Access] Store redirect info when redirect mode is manual"

This reverts commit 1bb563e04e4e4d241fcb315777ebcf3238c0af14.

Reason for revert: required to unblock reverting r1259836 which introduced https://crbug.com/330364341

Original change's description:
> [Private Network Access] Store redirect info when redirect mode is manual
>
> When a redirected navigation request needs a preflight because of
> Private Network Access, the redirect_info is not properly stored in
> CorsURLLoader.  In FollowRedirect, the request URL and methods, etc will
> be overwritten with the stored redirect_info which is currently empty,
> causing the ERR_INVALID_URL in the associated bug.
>
> This CL stores the redirect_info properly for this case.
>
> The "missing cors headers" test cases are expected to fail because
> kPrivateNetworkAccessForNavigationsWarningOnly is by default enabled so
> requests actually succeeds
> .
> When kPrivateNetworkAccessForNavigations is disabled
> (virtual/pna-navigations-disabled), the preflight aren't sent so we
> expect failures too.
>
> Bug: 327017904,327022946
> Change-Id: I5631a2ba865c9a39dde7095888d7df24539194f2
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5331430
> Reviewed-by: Yifan Luo <lyf@chromium.org>
> Reviewed-by: Adam Rice <ricea@chromium.org>
> Commit-Queue: Jonathan Hao <phao@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#1266894}

Bug: 327017904,327022946
Change-Id: I5c7131baf484aa56369096dde166244d118309a5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5384287
Reviewed-by: Adam Rice <ricea@chromium.org>
Reviewed-by: Arthur Sonzogni <arthursonzogni@chromium.org>
Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1276115}