Merge "Disable historical appops" (c39db508) · Commits · e / os / android_frameworks_base

services/core/java/com/android/server/appop/HistoricalRegistry.java

+5 −1

Original line number	Diff line number	Diff line
		@@ -150,9 +150,11 @@ final class HistoricalRegistry {

		/**
		* Whether history is enabled.
		*
		* <p>The feature is permanently disabled in Android Q
		*/
		@GuardedBy("mInMemoryLock")
		private int mMode = AppOpsManager.HISTORICAL_MODE_ENABLED_ACTIVE;
		private final int mMode = AppOpsManager.HISTORICAL_MODE_DISABLED;

		/**
		* This granularity has been chosen to allow clean delineation for intervals
		@@ -451,6 +453,7 @@ final class HistoricalRegistry {

		void setHistoryParameters(@HistoricalMode int mode,
		long baseSnapshotInterval, long intervalCompressionMultiplier) {
		/*
		synchronized (mOnDiskLock) {
		synchronized (mInMemoryLock) {
		// NOTE: We allow this call if persistence is not initialized as
		@@ -479,6 +482,7 @@ final class HistoricalRegistry {
		}
		}
		}
		*/
		}

		void offsetHistory(long offsetMillis) {

+3 −1

Original line number	Diff line number	Diff line
		@@ -63,6 +63,7 @@ import com.android.server.LocalServices;

		import org.junit.After;
		import org.junit.Before;
		import org.junit.Ignore;
		import org.junit.Test;
		import org.junit.runner.RunWith;
		import org.mockito.quality.Strictness;
		@@ -233,6 +234,7 @@ public class AppOpsServiceTest {
		assertContainsOp(loggedOps, OP_READ_SMS, mTestStartMillis, -1, MODE_ALLOWED);
		}

		@Ignore("Historical appops are disabled in Android Q")
		@Test
		public void testGetOpsForPackage() {
		mAppOpsService.setMode(OP_READ_SMS, mMyUid, sMyPackageName, MODE_ALLOWED);
		@@ -274,7 +276,7 @@ public class AppOpsServiceTest {
		assertThat(getLoggedOps()).isNull();
		}


		@Ignore("Historical appops are disabled in Android Q")
		@Test
		public void testPackageRemovedHistoricalOps() throws InterruptedException {
		mAppOpsService.setMode(OP_READ_SMS, mMyUid, sMyPackageName, MODE_ALLOWED);