Commit 04d5df5a authored by Tanmoy Mollik's avatar Tanmoy Mollik Committed by Commit Bot

[Android] Hide SyncPromo after user gives sync consent

After going through the sync consent flow from SyncPromoPreference, this
preference should be hidden. Keeping the SyncPromoPreference when
SigninPromoController was not null was preventing that from happening.

Bug: 1126074
Change-Id: Ibad0838ec2594b1c6825561df4d6392359fe7c9c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2400418Reviewed-by: default avatarTanmoy Mollik <triploblastic@chromium.org>
Reviewed-by: default avatarAlice Wang <aliceywang@chromium.org>
Reviewed-by: default avatarMarc Treib <treib@chromium.org>
Commit-Queue: Tanmoy Mollik <triploblastic@chromium.org>
Cr-Commit-Position: refs/heads/master@{#807823}
parent 7755d9ab
...@@ -126,12 +126,6 @@ public class SyncPromoPreference extends Preference ...@@ -126,12 +126,6 @@ public class SyncPromoPreference extends Preference
return; return;
} }
if (mSigninPromoController != null) {
// Don't change the promo type if the new promo is already being shown.
setupPersonalizedSyncPromo();
return;
}
setupPromoHidden(); setupPromoHidden();
} }
......
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