commit | bf77eb5dfc9a9eb600dd275b59edcf1c293eda98 | [log] [tgz] |
---|---|---|
author | Victor Hugo Vianna Silva <victorvianna@google.com> | Mon Jul 08 17:20:48 2024 |
committer | Copybara-Service <copybara-worker@google.com> | Mon Jul 08 17:42:55 2024 |
tree | 1c09d85a62e4961c38967b386fd59804e4928ba5 | |
parent | 61892e7b5d2c42fc3f6f4576180062603eb4402e [diff] |
Merge TestSyncService::SetSignedIn*() methods into a single one Test-only change. Before this CL there were SetSignedInWithoutSyncFeature() and SetSignedInWithSyncFeatureOn(). Now there is a single SetSignedIn() which takes a signin::ConsentLevel parameter. This simplifies various instances of this pattern [1]. [1] https://source.chromium.org/chromium/chromium/src/+/main:components/unified_consent/unified_consent_service_unittest.cc;l=105;drc=2221b7963bfcca6429e7a82cab1e55f109955bb0 Change-Id: I7f81fc76bfdde32e172633a5ee788febcaeeddaa Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5682224 Reviewed-by: Colin Blundell <blundell@chromium.org> Code-Coverage: findit-for-me@appspot.gserviceaccount.com <findit-for-me@appspot.gserviceaccount.com> Auto-Submit: Victor Vianna <victorvianna@google.com> Commit-Queue: Victor Vianna <victorvianna@google.com> Reviewed-by: David Roger <droger@chromium.org> Reviewed-by: Mike Dougherty <michaeldo@chromium.org> Cr-Commit-Position: refs/heads/main@{#1324323} NOKEYCHECK=True GitOrigin-RevId: d6b7dd42aac2eab8ddac94926612d717482cad68