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

Commit 56c191d2 authored by Tsung-Mao Fang's avatar Tsung-Mao Fang Committed by Automerger Merge Worker
Browse files

Wi-Fi panel doesn't need to check permission am: 5e785a2d 

Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Settings/+/19634561



Change-Id: I78c410c92c0d7f8d828fab92db97303814a09ccf
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 070c0458 5e785a2d

res/values/config.xml

+6 −0
Original line number Diff line number Diff line
@@ -620,4 +620,10 @@ 


    <!-- Whether to put the apps with system UID into system component bucket or not -->

    <bool name="config_battery_combine_system_components">false</bool>



    <!-- An array of uid name for which packages exempt from Wi-Fi permission check. -->

    <string-array name="config_exempt_wifi_permission_uid_name" translatable="false">

        <item>@string/config_settingsintelligence_package_name</item>

        <item>android.uid.system:1000</item>

    </string-array>

</resources>

src/com/android/settings/wifi/slice/WifiSlice.java

+16 −1
Original line number Diff line number Diff line
@@ -98,7 +98,7 @@ public class WifiSlice implements CustomSliceable { 
    public Slice getSlice() {

        // If external calling package doesn't have Wi-Fi permission.

        final boolean isPermissionGranted =

                Utils.isSettingsIntelligence(mContext) || isPermissionGranted(mContext);

                isCallerExemptUid(mContext) || isPermissionGranted(mContext);

        final boolean isWifiEnabled = isWifiEnabled();

        ListBuilder listBuilder = getListBuilder(isWifiEnabled, null /* wifiSliceItem */,

                isPermissionGranted);
@@ -139,6 +139,21 @@ public class WifiSlice implements CustomSliceable { 
        return listBuilder.build();

    }



    private boolean isCallerExemptUid(Context context) {

        final String[] allowedUidNames = context.getResources().getStringArray(

                R.array.config_exempt_wifi_permission_uid_name);

        final String uidName =

                context.getPackageManager().getNameForUid(Binder.getCallingUid());

        Log.d(TAG, "calling uid name : " + uidName);



        for (String allowedUidName : allowedUidNames) {

            if (TextUtils.equals(uidName, allowedUidName)) {

                return true;

            }

        }

        return false;

    }



    private static boolean isPermissionGranted(Context settingsContext) {

        final int callingUid = Binder.getCallingUid();

        final String callingPackage = settingsContext.getPackageManager()

tests/robotests/src/com/android/settings/wifi/slice/ContextualWifiSliceTest.java

+1 −0
Original line number Diff line number Diff line
@@ -103,6 +103,7 @@ public class ContextualWifiSliceTest { 
                mContext.getString(R.string.config_settingsintelligence_package_name);

        ShadowBinder.setCallingUid(1);

        when(mPackageManager.getPackagesForUid(1)).thenReturn(new String[]{siPackageName});

        when(mPackageManager.getNameForUid(1)).thenReturn(siPackageName);

        ShadowWifiSlice.setWifiPermissible(true);

        mWifiSlice = new ContextualWifiSlice(mContext);

    }

tests/robotests/src/com/android/settings/wifi/slice/WifiSliceTest.java

+4 −0
Original line number Diff line number Diff line
@@ -109,6 +109,7 @@ public class WifiSliceTest { 
        mSIPackageName = mContext.getString(R.string.config_settingsintelligence_package_name);

        ShadowBinder.setCallingUid(USER_ID);

        when(mPackageManager.getPackagesForUid(USER_ID)).thenReturn(new String[]{mSIPackageName});

        when(mPackageManager.getNameForUid(USER_ID)).thenReturn(mSIPackageName);

        ShadowWifiSlice.setWifiPermissible(true);

        mWifiSlice = new WifiSlice(mContext, mWifiRestriction);

    }
@@ -116,6 +117,7 @@ public class WifiSliceTest { 
    @Test

    public void getWifiSlice_fromSIPackage_shouldHaveTitleAndToggle() {

        when(mPackageManager.getPackagesForUid(USER_ID)).thenReturn(new String[]{mSIPackageName});

        when(mPackageManager.getNameForUid(USER_ID)).thenReturn(mSIPackageName);

        ShadowWifiSlice.setWifiPermissible(false);



        final Slice wifiSlice = mWifiSlice.getSlice();
@@ -131,6 +133,7 @@ public class WifiSliceTest { 
    @Test

    public void getWifiSlice_notFromSIPackageAndWithWifiPermission_shouldHaveTitleAndToggle() {

        when(mPackageManager.getPackagesForUid(USER_ID)).thenReturn(new String[]{"com.test"});

        when(mPackageManager.getNameForUid(USER_ID)).thenReturn("com.test");

        ShadowWifiSlice.setWifiPermissible(true);



        final Slice wifiSlice = mWifiSlice.getSlice();
@@ -145,6 +148,7 @@ public class WifiSliceTest { 
    @Test

    public void getWifiSlice_notFromSIPackageAndWithoutWifiPermission_shouldReturnNoToggle() {

        when(mPackageManager.getPackagesForUid(USER_ID)).thenReturn(new String[]{"com.test"});

        when(mPackageManager.getNameForUid(USER_ID)).thenReturn("com.test");

        ShadowWifiSlice.setWifiPermissible(false);



        final Slice wifiSlice = mWifiSlice.getSlice();