Merge "Disable historical appops" into qt-dev am: 16128cc7a2

am: 4f7f78cce5

Change-Id: I0ebc8196a2b8cfbd45cb77b2e3affc2b5e649888
This commit is contained in:
Philip P. Moltmann
2019-06-28 21:06:12 -07:00
committed by android-build-merger
2 changed files with 7 additions and 1 deletions

View File

@@ -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) {

View File

@@ -46,6 +46,7 @@ import androidx.test.runner.AndroidJUnit4;
import org.junit.AfterClass;
import org.junit.Before;
import org.junit.BeforeClass;
import org.junit.Ignore;
import org.junit.Test;
import org.junit.runner.RunWith;
@@ -242,6 +243,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, mMyPackageName, MODE_ALLOWED);