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

Unified Diff: sync/android/javatests/src/org/chromium/sync/AndroidSyncSettingsTest.java

Issue 1135113006: Revert of [Sync] Remove periodic sync for Chrome for Android (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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: sync/android/javatests/src/org/chromium/sync/AndroidSyncSettingsTest.java
diff --git a/sync/android/javatests/src/org/chromium/sync/AndroidSyncSettingsTest.java b/sync/android/javatests/src/org/chromium/sync/AndroidSyncSettingsTest.java
index c6b3ff8e518f987b669b625e8c00dbeee4856880..13c43f64d55b103d3c722a2e9da2ea9de9567090 100644
--- a/sync/android/javatests/src/org/chromium/sync/AndroidSyncSettingsTest.java
+++ b/sync/android/javatests/src/org/chromium/sync/AndroidSyncSettingsTest.java
@@ -6,7 +6,6 @@
import android.accounts.Account;
import android.content.Context;
-import android.os.Bundle;
import android.test.InstrumentationTestCase;
import android.test.suitebuilder.annotation.SmallTest;
@@ -30,7 +29,6 @@
private int mGetIsSyncableCalls;
private int mSetIsSyncableCalls;
private int mSetSyncAutomaticallyCalls;
- private int mRemovePeriodicSyncCalls;
@Override
public boolean getMasterSyncAutomatically() {
@@ -60,12 +58,6 @@
public void setSyncAutomatically(Account account, String authority, boolean sync) {
mSetSyncAutomaticallyCalls++;
super.setSyncAutomatically(account, authority, sync);
- }
-
- @Override
- public void removePeriodicSync(Account account, String authority, Bundle extras) {
- mRemovePeriodicSyncCalls++;
- super.removePeriodicSync(account, authority, extras);
}
}
@@ -146,22 +138,6 @@
@SmallTest
@Feature({"Sync"})
- public void testAccountInitialization() throws InterruptedException {
- // mAccount was set to be syncable and not have periodic syncs.
- assertEquals(1, mSyncContentResolverDelegate.mSetIsSyncableCalls);
- assertEquals(1, mSyncContentResolverDelegate.mRemovePeriodicSyncCalls);
- mAndroid.updateAccount(null);
- // mAccount was set to be not syncable.
- assertEquals(2, mSyncContentResolverDelegate.mSetIsSyncableCalls);
- assertEquals(1, mSyncContentResolverDelegate.mRemovePeriodicSyncCalls);
- mAndroid.updateAccount(mAlternateAccount);
- // mAlternateAccount was set to be syncable and not have periodic syncs.
- assertEquals(3, mSyncContentResolverDelegate.mSetIsSyncableCalls);
- assertEquals(2, mSyncContentResolverDelegate.mRemovePeriodicSyncCalls);
- }
-
- @SmallTest
- @Feature({"Sync"})
public void testToggleMasterSyncFromSettings() throws InterruptedException {
mSyncContentResolverDelegate.setMasterSyncAutomatically(true);
mSyncContentResolverDelegate.waitForLastNotificationCompleted();

Powered by Google App Engine
This is Rietveld 408576698