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

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

Issue 459513002: Massive refactor of the Android invalidation code. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: A couple GN fixes. Created 6 years, 3 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/ProfileSyncService.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/sync/ProfileSyncService.java b/chrome/android/java/src/org/chromium/chrome/browser/sync/ProfileSyncService.java
index d47f8e0547ccdbe4b665ac18dce91019180169f3..aac2631f5590859a2074606ca3a5cf489094d596 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/sync/ProfileSyncService.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/sync/ProfileSyncService.java
@@ -11,6 +11,8 @@ import org.chromium.base.CalledByNative;
import org.chromium.base.ThreadUtils;
import org.chromium.base.VisibleForTesting;
import org.chromium.chrome.browser.identity.UniqueIdentificationGenerator;
+import org.chromium.chrome.browser.invalidation.InvalidationServiceFactory;
+import org.chromium.chrome.browser.profiles.Profile;
import org.chromium.sync.internal_api.pub.SyncDecryptionPassphraseType;
import org.chromium.sync.internal_api.pub.base.ModelType;
@@ -35,8 +37,11 @@ import java.util.concurrent.CopyOnWriteArrayList;
*/
public class ProfileSyncService {
+ /**
+ * Listener for the underlying sync status.
+ */
public interface SyncStateChangedListener {
- // Invoked when the underlying sync status has changed.
+ // Invoked when the status has changed.
public void syncStateChanged();
}
@@ -138,25 +143,13 @@ public class ProfileSyncService {
syncSignIn(account);
}
- public void requestSyncFromNativeChrome(
- int objectSource, String objectId, long version, String payload) {
- ThreadUtils.assertOnUiThread();
- nativeNudgeSyncer(
- mNativeProfileSyncServiceAndroid, objectSource, objectId, version, payload);
- }
-
+ // TODO(maxbogue): Remove once downstream use is removed. See http://crbug.com/259559.
+ // Callers should use InvalidationService.requestSyncFromNativeChromeForAllTypes() instead.
+ @Deprecated
public void requestSyncFromNativeChromeForAllTypes() {
ThreadUtils.assertOnUiThread();
- nativeNudgeSyncerForAllTypes(mNativeProfileSyncServiceAndroid);
- }
-
- /**
- * Nudge the syncer to start a new sync cycle.
- */
- @VisibleForTesting
- public void requestSyncCycleForTest() {
- ThreadUtils.assertOnUiThread();
- requestSyncFromNativeChromeForAllTypes();
+ InvalidationServiceFactory.getForProfile(Profile.getLastUsedProfile())
+ .requestSyncFromNativeChromeForAllTypes();
}
public String querySyncStatus() {
@@ -545,10 +538,6 @@ public class ProfileSyncService {
}
// Native methods
- private native void nativeNudgeSyncer(
- long nativeProfileSyncServiceAndroid, int objectSource, String objectId, long version,
- String payload);
- private native void nativeNudgeSyncerForAllTypes(long nativeProfileSyncServiceAndroid);
private native long nativeInit();
private native void nativeEnableSync(long nativeProfileSyncServiceAndroid);
private native void nativeDisableSync(long nativeProfileSyncServiceAndroid);

Powered by Google App Engine
This is Rietveld 408576698