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

Commit db8be2d8 authored by Yuri Lin's avatar Yuri Lin Committed by Automerger Merge Worker
Browse files

Merge "Move zen mode evaluation back into setConfig." into udc-dev am: a80ba02e am: 0ff41af0 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/22976352



Change-Id: I419815d171c05352a8adcc4773ec197a652ca3c3
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 3e2b6cde 0ff41af0

services/core/java/com/android/server/notification/ZenModeHelper.java

+20 −41
Original line number Original line Diff line number Diff line
@@ -226,7 +226,6 @@ public class ZenModeHelper { 
        mPm = mContext.getPackageManager();

        mPm = mContext.getPackageManager();

        mHandler.postMetricsTimer();

        mHandler.postMetricsTimer();

        cleanUpZenRules();

        cleanUpZenRules();

        evaluateZenMode("onSystemReady", true);

        mIsBootComplete = true;

        mIsBootComplete = true;

        showZenUpgradeNotification(mZenMode);

        showZenUpgradeNotification(mZenMode);

    }

    }
@@ -936,7 +935,10 @@ public class ZenModeHelper { 
            if (policyChanged) {

            if (policyChanged) {

                dispatchOnPolicyChanged();

                dispatchOnPolicyChanged();

            }

            }

            mHandler.postApplyConfig(config, reason, triggeringComponent, setRingerMode);

            final String val = Integer.toString(config.hashCode());

            Global.putString(mContext.getContentResolver(), Global.ZEN_MODE_CONFIG_ETAG, val);

            evaluateZenMode(reason, setRingerMode);

            mConditions.evaluateConfig(config, triggeringComponent, true /*processSubscriptions*/);

            return true;

            return true;

        } catch (SecurityException e) {

        } catch (SecurityException e) {

            Log.wtf(TAG, "Invalid rule in config", e);

            Log.wtf(TAG, "Invalid rule in config", e);
@@ -946,14 +948,6 @@ public class ZenModeHelper { 
        }

        }

    }

    }





    private void applyConfig(ZenModeConfig config, String reason,

            ComponentName triggeringComponent, boolean setRingerMode) {

        final String val = Integer.toString(config.hashCode());

        Global.putString(mContext.getContentResolver(), Global.ZEN_MODE_CONFIG_ETAG, val);

        evaluateZenMode(reason, setRingerMode);

        mConditions.evaluateConfig(config, triggeringComponent, true /*processSubscriptions*/);

    }



    private int getZenModeSetting() {

    private int getZenModeSetting() {

        return Global.getInt(mContext.getContentResolver(), Global.ZEN_MODE, Global.ZEN_MODE_OFF);

        return Global.getInt(mContext.getContentResolver(), Global.ZEN_MODE, Global.ZEN_MODE_OFF);

    }

    }
@@ -987,21 +981,23 @@ public class ZenModeHelper { 
        mZenMode = zen;

        mZenMode = zen;

        setZenModeSetting(mZenMode);

        setZenModeSetting(mZenMode);

        updateConsolidatedPolicy(reason);

        updateConsolidatedPolicy(reason);

        updateRingerModeAffectedStreams();

        boolean shouldApplyToRinger = setRingerMode && (zen != zenBefore || (

        if (setRingerMode && (zen != zenBefore || (zen == Global.ZEN_MODE_IMPORTANT_INTERRUPTIONS

                zen == Global.ZEN_MODE_IMPORTANT_INTERRUPTIONS

                && policyHashBefore != mConsolidatedPolicy.hashCode()))) {

                        && policyHashBefore != mConsolidatedPolicy.hashCode()));

            applyZenToRingerMode();

        mHandler.postUpdateRingerAndAudio(shouldApplyToRinger);

        }

        applyRestrictions();

        if (zen != zenBefore) {

        if (zen != zenBefore) {

            mHandler.postDispatchOnZenModeChanged();

            mHandler.postDispatchOnZenModeChanged();

        }

        }

    }

    }





    private void updateRingerModeAffectedStreams() {

    private void updateRingerAndAudio(boolean shouldApplyToRinger) {

        if (mAudioManager != null) {

        if (mAudioManager != null) {

            mAudioManager.updateRingerModeAffectedStreamsInternal();

            mAudioManager.updateRingerModeAffectedStreamsInternal();

        }

        }

        if (shouldApplyToRinger) {

            applyZenToRingerMode();

        }

        applyRestrictions();

    }

    }





    private int computeZenMode() {

    private int computeZenMode() {
@@ -1620,22 +1616,7 @@ public class ZenModeHelper { 
    private final class H extends Handler {

    private final class H extends Handler {

        private static final int MSG_DISPATCH = 1;

        private static final int MSG_DISPATCH = 1;

        private static final int MSG_METRICS = 2;

        private static final int MSG_METRICS = 2;

        private static final int MSG_APPLY_CONFIG = 4;

        private static final int MSG_RINGER_AUDIO = 5;



        private final class ConfigMessageData {

            public final ZenModeConfig config;

            public ComponentName triggeringComponent;

            public final String reason;

            public final boolean setRingerMode;



            ConfigMessageData(ZenModeConfig config, String reason,

                    ComponentName triggeringComponent, boolean setRingerMode) {

                this.config = config;

                this.reason = reason;

                this.setRingerMode = setRingerMode;

                this.triggeringComponent = triggeringComponent;

            }

        }





        private static final long METRICS_PERIOD_MS = 6 * 60 * 60 * 1000;

        private static final long METRICS_PERIOD_MS = 6 * 60 * 60 * 1000;
@@ -1653,10 +1634,9 @@ public class ZenModeHelper { 
            sendEmptyMessageDelayed(MSG_METRICS, METRICS_PERIOD_MS);

            sendEmptyMessageDelayed(MSG_METRICS, METRICS_PERIOD_MS);

        }

        }





        private void postApplyConfig(ZenModeConfig config, String reason,

        private void postUpdateRingerAndAudio(boolean shouldApplyToRinger) {

                ComponentName triggeringComponent, boolean setRingerMode) {

            removeMessages(MSG_RINGER_AUDIO);

            sendMessage(obtainMessage(MSG_APPLY_CONFIG,

            sendMessage(obtainMessage(MSG_RINGER_AUDIO, shouldApplyToRinger));

                    new ConfigMessageData(config, reason, triggeringComponent, setRingerMode)));

        }

        }





        @Override

        @Override
@@ -1668,10 +1648,9 @@ public class ZenModeHelper { 
                case MSG_METRICS:

                case MSG_METRICS:

                    mMetrics.emit();

                    mMetrics.emit();

                    break;

                    break;

                case MSG_APPLY_CONFIG:

                case MSG_RINGER_AUDIO:

                    ConfigMessageData applyConfigData = (ConfigMessageData) msg.obj;

                    boolean shouldApplyToRinger = (boolean) msg.obj;

                    applyConfig(applyConfigData.config, applyConfigData.reason,

                    updateRingerAndAudio(shouldApplyToRinger);

                            applyConfigData.triggeringComponent, applyConfigData.setRingerMode);

            }

            }

        }

        }

    }

    }

