Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3359)

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/sync/SyncController.java

Issue 879533004: Rewrite AndroidSyncSettings to be significantly simpler. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@android-sync
Patch Set: Add another deprecated method for transitioning. Created 5 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/android/java/src/org/chromium/chrome/browser/sync/SyncController.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/sync/SyncController.java b/chrome/android/java/src/org/chromium/chrome/browser/sync/SyncController.java
index 55664d776a2af9862d03f6ce4921aead9fc2236d..1b5791be935ea3c9ba1640e4db77b3e80948a164 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/sync/SyncController.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/sync/SyncController.java
@@ -120,10 +120,9 @@ public class SyncController implements ProfileSyncService.SyncStateChangedListen
ThreadUtils.assertOnUiThread();
if (mAndroidSyncSettings.isMasterSyncEnabled()) {
Log.d(TAG, "Enabling sync");
- Account account = mChromeSigninController.getSignedInUser();
InvalidationController.get(mContext).start();
mProfileSyncService.enableSync();
- mAndroidSyncSettings.enableChromeSync(account);
+ mAndroidSyncSettings.enableChromeSync();
}
}
@@ -136,14 +135,13 @@ public class SyncController implements ProfileSyncService.SyncStateChangedListen
ThreadUtils.assertOnUiThread();
if (mChromeSigninController.isSignedIn()) {
Log.d(TAG, "Disabling sync");
- Account account = mChromeSigninController.getSignedInUser();
InvalidationController.get(mContext).stop();
mProfileSyncService.disableSync();
if (mAndroidSyncSettings.isMasterSyncEnabled()) {
// Only disable Android's Chrome sync setting if we weren't disabled
// by the master sync setting. This way, when master sync is enabled
// they will both be on and sync will start again.
- mAndroidSyncSettings.disableChromeSync(account);
+ mAndroidSyncSettings.disableChromeSync();
}
}
}
@@ -157,19 +155,16 @@ public class SyncController implements ProfileSyncService.SyncStateChangedListen
@Override
public void syncStateChanged() {
ThreadUtils.assertOnUiThread();
- Account account = mChromeSigninController.getSignedInUser();
- // Don't do anything if there isn't an account.
- if (account == null) return;
boolean isSyncActive = !mProfileSyncService.isStartSuppressed();
// Make the Java state match the native state.
if (isSyncActive) {
InvalidationController.get(mContext).start();
- mAndroidSyncSettings.enableChromeSync(account);
+ mAndroidSyncSettings.enableChromeSync();
} else {
InvalidationController.get(mContext).stop();
if (mAndroidSyncSettings.isMasterSyncEnabled()) {
// See comment in stop().
- mAndroidSyncSettings.disableChromeSync(account);
+ mAndroidSyncSettings.disableChromeSync();
}
}
}

Powered by Google App Engine
This is Rietveld 408576698