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

Commit c9dfa59d authored by Pyuli Naithani's avatar Pyuli Naithani
Browse files

Add stats log for sensor privacy. 

Test: statsd_testdrive 381

BUG: 192269258

Change-Id: I69e766b789b4b4a6ca0ad26b3a312baca3dbccd2
parent af63867d

src/com/android/settings/display/AdaptiveSleepCameraStatePreferenceController.java

+2 −1
Original line number Diff line number Diff line
@@ -17,6 +17,7 @@ 
package com.android.settings.display;



import static android.hardware.SensorPrivacyManager.Sensors.CAMERA;

import static android.hardware.SensorPrivacyManager.Sources.DIALOG;



import android.content.Context;

import android.hardware.SensorPrivacyManager;
@@ -77,7 +78,7 @@ public class AdaptiveSleepCameraStatePreferenceController { 
            mPreference.setSummary(R.string.adaptive_sleep_camera_lock_summary);

            mPreference.setPositiveButtonText(R.string.allow);

            mPreference.setPositiveButtonOnClickListener(

                    p -> mPrivacyManager.setSensorPrivacy(CAMERA, false));

                    p -> mPrivacyManager.setSensorPrivacy(DIALOG, CAMERA, false));

        }

    }

}

src/com/android/settings/display/SmartAutoRotateCameraStateController.java

+2 −1
Original line number Diff line number Diff line
@@ -17,6 +17,7 @@ 
package com.android.settings.display;



import static android.hardware.SensorPrivacyManager.Sensors.CAMERA;

import static android.hardware.SensorPrivacyManager.Sources.DIALOG;



import static com.android.settings.display.SmartAutoRotateController.isRotationResolverServiceAvailable;
@@ -63,7 +64,7 @@ public class SmartAutoRotateCameraStateController extends BasePreferenceControll 
        ((BannerMessagePreference) mPreference)

                .setPositiveButtonText(R.string.allow)

                .setPositiveButtonOnClickListener(v -> {

                    mPrivacyManager.setSensorPrivacy(CAMERA, false);

                    mPrivacyManager.setSensorPrivacy(DIALOG, CAMERA, false);

                });

    }

src/com/android/settings/privacy/SensorToggleController.java

+3 −1
Original line number Diff line number Diff line
@@ -16,6 +16,8 @@ 


package com.android.settings.privacy;



import static android.hardware.SensorPrivacyManager.Sources.SETTINGS;



import android.content.Context;



import androidx.preference.PreferenceScreen;
@@ -51,7 +53,7 @@ public abstract class SensorToggleController extends TogglePreferenceController 


    @Override

    public boolean setChecked(boolean isChecked) {

        mSensorPrivacyManagerHelper.setSensorBlocked(getSensor(), !isChecked);

        mSensorPrivacyManagerHelper.setSensorBlocked(SETTINGS, getSensor(), !isChecked);

        return true;

    }

src/com/android/settings/utils/SensorPrivacyManagerHelper.java

