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

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

Issue 1138013008: [Sync] Make it impossible to get a reference to AndroidSyncSettings. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase and address nit. Created 5 years, 7 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 894f541a75f7720bcdad787743918a38d95c3e33..3798dcb2a3094a3bedb35c1212a9c76056629634 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
@@ -63,7 +63,6 @@ public class SyncController implements ApplicationStateListener,
private final Context mContext;
private final ChromeSigninController mChromeSigninController;
- private final AndroidSyncSettings mAndroidSyncSettings;
private final ProfileSyncService mProfileSyncService;
private final SyncNotificationController mSyncNotificationController;
@@ -77,8 +76,7 @@ public class SyncController implements ApplicationStateListener,
private SyncController(Context context) {
mContext = context;
mChromeSigninController = ChromeSigninController.get(mContext);
- mAndroidSyncSettings = AndroidSyncSettings.get(context);
- mAndroidSyncSettings.registerObserver(this);
+ AndroidSyncSettings.registerObserver(context, this);
mProfileSyncService = ProfileSyncService.get(mContext);
mProfileSyncService.addSyncStateChangedListener(this);
@@ -149,7 +147,7 @@ public class SyncController implements ApplicationStateListener,
* Updates sync to reflect the state of the Android sync settings.
*/
public void updateSyncStateFromAndroid() {
- if (mAndroidSyncSettings.isSyncEnabled()) {
+ if (AndroidSyncSettings.isSyncEnabled(mContext)) {
start();
} else {
stop();
@@ -163,11 +161,11 @@ public class SyncController implements ApplicationStateListener,
*/
public void start() {
ThreadUtils.assertOnUiThread();
- if (mAndroidSyncSettings.isMasterSyncEnabled()) {
+ if (AndroidSyncSettings.isMasterSyncEnabled(mContext)) {
Log.d(TAG, "Enabling sync");
InvalidationController.get(mContext).start();
mProfileSyncService.enableSync();
- mAndroidSyncSettings.enableChromeSync();
+ AndroidSyncSettings.enableChromeSync(mContext);
}
}
@@ -182,11 +180,11 @@ public class SyncController implements ApplicationStateListener,
Log.d(TAG, "Disabling sync");
InvalidationController.get(mContext).stop();
mProfileSyncService.disableSync();
- if (mAndroidSyncSettings.isMasterSyncEnabled()) {
+ if (AndroidSyncSettings.isMasterSyncEnabled(mContext)) {
// 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();
+ AndroidSyncSettings.disableChromeSync(mContext);
}
}
}
@@ -204,12 +202,12 @@ public class SyncController implements ApplicationStateListener,
// Make the Java state match the native state.
if (isSyncActive) {
InvalidationController.get(mContext).start();
- mAndroidSyncSettings.enableChromeSync();
+ AndroidSyncSettings.enableChromeSync(mContext);
} else {
InvalidationController.get(mContext).stop();
- if (mAndroidSyncSettings.isMasterSyncEnabled()) {
+ if (AndroidSyncSettings.isMasterSyncEnabled(mContext)) {
// See comment in stop().
- mAndroidSyncSettings.disableChromeSync();
+ AndroidSyncSettings.disableChromeSync(mContext);
}
}
}
@@ -263,7 +261,7 @@ public class SyncController implements ApplicationStateListener,
}
private void onFirstStart() {
- if (mAndroidSyncSettings.isSyncEnabled()) {
+ if (AndroidSyncSettings.isSyncEnabled(mContext)) {
InvalidationController controller = InvalidationController.get(mContext);
controller.refreshRegisteredTypes(mProfileSyncService.getPreferredDataTypes());
}

Powered by Google App Engine
This is Rietveld 408576698