services/tests/uiservicestests/src/com/android/server/notification/RoleObserverTest.java

+4 −1
Original line number Original line Diff line number Diff line
@@ -55,6 +55,7 @@ import android.telecom.TelecomManager; 
import android.telephony.TelephonyManager;

import android.telephony.TelephonyManager;

import android.test.suitebuilder.annotation.SmallTest;

import android.test.suitebuilder.annotation.SmallTest;

import android.testing.AndroidTestingRunner;

import android.testing.AndroidTestingRunner;

import android.testing.TestableLooper;

import android.testing.TestableLooper.RunWithLooper;

import android.testing.TestableLooper.RunWithLooper;

import android.util.ArraySet;

import android.util.ArraySet;

import android.util.AtomicFile;

import android.util.AtomicFile;
@@ -103,6 +104,7 @@ public class RoleObserverTest extends UiServiceTestCase { 
    private RoleManager mRoleManager;

    private RoleManager mRoleManager;

    @Mock

    @Mock

    private Looper mMainLooper;

    private Looper mMainLooper;

    private TestableLooper mTestableLooper;

    NotificationRecordLoggerFake mNotificationRecordLogger = new NotificationRecordLoggerFake();

    NotificationRecordLoggerFake mNotificationRecordLogger = new NotificationRecordLoggerFake();

    private InstanceIdSequence mNotificationInstanceIdSequence = new InstanceIdSequenceFake(

    private InstanceIdSequence mNotificationInstanceIdSequence = new InstanceIdSequenceFake(

            1 << 30);

            1 << 30);
@@ -144,9 +146,10 @@ public class RoleObserverTest extends UiServiceTestCase { 
        mService = new TestableNotificationManagerService(mContext, mNotificationRecordLogger,

        mService = new TestableNotificationManagerService(mContext, mNotificationRecordLogger,

                mNotificationInstanceIdSequence);

                mNotificationInstanceIdSequence);

        mRoleObserver = mService.new RoleObserver(mContext, mRoleManager, mPm, mMainLooper);

        mRoleObserver = mService.new RoleObserver(mContext, mRoleManager, mPm, mMainLooper);

        mTestableLooper = TestableLooper.get(this);





        try {

        try {

            mService.init(mService.new WorkerHandler(mock(Looper.class)),

            mService.init(mService.new WorkerHandler(mTestableLooper.getLooper()),

                    mock(RankingHandler.class),

                    mock(RankingHandler.class),

                    mock(IPackageManager.class), mock(PackageManager.class),

                    mock(IPackageManager.class), mock(PackageManager.class),

                    mock(LightsManager.class),

                    mock(LightsManager.class),

services/tests/uiservicestests/src/com/android/server/notification/ZenModeHelperTest.java

+32 −3
Original line number Original line Diff line number Diff line
@@ -175,6 +175,7 @@ public class ZenModeHelperTest extends UiServiceTestCase { 
        mConditionProviders = new ConditionProviders(mContext, new UserProfiles(),

        mConditionProviders = new ConditionProviders(mContext, new UserProfiles(),

                AppGlobals.getPackageManager());

                AppGlobals.getPackageManager());

        mConditionProviders.addSystemProvider(new CountdownConditionProvider());

        mConditionProviders.addSystemProvider(new CountdownConditionProvider());

        mConditionProviders.addSystemProvider(new ScheduleConditionProvider());

        mZenModeHelper = new ZenModeHelper(mContext, mTestableLooper.getLooper(),

        mZenModeHelper = new ZenModeHelper(mContext, mTestableLooper.getLooper(),

                mConditionProviders, mStatsEventBuilderFactory);

                mConditionProviders, mStatsEventBuilderFactory);
@@ -815,6 +816,32 @@ public class ZenModeHelperTest extends UiServiceTestCase { 
                mZenModeHelper.TAG);

                mZenModeHelper.TAG);

    }

    }





    @Test

    public void testSetConfig_updatesAudioEventually() {

        AudioManagerInternal mAudioManager = mock(AudioManagerInternal.class);

        mZenModeHelper.mAudioManager = mAudioManager;

        setupZenConfig();



        // Change the config a little bit, but enough that it would turn zen mode on

        ZenModeConfig newConfig = mZenModeHelper.mConfig.copy();

        newConfig.manualRule = new ZenModeConfig.ZenRule();

        newConfig.manualRule.zenMode = ZEN_MODE_IMPORTANT_INTERRUPTIONS;

        newConfig.manualRule.enabled = true;

        mZenModeHelper.setConfig(newConfig, null, "test");



        // audio manager shouldn't do anything until the handler processes its messagse

        verify(mAudioManager, never()).updateRingerModeAffectedStreamsInternal();



        // now process the looper's messages

        mTestableLooper.processAllMessages();



        // Expect calls to audio manager

        verify(mAudioManager, times(1)).updateRingerModeAffectedStreamsInternal();



        // called during applyZenToRingerMode(), which should be true since zen changed

        verify(mAudioManager, atLeastOnce()).getRingerModeInternal();

    }



    @Test

    @Test

    public void testParcelConfig() {

    public void testParcelConfig() {

        mZenModeHelper.mZenMode = ZEN_MODE_IMPORTANT_INTERRUPTIONS;

        mZenModeHelper.mZenMode = ZEN_MODE_IMPORTANT_INTERRUPTIONS;
@@ -1827,7 +1854,11 @@ public class ZenModeHelperTest extends UiServiceTestCase { 




    @Test

    @Test

    public void testRulesWithSameUri() {

    public void testRulesWithSameUri() {

        Uri sharedUri = ZenModeConfig.toScheduleConditionId(new ScheduleInfo());

        // needs to be a valid schedule info object for the subscription to happen properly

        ScheduleInfo scheduleInfo = new ScheduleInfo();

        scheduleInfo.days = new int[]{1, 2};

        scheduleInfo.endHour = 1;

        Uri sharedUri = ZenModeConfig.toScheduleConditionId(scheduleInfo);

        AutomaticZenRule zenRule = new AutomaticZenRule("name",

        AutomaticZenRule zenRule = new AutomaticZenRule("name",

                new ComponentName("android", "ScheduleConditionProvider"),

                new ComponentName("android", "ScheduleConditionProvider"),

                sharedUri,

                sharedUri,
@@ -1874,14 +1905,12 @@ public class ZenModeHelperTest extends UiServiceTestCase { 




        // note that caller=null because that's how it comes in from NMS.setZenMode

        // note that caller=null because that's how it comes in from NMS.setZenMode

        mZenModeHelper.setManualZenMode(ZEN_MODE_IMPORTANT_INTERRUPTIONS, null, null, "");

        mZenModeHelper.setManualZenMode(ZEN_MODE_IMPORTANT_INTERRUPTIONS, null, null, "");

        mTestableLooper.processAllMessages();





        // confirm that setting zen mode via setManualZenMode changed the zen mode correctly

        // confirm that setting zen mode via setManualZenMode changed the zen mode correctly

        assertEquals(ZEN_MODE_IMPORTANT_INTERRUPTIONS, mZenModeHelper.mZenMode);

        assertEquals(ZEN_MODE_IMPORTANT_INTERRUPTIONS, mZenModeHelper.mZenMode);





        // and also that it works to turn it back off again

        // and also that it works to turn it back off again

        mZenModeHelper.setManualZenMode(Global.ZEN_MODE_OFF, null, null, "");

        mZenModeHelper.setManualZenMode(Global.ZEN_MODE_OFF, null, null, "");

        mTestableLooper.processAllMessages();

        assertEquals(Global.ZEN_MODE_OFF, mZenModeHelper.mZenMode);

        assertEquals(Global.ZEN_MODE_OFF, mZenModeHelper.mZenMode);

    }

    }