commit ae515b50ae14b27c5c2260f49ac8fe40566a5655
author David Roger <droger@chromium.org> Thu May 02 18:15:46 2024
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> Thu May 02 18:15:46 2024
tree 017faf12bed1fdf3a0172ba0112c23e30ac6bae7
parent 2b4e65d4accf8118fea59c1a8a0188c76b002c9c

Reland "[Uno-d] Disable account storage for addresses when in auth error"

This reverts commit 04c57c4843b05dee8781365e0ba8269399d5f8bb.

Reason for revert: initial version had compilation errors, caused by
a conflict with https://crrev.com/c/5509512.

Original change's description:
> Revert "[Uno-d] Disable account storage for addresses when in auth error"
>
> This reverts commit 22dfcb968327f5d7c480c88a3b6ef3255210b502.
>
> Reason for revert: Breaks compilation, for example https://ci.chromium.org/ui/p/chromium/builders/ci/ios-simulator-noncq/44391/overview
>
> Original change's description:
> > [Uno-d] Disable account storage for addresses when in auth error
> >
> > Account storage was enabled while in pending state in the CL:
> > https://crrev.com/c/5432217
> > This CL conceptually reverts to the previous behavior, where
> > account storage is disabled when in signin pending state.
> >
> > The reason for the change is driven by product decisions, as there is
> > a requirement to have valid credentials in order for Chrome to keep
> > using the account storage.
> >
> > This may be explored again or revisited in the future.
> >
> > Bug: b/331601440
> > Change-Id: I551cd51864a2112f0ff0bfcb9c057a8f46a5dff7
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5499309
> > Reviewed-by: Florian Leimgruber <fleimgruber@google.com>
> > Reviewed-by: Ryan Sultanem <rsult@google.com>
> > Commit-Queue: David Roger <droger@chromium.org>
> > Cr-Commit-Position: refs/heads/main@{#1295514}
>
> Bug: b/331601440
> Change-Id: I57d66808d79dc0ef10accdbf61c63c9ce875974f
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5507218
> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
> Owners-Override: Nicolas MacBeth <nicolasmacbeth@google.com>
> Auto-Submit: Nicolas MacBeth <nicolasmacbeth@google.com>
> Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
> Cr-Commit-Position: refs/heads/main@{#1295516}

Bug: b/331601440
Change-Id: Id4ec537f59889b8d4c11dd4152ef68a37926d0f2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5511162
Reviewed-by: Florian Leimgruber <fleimgruber@google.com>
Commit-Queue: David Roger <droger@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1295615}