+6 −4
Original line number Diff line number Diff line
@@ -153,21 +153,23 @@ public class SensorPrivacyManagerHelper { 


    /**

     * Sets the sensor privacy for the current user.

     * @param source The source with which sensor privacy is toggled.

     * @param sensor The sensor to set for

     * @param blocked The state to set to

     */

    public void setSensorBlocked(int sensor, boolean blocked) {

        mSensorPrivacyManager.setSensorPrivacy(sensor, blocked);

    public void setSensorBlocked(int source, int sensor, boolean blocked) {

        mSensorPrivacyManager.setSensorPrivacy(source, sensor, blocked);

    }



    /**

     * Sets the sensor privacy for the given user.

     * @param source The source with which sensor privacy is toggled.

     * @param sensor The sensor to set for

     * @param blocked The state to set to

     * @param userId The user to set for

     */

    public void setSensorBlocked(int sensor, boolean blocked, int userId) {

        mSensorPrivacyManager.setSensorPrivacy(sensor, blocked, userId);

    public void setSensorBlocked(int source, int sensor, boolean blocked, int userId) {

        mSensorPrivacyManager.setSensorPrivacy(source, sensor, blocked, userId);

    }



    /**

tests/robotests/src/com/android/settings/privacy/SensorToggleControllerTest.java

+20 −18
Original line number Diff line number Diff line
@@ -18,11 +18,13 @@ package com.android.settings.privacy; 


import static android.hardware.SensorPrivacyManager.Sensors.CAMERA;

import static android.hardware.SensorPrivacyManager.Sensors.MICROPHONE;

import static android.hardware.SensorPrivacyManager.Sources.OTHER;



import static org.junit.Assert.assertFalse;

import static org.junit.Assert.assertTrue;

import static org.mockito.ArgumentMatchers.any;

import static org.mockito.ArgumentMatchers.anyBoolean;

import static org.mockito.ArgumentMatchers.anyInt;

import static org.mockito.ArgumentMatchers.eq;

import static org.mockito.Mockito.doAnswer;

import static org.mockito.Mockito.doReturn;
@@ -91,14 +93,14 @@ public class SensorToggleControllerTest { 
                listener.onSensorPrivacyChanged(MICROPHONE, mMicState);

            }

            return null;

        }).when(mSensorPrivacyManager).setSensorPrivacy(eq(MICROPHONE), anyBoolean());

        }).when(mSensorPrivacyManager).setSensorPrivacy(anyInt(), eq(MICROPHONE), anyBoolean());

        doAnswer(invocation -> {

            mCamState = invocation.getArgument(1);

            for (OnSensorPrivacyChangedListener listener : mMicListeners) {

                listener.onSensorPrivacyChanged(CAMERA, mMicState);

            }

            return null;

        }).when(mSensorPrivacyManager).setSensorPrivacy(eq(CAMERA), anyBoolean());

        }).when(mSensorPrivacyManager).setSensorPrivacy(anyInt(), eq(CAMERA), anyBoolean());



        doAnswer(invocation -> mMicListeners.add(invocation.getArgument(1)))

                .when(mSensorPrivacyManager).addSensorPrivacyListener(eq(MICROPHONE), any());
@@ -108,37 +110,37 @@ public class SensorToggleControllerTest { 


    @Test

    public void isChecked_disableMicrophoneSensorPrivacy_returnTrue() {

        mSensorPrivacyManager.setSensorPrivacy(MICROPHONE, false);

        mSensorPrivacyManager.setSensorPrivacy(OTHER, MICROPHONE, false);

        MicToggleController micToggleController = new MicToggleController(mContext, "mic_toggle");

        assertTrue(micToggleController.isChecked());

    }



    @Test

    public void isChecked_enableMicrophoneSensorPrivacy_returnFalse() {

        mSensorPrivacyManager.setSensorPrivacy(MICROPHONE, true);

        mSensorPrivacyManager.setSensorPrivacy(OTHER, MICROPHONE, true);

        MicToggleController micToggleController = new MicToggleController(mContext, "mic_toggle");

        assertFalse(micToggleController.isChecked());

    }



    @Test

    public void isChecked_disableMicrophoneSensorPrivacyThenChanged_returnFalse() {

        mSensorPrivacyManager.setSensorPrivacy(MICROPHONE, false);

        mSensorPrivacyManager.setSensorPrivacy(OTHER, MICROPHONE, false);

        MicToggleController micToggleController = new MicToggleController(mContext, "mic_toggle");

        mSensorPrivacyManager.setSensorPrivacy(MICROPHONE, true);

        mSensorPrivacyManager.setSensorPrivacy(OTHER, MICROPHONE, true);

        assertFalse(micToggleController.isChecked());

    }



    @Test

    public void isChecked_enableMicrophoneSensorPrivacyThenChanged_returnTrue() {

        mSensorPrivacyManager.setSensorPrivacy(MICROPHONE, true);

        mSensorPrivacyManager.setSensorPrivacy(OTHER, MICROPHONE, true);

        MicToggleController micToggleController = new MicToggleController(mContext, "mic_toggle");

        mSensorPrivacyManager.setSensorPrivacy(MICROPHONE, false);

        mSensorPrivacyManager.setSensorPrivacy(OTHER, MICROPHONE, false);

        assertTrue(micToggleController.isChecked());

    }



    @Test

    public void isMicrophoneSensorPrivacyEnabled_uncheckMicToggle_returnTrue() {

        mSensorPrivacyManager.setSensorPrivacy(MICROPHONE, false);

        mSensorPrivacyManager.setSensorPrivacy(OTHER, MICROPHONE, false);

        MicToggleController micToggleController = new MicToggleController(mContext, "mic_toggle");

        micToggleController.setChecked(false);

        assertTrue(mMicState);
@@ -146,7 +148,7 @@ public class SensorToggleControllerTest { 


    @Test

    public void isMicrophoneSensorPrivacyEnabled_checkMicToggle_returnFalse() {

        mSensorPrivacyManager.setSensorPrivacy(MICROPHONE, true);

        mSensorPrivacyManager.setSensorPrivacy(OTHER, MICROPHONE, true);

        MicToggleController micToggleController = new MicToggleController(mContext, "mic_toggle");

        micToggleController.setChecked(true);

        assertFalse(mMicState);
@@ -154,7 +156,7 @@ public class SensorToggleControllerTest { 


    @Test

    public void isChecked_disableCameraSensorPrivacy_returnTrue() {

        mSensorPrivacyManager.setSensorPrivacy(CAMERA, false);

        mSensorPrivacyManager.setSensorPrivacy(OTHER, CAMERA, false);

        CameraToggleController camToggleController =

                new CameraToggleController(mContext, "cam_toggle");

        assertTrue(camToggleController.isChecked());
@@ -162,7 +164,7 @@ public class SensorToggleControllerTest { 


    @Test

    public void isChecked_enableCameraSensorPrivacy_returnFalse() {

        mSensorPrivacyManager.setSensorPrivacy(CAMERA, true);

        mSensorPrivacyManager.setSensorPrivacy(OTHER, CAMERA, true);

        CameraToggleController camToggleController =

                new CameraToggleController(mContext, "cam_toggle");

        assertFalse(camToggleController.isChecked());
@@ -170,25 +172,25 @@ public class SensorToggleControllerTest { 


    @Test

    public void isChecked_disableCameraSensorPrivacyThenChanged_returnFalse() {

        mSensorPrivacyManager.setSensorPrivacy(CAMERA, false);

        mSensorPrivacyManager.setSensorPrivacy(OTHER, CAMERA, false);

        CameraToggleController camToggleController =

                new CameraToggleController(mContext, "cam_toggle");

        mSensorPrivacyManager.setSensorPrivacy(CAMERA, true);

        mSensorPrivacyManager.setSensorPrivacy(OTHER, CAMERA, true);

        assertFalse(camToggleController.isChecked());

    }



    @Test

    public void isChecked_enableCameraSensorPrivacyThenChanged_returnTrue() {

        mSensorPrivacyManager.setSensorPrivacy(CAMERA, true);

        mSensorPrivacyManager.setSensorPrivacy(OTHER, CAMERA, true);

        CameraToggleController camToggleController =

                new CameraToggleController(mContext, "cam_toggle");

        mSensorPrivacyManager.setSensorPrivacy(CAMERA, false);

        mSensorPrivacyManager.setSensorPrivacy(OTHER, CAMERA, false);

        assertTrue(camToggleController.isChecked());

    }



    @Test

    public void isCameraSensorPrivacyEnabled_uncheckMicToggle_returnTrue() {

        mSensorPrivacyManager.setSensorPrivacy(CAMERA, false);

        mSensorPrivacyManager.setSensorPrivacy(OTHER, CAMERA, false);

        CameraToggleController camToggleController =

                new CameraToggleController(mContext, "cam_toggle");

        camToggleController.setChecked(false);
@@ -197,7 +199,7 @@ public class SensorToggleControllerTest { 


    @Test

    public void isCameraSensorPrivacyEnabled_checkMicToggle_returnFalse() {

        mSensorPrivacyManager.setSensorPrivacy(CAMERA, true);

        mSensorPrivacyManager.setSensorPrivacy(OTHER, CAMERA, true);

        CameraToggleController camToggleController =

                new CameraToggleController(mContext, "cam_toggle");

        camToggleController.setChecked(true);