Commit ce5bdb6f authored by Kushagra Sinha's avatar Kushagra Sinha Committed by Commit Bot

Do not require Sync consent in EduAccountLoginHandler

EduAccountLoginHandler currently uses |signin::ConsentLevel::kSync| for
fetching the account to be used for initiating the parental reauth in
EDU co-existence flow. Since this account is guaranteed and must be the
Device Account, we should use that regardless of Sync consent status.

Fix this by supplying |signin::ConsentLevel::kNotRequired| while
fetching the access token for the Device Account in
EduAccountLoginHandler.

Bug: 1123844
Change-Id: I002f20b754dc63378a4b1b086efbf8cd6b7b3485
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2484698Reviewed-by: default avatarDavid Roger <droger@chromium.org>
Commit-Queue: Kush Sinha <sinhak@chromium.org>
Cr-Commit-Position: refs/heads/master@{#818433}
parent 9f0606f9
...@@ -261,7 +261,8 @@ void EduAccountLoginHandler::FetchAccessToken( ...@@ -261,7 +261,8 @@ void EduAccountLoginHandler::FetchAccessToken(
&EduAccountLoginHandler::CreateReAuthProofTokenForParent, &EduAccountLoginHandler::CreateReAuthProofTokenForParent,
base::Unretained(this), std::move(obfuscated_gaia_id), base::Unretained(this), std::move(obfuscated_gaia_id),
std::move(password)), std::move(password)),
signin::PrimaryAccountAccessTokenFetcher::Mode::kImmediate); signin::PrimaryAccountAccessTokenFetcher::Mode::kImmediate,
signin::ConsentLevel::kNotRequired);
} }
void EduAccountLoginHandler::FetchReAuthProofTokenForParent( void EduAccountLoginHandler::FetchReAuthProofTokenForParent(
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment