Commit b4178421 authored by Boris Sazonov's avatar Boris Sazonov Committed by Commit Bot

[Signin][Android] Disable sign-in promos if MobileIdentityConsistency is on

Adds to SigninManager.isSigninSupported a check for
MobileIdentityConsistency feature to hide sign-in UI surfaces if the
feature is enabled.

Bug: 968140
Change-Id: I39156d4baf852d65117380bf98a983830d61dd6f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1645308
Auto-Submit: Boris Sazonov <bsazonov@chromium.org>
Reviewed-by: default avatarMihai Sardarescu <msarda@chromium.org>
Commit-Queue: Boris Sazonov <bsazonov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#666349}
parent 49100e31
...@@ -27,6 +27,7 @@ import org.chromium.base.annotations.NativeMethods; ...@@ -27,6 +27,7 @@ import org.chromium.base.annotations.NativeMethods;
import org.chromium.base.metrics.RecordHistogram; import org.chromium.base.metrics.RecordHistogram;
import org.chromium.base.metrics.RecordUserAction; import org.chromium.base.metrics.RecordUserAction;
import org.chromium.base.task.PostTask; import org.chromium.base.task.PostTask;
import org.chromium.chrome.browser.ChromeFeatureList;
import org.chromium.components.signin.AccountIdProvider; import org.chromium.components.signin.AccountIdProvider;
import org.chromium.components.signin.AccountManagerFacade; import org.chromium.components.signin.AccountManagerFacade;
import org.chromium.components.signin.AccountTrackerService; import org.chromium.components.signin.AccountTrackerService;
...@@ -301,7 +302,8 @@ public class SigninManager implements AccountTrackerService.OnSystemAccountsSeed ...@@ -301,7 +302,8 @@ public class SigninManager implements AccountTrackerService.OnSystemAccountsSeed
*/ */
public boolean isSigninSupported() { public boolean isSigninSupported() {
return !ApiCompatibilityUtils.isDemoUser(mContext) return !ApiCompatibilityUtils.isDemoUser(mContext)
&& mDelegate.isGooglePlayServicesPresent(mContext); && mDelegate.isGooglePlayServicesPresent(mContext)
&& !ChromeFeatureList.isEnabled(ChromeFeatureList.MOBILE_IDENTITY_CONSISTENCY);
} }
/** /**
......
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