[Android][Signin] Remove redundant check in ForcedSigninProcessor

This CL removes the redundant check FirstRunUtils.canAllowSync() in
ForcedSigninProcessor.signinAndEnableSync(), as this method is
always called with an account from AccountManagerFacade,
FirstRunUtils.canAllowSync() is always true in
ForcedSigninProcessor.signinAndEnableSync().

Bug: 1045525
Change-Id: I6e7632b2b27de917497d9982cd7e95a4e76527c0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2623289
Commit-Queue: Wenyu Fu <wenyufu@chromium.org>
Reviewed-by: Wenyu Fu <wenyufu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#842557}

chrome/android/java/src/org/chromium/chrome/browser/firstrun/ForcedSigninProcessor.java

@@ -62,8 +62,7 @@ public final class ForcedSigninProcessor {
      */
     private static void signinAndEnableSync(final Account childAccount) {
         final Profile profile = Profile.getLastUsedRegularProfile();
-        if (FirstRunUtils.canAllowSync()
-                && IdentityServicesProvider.get().getIdentityManager(profile).hasPrimaryAccount()) {
+        if (IdentityServicesProvider.get().getIdentityManager(profile).hasPrimaryAccount()) {
             // TODO(https://crbug.com/1044206): Remove this.
             ProfileSyncService.get().setFirstSetupComplete(SyncFirstSetupCompleteSource.BASIC_FLOW);
         }
@@ -71,7 +70,7 @@ public final class ForcedSigninProcessor {
                 IdentityServicesProvider.get().getSigninManager(profile);
         // By definition we have finished all the checks for first run.
         signinManager.onFirstRunCheckDone();
-        if (FirstRunUtils.canAllowSync() && signinManager.isSignInAllowed()) {
+        if (signinManager.isSignInAllowed()) {
             signinManager.signinAndEnableSync(SigninAccessPoint.FORCED_SIGNIN, childAccount,
                     new SigninManager.SignInCallback() {
                         @Override