Commit 55d4f144 authored by Thomas Tangl's avatar Thomas Tangl Committed by Commit Bot

[unified-consent] Only record sync-paused suppress reason when needed

The sync-paused suppress reason should only be recorded when the
consent bump needs to be shown.

Bug: 876235
Change-Id: I65667f941bc7e15a49681e0b61d08ab070fc71da
Reviewed-on: https://chromium-review.googlesource.com/1183237Reviewed-by: default avatarElly Fong-Jones <ellyjones@chromium.org>
Commit-Queue: Thomas Tangl <tangltom@chromium.org>
Cr-Commit-Position: refs/heads/master@{#584745}
parent 66ce35f8
......@@ -79,8 +79,10 @@ class ConsentBumpActivator : public BrowserListObserver,
signin_manager->GetAuthenticatedAccountId())) {
unified_consent::UnifiedConsentService* consent_service =
UnifiedConsentServiceFactory::GetForProfile(profile_);
consent_service->RecordConsentBumpSuppressReason(
unified_consent::ConsentBumpSuppressReason::kSyncPaused);
if (consent_service->ShouldShowConsentBump()) {
consent_service->RecordConsentBumpSuppressReason(
unified_consent::ConsentBumpSuppressReason::kSyncPaused);
}
return;
}
......
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