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

Commit 6d5fbaeb authored by Harshit Mahajan's avatar Harshit Mahajan
Browse files

Revert^2 "[CrashRecovery] Refactor SystemProperties as api" 

This reverts commit ef811dd8.

Reason for revert: Fixing this one
Test: atest PackageWatchdogTest:com.android.server.PackageWatchdogTest

Change-Id: I2d418152eae87aefa9c2deb45fc5a95a99afdd95
parent be9ab02c

packages/CrashRecovery/services/java/com/android/server/PackageWatchdog.java

+15 −22
Original line number Original line Diff line number Diff line
@@ -33,12 +33,12 @@ import android.os.Looper; 
import android.os.Process;

import android.os.Process;

import android.os.SystemProperties;

import android.os.SystemProperties;

import android.provider.DeviceConfig;

import android.provider.DeviceConfig;

import android.sysprop.CrashRecoveryProperties;

import android.text.TextUtils;

import android.text.TextUtils;

import android.util.ArrayMap;

import android.util.ArrayMap;

import android.util.ArraySet;

import android.util.ArraySet;

import android.util.AtomicFile;

import android.util.AtomicFile;

import android.util.LongArrayQueue;

import android.util.LongArrayQueue;

import android.util.MathUtils;

import android.util.Slog;

import android.util.Slog;

import android.util.Xml;

