commit e9651ef40487b14ff621776a8bfd8c6353fac286
author Joel Hockey <joelhockey@chromium.org> Tue Mar 25 05:25:57 2025
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> Tue Mar 25 05:25:57 2025
tree 84b8e6d67de168d89a227422c821af299842ce70
parent 6639138e7a68b37a61966ce514ca55610e034380

Revert "`ShouldSetLoadWithStorageAccess` to calculate storage status"

This reverts commit b9e8c5182ce229f7416ec26d457e5706dbc71161.

Reason for revert: failing tests on multiple bots
net_unittests StorageAccessStatusTest
https://ci.chromium.org/ui/p/chrome/builders/ci/chromeos-betty-chrome-tests/15367/overview

Original change's description:
> `ShouldSetLoadWithStorageAccess` to calculate storage status
>
> `ShouldSetLoadWithStorageAccess` relied on
> `URLRequest::storage_access_status()` calculated earlier. This CL
> introduces a new `URLRequest::StorageAccessStatus` state `Unset` to
> determine if the state was not calculated earlier and let
> `ShouldSetLoadWithStorageAccess` calculate it in this case. This will
> Allow us to not calculate storage access status before it's necessary.
>
> Bug: 382291442
> Low-Coverage-Reason: COVERAGE_UNDERREPORTED
> Change-Id: I98ba67048646cb4abd92eb0a77c92bf1a0585727
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6361478
> Reviewed-by: Maks Orlovich <morlovich@chromium.org>
> Commit-Queue: Sandor «Alex» Major <sandormajor@chromium.org>
> Reviewed-by: Chris Fredrickson <cfredric@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#1437118}

Bug: 382291442
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Change-Id: I0606cfc91c7862a438b0467740337861d869e95e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6390235
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Auto-Submit: Joel Hockey <joelhockey@chromium.org>
Owners-Override: Joel Hockey <joelhockey@chromium.org>
Commit-Queue: Joel Hockey <joelhockey@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1437311}