Merge "Fix for syncs being dropped when appIdle is on" (768464c6) · Commits · e / os / android_frameworks_base

services/core/java/com/android/server/content/SyncManager.java

+22 −19

Original line number	Diff line number	Diff line
		@@ -2682,24 +2682,6 @@ public class SyncManager {
		}
		continue;
		}
		if (!isOperationValidLocked(op)) {
		operationIterator.remove();
		mSyncStorageEngine.deleteFromPending(op.pendingOperation);
		continue;
		}
		// If the next run time is in the future, even given the flexible scheduling,
		// return the time.
		if (op.effectiveRunTime - op.flexTime > now) {
		if (nextReadyToRunTime > op.effectiveRunTime) {
		nextReadyToRunTime = op.effectiveRunTime;
		}
		if (isLoggable) {
		Log.v(TAG, " Not running sync operation: Sync too far in future."
		+ "effective: " + op.effectiveRunTime + " flex: " + op.flexTime
		+ " now: " + now);
		}
		continue;
		}
		String packageName = getPackageName(op.target);
		ApplicationInfo ai = null;
		if (packageName != null) {
		@@ -2708,6 +2690,9 @@ public class SyncManager {
		PackageManager.GET_UNINSTALLED_PACKAGES
		\| PackageManager.GET_DISABLED_COMPONENTS);
		} catch (NameNotFoundException e) {
		operationIterator.remove();
		mSyncStorageEngine.deleteFromPending(op.pendingOperation);
		continue;
		}
		}
		// If app is considered idle, then skip for now and backoff
		@@ -2722,6 +2707,24 @@ public class SyncManager {
		} else {
		op.appIdle = false;
		}
		if (!isOperationValidLocked(op)) {
		operationIterator.remove();
		mSyncStorageEngine.deleteFromPending(op.pendingOperation);
		continue;
		}
		// If the next run time is in the future, even given the flexible scheduling,
		// return the time.
		if (op.effectiveRunTime - op.flexTime > now) {
		if (nextReadyToRunTime > op.effectiveRunTime) {
		nextReadyToRunTime = op.effectiveRunTime;
		}
		if (isLoggable) {
		Log.v(TAG, " Not running sync operation: Sync too far in future."
		+ "effective: " + op.effectiveRunTime + " flex: " + op.flexTime
		+ " now: " + now);
		}
		continue;
		}
		// Add this sync to be run.
		operations.add(op);
		}