[Android][Test] Remove SyncTestRule#signinAndEnableSyncInternal()

This CL removes the unused method
SyncTestRule#signinAndEnableSyncInternal() as all its usages are
migrated.

Bug: 1126814
Change-Id: I98d876133dc1f2b5068a2f9f451ff963fad4683e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2410123Reviewed-by: Marc Treib <treib@chromium.org>
Commit-Queue: Alice Wang <aliceywang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#807432}

chrome/android/javatests/src/org/chromium/chrome/browser/sync/SyncTestRule.java

@@ -20,7 +20,6 @@ import org.junit.runner.Description;
 import org.junit.runners.model.Statement;
 
 import org.chromium.base.Promise;
-import org.chromium.chrome.browser.SyncFirstSetupCompleteSource;
 import org.chromium.chrome.browser.app.ChromeActivity;
 import org.chromium.chrome.browser.autofill.PersonalDataManager;
 import org.chromium.chrome.browser.autofill.PersonalDataManager.CreditCard;
@@ -28,15 +27,12 @@ import org.chromium.chrome.browser.identity.UniqueIdentificationGenerator;
 import org.chromium.chrome.browser.identity.UniqueIdentificationGeneratorFactory;
 import org.chromium.chrome.browser.identity.UuidBasedUniqueIdentificationGenerator;
 import org.chromium.chrome.browser.profiles.Profile;
-import org.chromium.chrome.browser.signin.IdentityServicesProvider;
-import org.chromium.chrome.browser.signin.SigninManager;
 import org.chromium.chrome.browser.signin.UnifiedConsentServiceBridge;
 import org.chromium.chrome.test.ChromeActivityTestRule;
 import org.chromium.chrome.test.util.browser.signin.AccountManagerTestRule;
 import org.chromium.chrome.test.util.browser.signin.SigninTestUtil;
 import org.chromium.chrome.test.util.browser.sync.SyncTestUtil;
 import org.chromium.components.signin.base.CoreAccountInfo;
-import org.chromium.components.signin.metrics.SigninAccessPoint;
 import org.chromium.components.sync.ModelType;
 import org.chromium.components.sync.protocol.AutofillWalletSpecifics;
 import org.chromium.components.sync.protocol.EntitySpecifics;
@@ -418,40 +414,6 @@ public class SyncTestRule extends ChromeActivityTestRule<ChromeActivity> {
         return null;
     }
 
-    /**
-     * TODO(https://crbug.com/1126814): Remove this method once all its usages are migrated.
-     */
-    @Deprecated
-    private void signinAndEnableSyncInternal(final Account account, boolean setFirstSetupComplete) {
-        TestThreadUtils.runOnUiThreadBlocking(() -> {
-            IdentityServicesProvider.get()
-                    .getSigninManager(Profile.getLastUsedRegularProfile())
-                    .signinAndEnableSync(
-                            SigninAccessPoint.UNKNOWN, account, new SigninManager.SignInCallback() {
-                                @Override
-                                public void onSignInComplete() {
-                                    if (setFirstSetupComplete) {
-                                        mProfileSyncService.setFirstSetupComplete(
-                                                SyncFirstSetupCompleteSource.BASIC_FLOW);
-                                    }
-                                }
-
-                                @Override
-                                public void onSignInAborted() {
-                                    Assert.fail("Sign-in was aborted");
-                                }
-                            });
-        });
-        enableUKM();
-        if (setFirstSetupComplete) {
-            SyncTestUtil.waitForSyncActive();
-            SyncTestUtil.triggerSyncAndWaitForCompletion();
-        } else {
-            SyncTestUtil.waitForSyncTransportActive();
-        }
-        Assert.assertEquals(account, mAccountManagerTestRule.getCurrentSignedInAccount());
-    }
-
     private static void enableUKM() {
         TestThreadUtils.runOnUiThreadBlocking(() -> {
             // Outside of tests, URL-keyed anonymized data collection is enabled by sign-in UI.