Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit f816bb58 authored by Philip P. Moltmann's avatar Philip P. Moltmann
Browse files

Disable historical appops

Fixes: 133419419
Test: atest CtsAppOpsTestCases
Change-Id: I1790581b5ac6608c59d731d9d28110f125186a39
Merged-In: I6debd3cbb6768b256eb558a81ba13efea375769b
parent 949a22d7
Loading
Loading
Loading
Loading
+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) {
+2 −0
Original line number Diff line number Diff line
@@ -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);