[cr149][Android] Remove getSetupInProgressHandle()
Chromium changes: https://chromium.googlesource.com/chromium/src/+/24cc34c57c5013cf7e27d1c9319cd506770b6477 commit 24cc34c57c5013cf7e27d1c9319cd506770b6477 Author: Victor Hugo Vianna Silva <victorvianna@google.com> Date: Tue Apr 21 05:19:50 2026 -0700 [sync/android] Remove getSetupInProgressHandle() There was only one remaining call. It was originally added for the "advanced sync setup" flow, to ensure sync only started after the user confirmed data type settings (thus preventing an irreversible upload of unwanted data). That flow doesn't exist anymore, since existing data is not uploaded when the user signs in. So the call is removed. Strictly speaking, there is a chance this changes behavior, as the call was done for both syncing and signed-in users. But if that's the case, it's a bug to be fixed separately. Notice this function doesn't exist on iOS anymore. Bug: 40066949 Change-Id: I6aacd39823972c5bc1b7181a9f1a52f57a62eba2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7771958 Commit-Queue: Victor Vianna <victorvianna@google.com> Reviewed-by: Mahmoud Rashad <mmrashad@google.com> Cr-Commit-Position: refs/heads/main@{#1618071}
This commit is contained in:
@@ -64,21 +64,6 @@ public class BraveManageSyncSettingsTest {
|
||||
void syncPasswordsOverridden(Boolean isChromeOS, Boolean handlerShouldBeOverridden) {
|
||||
setupMockSyncService();
|
||||
|
||||
// The next line triggers presubmit warning
|
||||
// Banned functions were used.
|
||||
// ...
|
||||
// It is safe to ignore this warning if you are just moving an existing
|
||||
// call, or if you want special handling for users in the legacy state.
|
||||
// Support the legacy state is the case for Brave Sync
|
||||
when(mSyncService.hasSyncConsent()).thenReturn(true);
|
||||
|
||||
when(mSyncService.getSetupInProgressHandle())
|
||||
.thenReturn(
|
||||
new SyncService.SyncSetupInProgressHandle() {
|
||||
@Override
|
||||
public void close() {}
|
||||
});
|
||||
|
||||
BraveManageSyncSettings.setIsRunningOnChromeOSForTesting(isChromeOS);
|
||||
BraveManageSyncSettings fragment = startManageSyncPreferences();
|
||||
|
||||
|
||||
Reference in New Issue
Block a user