import android.util.Xml;
@@ -130,16 +130,6 @@ public class PackageWatchdog { 
    static final int DEFAULT_BOOT_LOOP_TRIGGER_COUNT = 5;

    static final int DEFAULT_BOOT_LOOP_TRIGGER_COUNT = 5;

    static final long DEFAULT_BOOT_LOOP_TRIGGER_WINDOW_MS = TimeUnit.MINUTES.toMillis(10);

    static final long DEFAULT_BOOT_LOOP_TRIGGER_WINDOW_MS = TimeUnit.MINUTES.toMillis(10);





    // These properties track individual system server boot events, and are reset once the boot

    // threshold is met, or the boot loop trigger window is exceeded between boot events.

    private static final String PROP_RESCUE_BOOT_COUNT = "sys.rescue_boot_count";

    private static final String PROP_RESCUE_BOOT_START = "sys.rescue_boot_start";



    // These properties track multiple calls made to observers tracking boot loops. They are reset

    // when the de-escalation window is exceeded between boot events.

    private static final String PROP_BOOT_MITIGATION_WINDOW_START = "sys.boot_mitigation_start";

    private static final String PROP_BOOT_MITIGATION_COUNT = "sys.boot_mitigation_count";



    private long mNumberOfNativeCrashPollsRemaining;

    private long mNumberOfNativeCrashPollsRemaining;





    private static final int DB_VERSION = 1;

    private static final int DB_VERSION = 1;
@@ -1703,41 +1693,44 @@ public class PackageWatchdog { 
        }

        }





        private int getCount() {

        private int getCount() {

            return SystemProperties.getInt(PROP_RESCUE_BOOT_COUNT, 0);

            return CrashRecoveryProperties.rescueBootCount().orElse(0);

        }

        }





        private void setCount(int count) {

        private void setCount(int count) {

            SystemProperties.set(PROP_RESCUE_BOOT_COUNT, Integer.toString(count));

            CrashRecoveryProperties.rescueBootCount(count);

        }

        }





        public long getStart() {

        public long getStart() {

            return SystemProperties.getLong(PROP_RESCUE_BOOT_START, 0);

            return CrashRecoveryProperties.rescueBootStart().orElse(0L);

        }

        }





        public int getMitigationCount() {

        public int getMitigationCount() {

            return SystemProperties.getInt(PROP_BOOT_MITIGATION_COUNT, 0);

            return CrashRecoveryProperties.bootMitigationCount().orElse(0);

        }

        }





        public void setStart(long start) {

        public void setStart(long start) {

            setPropertyStart(PROP_RESCUE_BOOT_START, start);

            CrashRecoveryProperties.rescueBootStart(getStartTime(start));

        }

        }





        public void setMitigationStart(long start) {

        public void setMitigationStart(long start) {

            setPropertyStart(PROP_BOOT_MITIGATION_WINDOW_START, start);

            CrashRecoveryProperties.bootMitigationStart(getStartTime(start));

        }

        }





        public long getMitigationStart() {

        public long getMitigationStart() {

            return SystemProperties.getLong(PROP_BOOT_MITIGATION_WINDOW_START, 0);

            return CrashRecoveryProperties.bootMitigationStart().orElse(0L);

        }

        }





        public void setMitigationCount(int count) {

        public void setMitigationCount(int count) {

            SystemProperties.set(PROP_BOOT_MITIGATION_COUNT, Integer.toString(count));

            CrashRecoveryProperties.bootMitigationCount(count);

        }



        private static long constrain(long amount, long low, long high) {

            return amount < low ? low : (amount > high ? high : amount);

        }

        }





        public void setPropertyStart(String property, long start) {

        public long getStartTime(long start) {

            final long now = mSystemClock.uptimeMillis();

            final long now = mSystemClock.uptimeMillis();

            final long newStart = MathUtils.constrain(start, 0, now);

            return constrain(start, 0, now);

            SystemProperties.set(property, Long.toString(newStart));

        }

        }





        public void saveMitigationCountToMetadata() {

        public void saveMitigationCountToMetadata() {

packages/CrashRecovery/services/java/com/android/server/RescueParty.java

+10 −17
Original line number Original line Diff line number Diff line
@@ -37,6 +37,7 @@ import android.os.SystemProperties; 
import android.os.UserHandle;

import android.os.UserHandle;

import android.provider.DeviceConfig;

import android.provider.DeviceConfig;

import android.provider.Settings;

import android.provider.Settings;

import android.sysprop.CrashRecoveryProperties;

import android.text.TextUtils;

import android.text.TextUtils;

import android.util.ArraySet;

import android.util.ArraySet;

import android.util.ExceptionUtils;

import android.util.ExceptionUtils;
@@ -74,12 +75,6 @@ import java.util.concurrent.TimeUnit; 
 * @hide

 * @hide

 */

 */

public class RescueParty {

public class RescueParty {

    @VisibleForTesting

    static final String PROP_ENABLE_RESCUE = "persist.sys.enable_rescue";

    static final String PROP_ATTEMPTING_FACTORY_RESET = "sys.attempting_factory_reset";

    static final String PROP_ATTEMPTING_REBOOT = "sys.attempting_reboot";

    static final String PROP_MAX_RESCUE_LEVEL_ATTEMPTED = "sys.max_rescue_level_attempted";

    static final String PROP_LAST_FACTORY_RESET_TIME_MS = "persist.sys.last_factory_reset";

    @VisibleForTesting

    @VisibleForTesting

    static final int LEVEL_NONE = 0;

    static final int LEVEL_NONE = 0;

    @VisibleForTesting

    @VisibleForTesting
@@ -93,8 +88,6 @@ public class RescueParty { 
    @VisibleForTesting

    @VisibleForTesting

    static final int LEVEL_FACTORY_RESET = 5;

    static final int LEVEL_FACTORY_RESET = 5;

    @VisibleForTesting

    @VisibleForTesting

    static final String PROP_RESCUE_BOOT_COUNT = "sys.rescue_boot_count";

    @VisibleForTesting

    static final String TAG = "RescueParty";

    static final String TAG = "RescueParty";

    @VisibleForTesting

    @VisibleForTesting

    static final long DEFAULT_OBSERVING_DURATION_MS = TimeUnit.DAYS.toMillis(2);

    static final long DEFAULT_OBSERVING_DURATION_MS = TimeUnit.DAYS.toMillis(2);
@@ -131,7 +124,7 @@ public class RescueParty { 




    private static boolean isDisabled() {

    private static boolean isDisabled() {

        // Check if we're explicitly enabled for testing

        // Check if we're explicitly enabled for testing

        if (SystemProperties.getBoolean(PROP_ENABLE_RESCUE, false)) {

        if (CrashRecoveryProperties.enableRescueParty().orElse(false)) {

            return false;

            return false;

        }

        }
@@ -177,11 +170,11 @@ public class RescueParty { 
    }

    }





    static boolean isFactoryResetPropertySet() {

    static boolean isFactoryResetPropertySet() {

        return SystemProperties.getBoolean(PROP_ATTEMPTING_FACTORY_RESET, false);

        return CrashRecoveryProperties.attemptingFactoryReset().orElse(false);

    }

    }





    static boolean isRebootPropertySet() {

    static boolean isRebootPropertySet() {

        return SystemProperties.getBoolean(PROP_ATTEMPTING_REBOOT, false);

        return CrashRecoveryProperties.attemptingReboot().orElse(false);

    }

    }





    /**

    /**
@@ -439,7 +432,7 @@ public class RescueParty { 
            case LEVEL_WARM_REBOOT:

            case LEVEL_WARM_REBOOT:

                // Request the reboot from a separate thread to avoid deadlock on PackageWatchdog

                // Request the reboot from a separate thread to avoid deadlock on PackageWatchdog

                // when device shutting down.

                // when device shutting down.

                SystemProperties.set(PROP_ATTEMPTING_REBOOT, "true");

                CrashRecoveryProperties.attemptingReboot(true);

                runnable = () -> {

                runnable = () -> {

                    try {

                    try {

                        PowerManager pm = context.getSystemService(PowerManager.class);

                        PowerManager pm = context.getSystemService(PowerManager.class);
@@ -461,9 +454,9 @@ public class RescueParty { 
                if (isRebootPropertySet()) {

                if (isRebootPropertySet()) {

                    break;

                    break;

                }

                }

                SystemProperties.set(PROP_ATTEMPTING_FACTORY_RESET, "true");

                CrashRecoveryProperties.attemptingFactoryReset(true);

                long now = System.currentTimeMillis();

                long now = System.currentTimeMillis();

                SystemProperties.set(PROP_LAST_FACTORY_RESET_TIME_MS, Long.toString(now));

                CrashRecoveryProperties.lastFactoryResetTimeMs(now);

                runnable = new Runnable() {

                runnable = new Runnable() {

                    @Override

                    @Override

                    public void run() {

                    public void run() {
@@ -513,10 +506,10 @@ public class RescueParty { 
    private static void resetAllSettingsIfNecessary(Context context, int mode,

    private static void resetAllSettingsIfNecessary(Context context, int mode,

            int level) throws Exception {

            int level) throws Exception {

        // No need to reset Settings again if they are already reset in the current level once.

        // No need to reset Settings again if they are already reset in the current level once.

        if (SystemProperties.getInt(PROP_MAX_RESCUE_LEVEL_ATTEMPTED, LEVEL_NONE) >= level) {

        if (CrashRecoveryProperties.maxRescueLevelAttempted().orElse(LEVEL_NONE) >= level) {

            return;

            return;

        }

        }

        SystemProperties.set(PROP_MAX_RESCUE_LEVEL_ATTEMPTED, Integer.toString(level));

        CrashRecoveryProperties.maxRescueLevelAttempted(level);

        // Try our best to reset all settings possible, and once finished

        // Try our best to reset all settings possible, and once finished

        // rethrow any exception that we encountered

        // rethrow any exception that we encountered

        Exception res = null;

        Exception res = null;
@@ -731,7 +724,7 @@ public class RescueParty { 
         * Will return {@code false} if a factory reset was already offered recently.

         * Will return {@code false} if a factory reset was already offered recently.

         */

         */

        private boolean shouldThrottleReboot() {

        private boolean shouldThrottleReboot() {

            Long lastResetTime = SystemProperties.getLong(PROP_LAST_FACTORY_RESET_TIME_MS, 0);

            Long lastResetTime = CrashRecoveryProperties.lastFactoryResetTimeMs().orElse(0L);

            long now = System.currentTimeMillis();

            long now = System.currentTimeMillis();

            long throttleDurationMin = SystemProperties.getLong(PROP_THROTTLE_DURATION_MIN_FLAG,

            long throttleDurationMin = SystemProperties.getLong(PROP_THROTTLE_DURATION_MIN_FLAG,

                    DEFAULT_FACTORY_RESET_THROTTLE_DURATION_MIN);

                    DEFAULT_FACTORY_RESET_THROTTLE_DURATION_MIN);

packages/CrashRecovery/services/java/com/android/server/rollback/RollbackPackageHealthObserver.java

+2 −2
Original line number Original line Diff line number Diff line
@@ -34,6 +34,7 @@ import android.os.Handler; 
import android.os.HandlerThread;

import android.os.HandlerThread;

import android.os.PowerManager;

import android.os.PowerManager;

import android.os.SystemProperties;

import android.os.SystemProperties;

import android.sysprop.CrashRecoveryProperties;

import android.util.ArraySet;

import android.util.ArraySet;

import android.util.Slog;

import android.util.Slog;

import android.util.SparseArray;

import android.util.SparseArray;
@@ -70,7 +71,6 @@ import java.util.function.Consumer; 
final class RollbackPackageHealthObserver implements PackageHealthObserver {

final class RollbackPackageHealthObserver implements PackageHealthObserver {

    private static final String TAG = "RollbackPackageHealthObserver";

    private static final String TAG = "RollbackPackageHealthObserver";

    private static final String NAME = "rollback-observer";

    private static final String NAME = "rollback-observer";

    private static final String PROP_ATTEMPTING_REBOOT = "sys.attempting_reboot";

    private static final int PERSISTENT_MASK = ApplicationInfo.FLAG_PERSISTENT

    private static final int PERSISTENT_MASK = ApplicationInfo.FLAG_PERSISTENT

            | ApplicationInfo.FLAG_SYSTEM;

            | ApplicationInfo.FLAG_SYSTEM;
@@ -450,7 +450,7 @@ final class RollbackPackageHealthObserver implements PackageHealthObserver { 
                markStagedSessionHandled(rollback.getRollbackId());

                markStagedSessionHandled(rollback.getRollbackId());

                // Wait for all pending staged sessions to get handled before rebooting.

                // Wait for all pending staged sessions to get handled before rebooting.

                if (isPendingStagedSessionsEmpty()) {

                if (isPendingStagedSessionsEmpty()) {

                    SystemProperties.set(PROP_ATTEMPTING_REBOOT, "true");

                    CrashRecoveryProperties.attemptingReboot(true);

                    mContext.getSystemService(PowerManager.class).reboot("Rollback staged install");

                    mContext.getSystemService(PowerManager.class).reboot("Rollback staged install");

                }

                }

            }

            }

services/tests/mockingservicestests/Android.bp

+1 −0
Original line number Original line Diff line number Diff line
@@ -53,6 +53,7 @@ android_test { 
        "mockito-target-extended-minus-junit4",

        "mockito-target-extended-minus-junit4",

        "platform-compat-test-rules",

        "platform-compat-test-rules",

        "platform-test-annotations",

        "platform-test-annotations",

        "PlatformProperties",

        "service-blobstore",

        "service-blobstore",

        "service-jobscheduler",

        "service-jobscheduler",

        "service-permission.impl",

        "service-permission.impl",

services/tests/mockingservicestests/src/com/android/server/RescuePartyTest.java

+19 −19
Original line number Original line Diff line number Diff line
@@ -45,6 +45,7 @@ import android.os.SystemProperties; 
import android.os.UserHandle;

import android.os.UserHandle;

import android.provider.DeviceConfig;

import android.provider.DeviceConfig;

import android.provider.Settings;

import android.provider.Settings;

import android.sysprop.CrashRecoveryProperties;

import android.util.ArraySet;

import android.util.ArraySet;





import com.android.dx.mockito.inline.extended.ExtendedMockito;

import com.android.dx.mockito.inline.extended.ExtendedMockito;
@@ -95,7 +96,6 @@ public class RescuePartyTest { 
            "persist.device_config.configuration.disable_rescue_party";

            "persist.device_config.configuration.disable_rescue_party";

    private static final String PROP_DISABLE_FACTORY_RESET_FLAG =

    private static final String PROP_DISABLE_FACTORY_RESET_FLAG =

            "persist.device_config.configuration.disable_rescue_party_factory_reset";

            "persist.device_config.configuration.disable_rescue_party_factory_reset";

    private static final String PROP_LAST_FACTORY_RESET_TIME_MS = "persist.sys.last_factory_reset";





    private static final int THROTTLING_DURATION_MIN = 10;

    private static final int THROTTLING_DURATION_MIN = 10;
@@ -211,8 +211,8 @@ public class RescuePartyTest { 




        doReturn(CURRENT_NETWORK_TIME_MILLIS).when(() -> RescueParty.getElapsedRealtime());

        doReturn(CURRENT_NETWORK_TIME_MILLIS).when(() -> RescueParty.getElapsedRealtime());





        SystemProperties.set(RescueParty.PROP_RESCUE_BOOT_COUNT, Integer.toString(0));

        CrashRecoveryProperties.rescueBootCount(0);

        SystemProperties.set(RescueParty.PROP_ENABLE_RESCUE, Boolean.toString(true));

        CrashRecoveryProperties.enableRescueParty(true);

        SystemProperties.set(PROP_DEVICE_CONFIG_DISABLE_FLAG, Boolean.toString(false));

        SystemProperties.set(PROP_DEVICE_CONFIG_DISABLE_FLAG, Boolean.toString(false));

    }

    }
@@ -255,7 +255,7 @@ public class RescuePartyTest { 
        noteBoot(4);

        noteBoot(4);

        assertTrue(RescueParty.isRebootPropertySet());

        assertTrue(RescueParty.isRebootPropertySet());





        SystemProperties.set(RescueParty.PROP_ATTEMPTING_REBOOT, Boolean.toString(false));

        CrashRecoveryProperties.attemptingReboot(false);

        noteBoot(5);

        noteBoot(5);

        assertTrue(RescueParty.isFactoryResetPropertySet());

        assertTrue(RescueParty.isFactoryResetPropertySet());

    }

    }
@@ -280,7 +280,7 @@ public class RescuePartyTest { 
        noteAppCrash(4, true);

        noteAppCrash(4, true);

        assertTrue(RescueParty.isRebootPropertySet());

        assertTrue(RescueParty.isRebootPropertySet());





        SystemProperties.set(RescueParty.PROP_ATTEMPTING_REBOOT, Boolean.toString(false));

        CrashRecoveryProperties.attemptingReboot(false);

        noteAppCrash(5, true);

        noteAppCrash(5, true);

        assertTrue(RescueParty.isFactoryResetPropertySet());

        assertTrue(RescueParty.isFactoryResetPropertySet());

    }

    }
@@ -438,7 +438,7 @@ public class RescuePartyTest { 
            noteBoot(i + 1);

            noteBoot(i + 1);

        }

        }

        assertFalse(RescueParty.isFactoryResetPropertySet());

        assertFalse(RescueParty.isFactoryResetPropertySet());

        SystemProperties.set(RescueParty.PROP_ATTEMPTING_REBOOT, Boolean.toString(false));

        CrashRecoveryProperties.attemptingReboot(false);

        noteBoot(LEVEL_FACTORY_RESET + 1);

        noteBoot(LEVEL_FACTORY_RESET + 1);

        assertTrue(RescueParty.isAttemptingFactoryReset());

        assertTrue(RescueParty.isAttemptingFactoryReset());

        assertTrue(RescueParty.isFactoryResetPropertySet());

        assertTrue(RescueParty.isFactoryResetPropertySet());
@@ -456,7 +456,7 @@ public class RescuePartyTest { 
        noteBoot(mitigationCount++);

        noteBoot(mitigationCount++);

        assertFalse(RescueParty.isFactoryResetPropertySet());

        assertFalse(RescueParty.isFactoryResetPropertySet());

        noteBoot(mitigationCount++);

        noteBoot(mitigationCount++);

        SystemProperties.set(RescueParty.PROP_ATTEMPTING_REBOOT, Boolean.toString(false));

        CrashRecoveryProperties.attemptingReboot(false);

        noteBoot(mitigationCount + 1);

        noteBoot(mitigationCount + 1);

        assertTrue(RescueParty.isAttemptingFactoryReset());

        assertTrue(RescueParty.isAttemptingFactoryReset());

        assertTrue(RescueParty.isFactoryResetPropertySet());

        assertTrue(RescueParty.isFactoryResetPropertySet());
@@ -464,10 +464,10 @@ public class RescuePartyTest { 




    @Test

    @Test

    public void testThrottlingOnBootFailures() {

    public void testThrottlingOnBootFailures() {

        SystemProperties.set(RescueParty.PROP_ATTEMPTING_REBOOT, Boolean.toString(false));

        CrashRecoveryProperties.attemptingReboot(false);

        long now = System.currentTimeMillis();

        long now = System.currentTimeMillis();

        long beforeTimeout = now - TimeUnit.MINUTES.toMillis(THROTTLING_DURATION_MIN - 1);

        long beforeTimeout = now - TimeUnit.MINUTES.toMillis(THROTTLING_DURATION_MIN - 1);

        SystemProperties.set(PROP_LAST_FACTORY_RESET_TIME_MS, Long.toString(beforeTimeout));

        CrashRecoveryProperties.lastFactoryResetTimeMs(beforeTimeout);

        for (int i = 1; i <= LEVEL_FACTORY_RESET; i++) {

        for (int i = 1; i <= LEVEL_FACTORY_RESET; i++) {

            noteBoot(i);

            noteBoot(i);

        }

        }
@@ -476,10 +476,10 @@ public class RescuePartyTest { 




    @Test

    @Test

    public void testThrottlingOnAppCrash() {

    public void testThrottlingOnAppCrash() {

        SystemProperties.set(RescueParty.PROP_ATTEMPTING_REBOOT, Boolean.toString(false));

        CrashRecoveryProperties.attemptingReboot(false);

        long now = System.currentTimeMillis();

        long now = System.currentTimeMillis();

        long beforeTimeout = now - TimeUnit.MINUTES.toMillis(THROTTLING_DURATION_MIN - 1);

        long beforeTimeout = now - TimeUnit.MINUTES.toMillis(THROTTLING_DURATION_MIN - 1);

        SystemProperties.set(PROP_LAST_FACTORY_RESET_TIME_MS, Long.toString(beforeTimeout));

        CrashRecoveryProperties.lastFactoryResetTimeMs(beforeTimeout);

        for (int i = 0; i <= LEVEL_FACTORY_RESET; i++) {

        for (int i = 0; i <= LEVEL_FACTORY_RESET; i++) {

            noteAppCrash(i + 1, true);

            noteAppCrash(i + 1, true);

        }

        }
@@ -488,10 +488,10 @@ public class RescuePartyTest { 




    @Test

    @Test

    public void testNotThrottlingAfterTimeoutOnBootFailures() {

    public void testNotThrottlingAfterTimeoutOnBootFailures() {

        SystemProperties.set(RescueParty.PROP_ATTEMPTING_REBOOT, Boolean.toString(false));

        CrashRecoveryProperties.attemptingReboot(false);

        long now = System.currentTimeMillis();

        long now = System.currentTimeMillis();

        long afterTimeout = now - TimeUnit.MINUTES.toMillis(THROTTLING_DURATION_MIN + 1);

        long afterTimeout = now - TimeUnit.MINUTES.toMillis(THROTTLING_DURATION_MIN + 1);

        SystemProperties.set(PROP_LAST_FACTORY_RESET_TIME_MS, Long.toString(afterTimeout));

        CrashRecoveryProperties.lastFactoryResetTimeMs(afterTimeout);

        for (int i = 1; i <= LEVEL_FACTORY_RESET; i++) {

        for (int i = 1; i <= LEVEL_FACTORY_RESET; i++) {

            noteBoot(i);

            noteBoot(i);

        }

        }
@@ -499,10 +499,10 @@ public class RescuePartyTest { 
    }

    }

    @Test

    @Test

    public void testNotThrottlingAfterTimeoutOnAppCrash() {

    public void testNotThrottlingAfterTimeoutOnAppCrash() {

        SystemProperties.set(RescueParty.PROP_ATTEMPTING_REBOOT, Boolean.toString(false));

        CrashRecoveryProperties.attemptingReboot(false);

        long now = System.currentTimeMillis();

        long now = System.currentTimeMillis();

        long afterTimeout = now - TimeUnit.MINUTES.toMillis(THROTTLING_DURATION_MIN + 1);

        long afterTimeout = now - TimeUnit.MINUTES.toMillis(THROTTLING_DURATION_MIN + 1);

        SystemProperties.set(PROP_LAST_FACTORY_RESET_TIME_MS, Long.toString(afterTimeout));

        CrashRecoveryProperties.lastFactoryResetTimeMs(afterTimeout);

        for (int i = 0; i <= LEVEL_FACTORY_RESET; i++) {

        for (int i = 0; i <= LEVEL_FACTORY_RESET; i++) {

            noteAppCrash(i + 1, true);

            noteAppCrash(i + 1, true);

        }

        }
@@ -525,26 +525,26 @@ public class RescuePartyTest { 




    @Test

    @Test

    public void testExplicitlyEnablingAndDisablingRescue() {

    public void testExplicitlyEnablingAndDisablingRescue() {

        SystemProperties.set(RescueParty.PROP_ENABLE_RESCUE, Boolean.toString(false));

        CrashRecoveryProperties.enableRescueParty(false);

        SystemProperties.set(PROP_DISABLE_RESCUE, Boolean.toString(true));

        SystemProperties.set(PROP_DISABLE_RESCUE, Boolean.toString(true));

        assertEquals(RescuePartyObserver.getInstance(mMockContext).execute(sFailingPackage,

        assertEquals(RescuePartyObserver.getInstance(mMockContext).execute(sFailingPackage,

                PackageWatchdog.FAILURE_REASON_APP_NOT_RESPONDING, 1), false);

                PackageWatchdog.FAILURE_REASON_APP_NOT_RESPONDING, 1), false);





        SystemProperties.set(RescueParty.PROP_ENABLE_RESCUE, Boolean.toString(true));

        CrashRecoveryProperties.enableRescueParty(true);

        assertTrue(RescuePartyObserver.getInstance(mMockContext).execute(sFailingPackage,

        assertTrue(RescuePartyObserver.getInstance(mMockContext).execute(sFailingPackage,

                PackageWatchdog.FAILURE_REASON_APP_NOT_RESPONDING, 1));

                PackageWatchdog.FAILURE_REASON_APP_NOT_RESPONDING, 1));

    }

    }





    @Test

    @Test

    public void testDisablingRescueByDeviceConfigFlag() {

    public void testDisablingRescueByDeviceConfigFlag() {

        SystemProperties.set(RescueParty.PROP_ENABLE_RESCUE, Boolean.toString(false));

        CrashRecoveryProperties.enableRescueParty(false);

        SystemProperties.set(PROP_DEVICE_CONFIG_DISABLE_FLAG, Boolean.toString(true));

        SystemProperties.set(PROP_DEVICE_CONFIG_DISABLE_FLAG, Boolean.toString(true));





        assertEquals(RescuePartyObserver.getInstance(mMockContext).execute(sFailingPackage,

        assertEquals(RescuePartyObserver.getInstance(mMockContext).execute(sFailingPackage,

                PackageWatchdog.FAILURE_REASON_APP_NOT_RESPONDING, 1), false);

                PackageWatchdog.FAILURE_REASON_APP_NOT_RESPONDING, 1), false);





        // Restore the property value initialized in SetUp()

        // Restore the property value initialized in SetUp()

        SystemProperties.set(RescueParty.PROP_ENABLE_RESCUE, Boolean.toString(true));

        CrashRecoveryProperties.enableRescueParty(true);

        SystemProperties.set(PROP_DEVICE_CONFIG_DISABLE_FLAG, Boolean.toString(false));

        SystemProperties.set(PROP_DEVICE_CONFIG_DISABLE_FLAG, Boolean.toString(false));

    }

    }