diff options
author | Yi-an Chen <theianchen@google.com> | 2023-04-03 00:15:21 +0000 |
---|---|---|
committer | Yi-an Chen <theianchen@google.com> | 2023-04-04 23:38:21 +0000 |
commit | e84dc424943b1ae29cfb6ed8ba9f854bc31a28a8 (patch) | |
tree | af5a2460412a12cfca4c20aa4550de93d2f0c26b | |
parent | da42d52c893eec83f53b26f90f62035f81d0c9cd (diff) | |
download | Permission-e84dc424943b1ae29cfb6ed8ba9f854bc31a28a8.tar.gz |
Clean up file structure for utils for coverage calculation
Bug: 275388028
We expect to remove 159 lines from the R coverage calculation
Test: build
Change-Id: I748c3bc9ad0a7fa7ac964775b250dad1134a2e78
14 files changed, 28 insertions, 25 deletions
diff --git a/PermissionController/src/com/android/permissioncontroller/permission/model/legacy/PermissionApps.java b/PermissionController/src/com/android/permissioncontroller/permission/model/legacy/PermissionApps.java index 53ba48ace..196bfc6af 100644 --- a/PermissionController/src/com/android/permissioncontroller/permission/model/legacy/PermissionApps.java +++ b/PermissionController/src/com/android/permissioncontroller/permission/model/legacy/PermissionApps.java @@ -38,8 +38,8 @@ import androidx.annotation.Nullable; import com.android.modules.utils.build.SdkLevel; import com.android.permissioncontroller.R; import com.android.permissioncontroller.permission.model.AppPermissionGroup; -import com.android.permissioncontroller.permission.utils.SubattributionUtils; import com.android.permissioncontroller.permission.utils.Utils; +import com.android.permissioncontroller.permission.utils.v31.SubattributionUtils; import java.util.ArrayList; import java.util.Collections; diff --git a/PermissionController/src/com/android/permissioncontroller/permission/service/PermissionControllerServiceImpl.java b/PermissionController/src/com/android/permissioncontroller/permission/service/PermissionControllerServiceImpl.java index 3c1a9df39..82620058d 100644 --- a/PermissionController/src/com/android/permissioncontroller/permission/service/PermissionControllerServiceImpl.java +++ b/PermissionController/src/com/android/permissioncontroller/permission/service/PermissionControllerServiceImpl.java @@ -59,12 +59,12 @@ import com.android.permissioncontroller.permission.model.Permission; import com.android.permissioncontroller.permission.model.livedatatypes.AppPermGroupUiInfo; import com.android.permissioncontroller.permission.model.livedatatypes.AppPermGroupUiInfo.PermGrantState; import com.android.permissioncontroller.permission.ui.AutoGrantPermissionsNotifier; -import com.android.permissioncontroller.permission.utils.AdminRestrictedPermissionsUtils; import com.android.permissioncontroller.permission.utils.ArrayUtils; import com.android.permissioncontroller.permission.utils.KotlinUtils; import com.android.permissioncontroller.permission.utils.PermissionMapping; import com.android.permissioncontroller.permission.utils.UserSensitiveFlagsUtils; import com.android.permissioncontroller.permission.utils.Utils; +import com.android.permissioncontroller.permission.utils.v31.AdminRestrictedPermissionsUtils; import com.android.role.controller.model.Role; import com.android.role.controller.model.Roles; diff --git a/PermissionController/src/com/android/permissioncontroller/permission/ui/auto/AutoAppPermissionFragment.java b/PermissionController/src/com/android/permissioncontroller/permission/ui/auto/AutoAppPermissionFragment.java index ad6b993fa..2de936469 100644 --- a/PermissionController/src/com/android/permissioncontroller/permission/ui/auto/AutoAppPermissionFragment.java +++ b/PermissionController/src/com/android/permissioncontroller/permission/ui/auto/AutoAppPermissionFragment.java @@ -55,11 +55,11 @@ import androidx.preference.TwoStatePreference; import com.android.car.ui.AlertDialogBuilder; import com.android.permissioncontroller.R; import com.android.permissioncontroller.auto.AutoSettingsFrameFragment; -import com.android.permissioncontroller.permission.ui.AdvancedConfirmDialogArgs; import com.android.permissioncontroller.permission.ui.GrantPermissionsViewHandler; import com.android.permissioncontroller.permission.ui.model.AppPermissionViewModel; import com.android.permissioncontroller.permission.ui.model.AppPermissionViewModel.ChangeRequest; import com.android.permissioncontroller.permission.ui.model.AppPermissionViewModelFactory; +import com.android.permissioncontroller.permission.ui.v33.AdvancedConfirmDialogArgs; import com.android.permissioncontroller.permission.utils.KotlinUtils; import com.android.permissioncontroller.permission.utils.PackageRemovalMonitor; import com.android.settingslib.RestrictedLockUtils; diff --git a/PermissionController/src/com/android/permissioncontroller/permission/ui/handheld/AppPermissionFragment.java b/PermissionController/src/com/android/permissioncontroller/permission/ui/handheld/AppPermissionFragment.java index d9fcc6c53..a6ba3a82b 100644 --- a/PermissionController/src/com/android/permissioncontroller/permission/ui/handheld/AppPermissionFragment.java +++ b/PermissionController/src/com/android/permissioncontroller/permission/ui/handheld/AppPermissionFragment.java @@ -73,13 +73,13 @@ import androidx.lifecycle.ViewModelProvider; import com.android.permissioncontroller.R; import com.android.permissioncontroller.permission.data.FullStoragePermissionAppsLiveData.FullStoragePackageState; -import com.android.permissioncontroller.permission.ui.AdvancedConfirmDialogArgs; import com.android.permissioncontroller.permission.ui.GrantPermissionsViewHandler; import com.android.permissioncontroller.permission.ui.model.AppPermissionViewModel; import com.android.permissioncontroller.permission.ui.model.AppPermissionViewModel.ButtonState; import com.android.permissioncontroller.permission.ui.model.AppPermissionViewModel.ButtonType; import com.android.permissioncontroller.permission.ui.model.AppPermissionViewModel.ChangeRequest; import com.android.permissioncontroller.permission.ui.model.AppPermissionViewModelFactory; +import com.android.permissioncontroller.permission.ui.v33.AdvancedConfirmDialogArgs; import com.android.permissioncontroller.permission.utils.KotlinUtils; import com.android.permissioncontroller.permission.utils.Utils; import com.android.settingslib.RestrictedLockUtils; diff --git a/PermissionController/src/com/android/permissioncontroller/permission/ui/handheld/PermissionAppsFragment.java b/PermissionController/src/com/android/permissioncontroller/permission/ui/handheld/PermissionAppsFragment.java index 90d7204cf..220507426 100644 --- a/PermissionController/src/com/android/permissioncontroller/permission/ui/handheld/PermissionAppsFragment.java +++ b/PermissionController/src/com/android/permissioncontroller/permission/ui/handheld/PermissionAppsFragment.java @@ -53,6 +53,7 @@ import com.android.permissioncontroller.permission.model.v31.AppPermissionUsage; import com.android.permissioncontroller.permission.model.v31.PermissionUsages; import com.android.permissioncontroller.permission.ui.Category; import com.android.permissioncontroller.permission.ui.ManagePermissionsActivity; +import com.android.permissioncontroller.permission.ui.handheld.v31.CardViewPreference; import com.android.permissioncontroller.permission.ui.model.PermissionAppsViewModel; import com.android.permissioncontroller.permission.ui.model.PermissionAppsViewModelFactory; import com.android.permissioncontroller.permission.utils.KotlinUtils; diff --git a/PermissionController/src/com/android/permissioncontroller/permission/ui/handheld/CardViewPreference.java b/PermissionController/src/com/android/permissioncontroller/permission/ui/handheld/v31/CardViewPreference.java index 6c76d906b..008813a83 100644 --- a/PermissionController/src/com/android/permissioncontroller/permission/ui/handheld/CardViewPreference.java +++ b/PermissionController/src/com/android/permissioncontroller/permission/ui/handheld/v31/CardViewPreference.java @@ -14,13 +14,15 @@ * limitations under the License. */ -package com.android.permissioncontroller.permission.ui.handheld; +package com.android.permissioncontroller.permission.ui.handheld.v31; import android.content.Context; import android.content.Intent; +import android.os.Build; import android.view.View; import android.widget.Button; +import androidx.annotation.RequiresApi; import androidx.preference.Preference; import androidx.preference.PreferenceViewHolder; @@ -30,6 +32,7 @@ import com.android.permissioncontroller.R; /** * A Preference representing a banner message represented as a CardView */ +@RequiresApi(Build.VERSION_CODES.S) public class CardViewPreference extends Preference { private String mAction; diff --git a/PermissionController/src/com/android/permissioncontroller/permission/ui/model/AppPermissionViewModel.kt b/PermissionController/src/com/android/permissioncontroller/permission/ui/model/AppPermissionViewModel.kt index e13f4584c..7a0734b4b 100644 --- a/PermissionController/src/com/android/permissioncontroller/permission/ui/model/AppPermissionViewModel.kt +++ b/PermissionController/src/com/android/permissioncontroller/permission/ui/model/AppPermissionViewModel.kt @@ -62,7 +62,7 @@ import com.android.permissioncontroller.permission.model.livedatatypes.LightAppP import com.android.permissioncontroller.permission.model.livedatatypes.LightPermission import com.android.permissioncontroller.permission.service.PermissionChangeStorageImpl import com.android.permissioncontroller.permission.service.v33.PermissionDecisionStorageImpl -import com.android.permissioncontroller.permission.ui.AdvancedConfirmDialogArgs +import com.android.permissioncontroller.permission.ui.v33.AdvancedConfirmDialogArgs import com.android.permissioncontroller.permission.ui.model.AppPermissionViewModel.ButtonType.ALLOW import com.android.permissioncontroller.permission.ui.model.AppPermissionViewModel.ButtonType.ALLOW_ALWAYS import com.android.permissioncontroller.permission.ui.model.AppPermissionViewModel.ButtonType.ALLOW_FOREGROUND diff --git a/PermissionController/src/com/android/permissioncontroller/permission/ui/model/GrantPermissionsViewModel.kt b/PermissionController/src/com/android/permissioncontroller/permission/ui/model/GrantPermissionsViewModel.kt index b43074898..448284117 100644 --- a/PermissionController/src/com/android/permissioncontroller/permission/ui/model/GrantPermissionsViewModel.kt +++ b/PermissionController/src/com/android/permissioncontroller/permission/ui/model/GrantPermissionsViewModel.kt @@ -120,7 +120,7 @@ import com.android.permissioncontroller.permission.ui.ManagePermissionsActivity import com.android.permissioncontroller.permission.ui.ManagePermissionsActivity.EXTRA_RESULT_PERMISSION_INTERACTED import com.android.permissioncontroller.permission.ui.ManagePermissionsActivity.EXTRA_RESULT_PERMISSION_RESULT import com.android.permissioncontroller.permission.ui.v34.PermissionRationaleActivity -import com.android.permissioncontroller.permission.utils.AdminRestrictedPermissionsUtils +import com.android.permissioncontroller.permission.utils.v31.AdminRestrictedPermissionsUtils import com.android.permissioncontroller.permission.utils.KotlinUtils import com.android.permissioncontroller.permission.utils.KotlinUtils.getDefaultPrecision import com.android.permissioncontroller.permission.utils.KotlinUtils.grantBackgroundRuntimePermissions diff --git a/PermissionController/src/com/android/permissioncontroller/permission/ui/model/v31/PermissionUsageDetailsViewModel.kt b/PermissionController/src/com/android/permissioncontroller/permission/ui/model/v31/PermissionUsageDetailsViewModel.kt index 7a11883f8..c5164eb57 100644 --- a/PermissionController/src/com/android/permissioncontroller/permission/ui/model/v31/PermissionUsageDetailsViewModel.kt +++ b/PermissionController/src/com/android/permissioncontroller/permission/ui/model/v31/PermissionUsageDetailsViewModel.kt @@ -55,7 +55,7 @@ import com.android.permissioncontroller.permission.ui.handheld.v31.shouldShowSub import com.android.permissioncontroller.permission.utils.KotlinUtils import com.android.permissioncontroller.permission.utils.KotlinUtils.getPackageLabel import com.android.permissioncontroller.permission.utils.PermissionMapping -import com.android.permissioncontroller.permission.utils.SubattributionUtils +import com.android.permissioncontroller.permission.utils.v31.SubattributionUtils import com.android.permissioncontroller.permission.utils.Utils import java.time.Instant import java.util.Objects diff --git a/PermissionController/src/com/android/permissioncontroller/permission/ui/model/v31/PermissionUsageDetailsViewModelLegacy.kt b/PermissionController/src/com/android/permissioncontroller/permission/ui/model/v31/PermissionUsageDetailsViewModelLegacy.kt index 667667200..84283a50e 100644 --- a/PermissionController/src/com/android/permissioncontroller/permission/ui/model/v31/PermissionUsageDetailsViewModelLegacy.kt +++ b/PermissionController/src/com/android/permissioncontroller/permission/ui/model/v31/PermissionUsageDetailsViewModelLegacy.kt @@ -44,7 +44,7 @@ import com.android.permissioncontroller.permission.utils.KotlinUtils import com.android.permissioncontroller.permission.utils.KotlinUtils.getPackageLabel import com.android.permissioncontroller.permission.utils.PermissionMapping import com.android.permissioncontroller.permission.utils.StringUtils -import com.android.permissioncontroller.permission.utils.SubattributionUtils +import com.android.permissioncontroller.permission.utils.v31.SubattributionUtils import com.android.permissioncontroller.permission.utils.Utils import java.time.Instant import java.util.concurrent.TimeUnit diff --git a/PermissionController/src/com/android/permissioncontroller/permission/ui/television/AppPermissionFragment.java b/PermissionController/src/com/android/permissioncontroller/permission/ui/television/AppPermissionFragment.java index a5961dcec..e2df47009 100644 --- a/PermissionController/src/com/android/permissioncontroller/permission/ui/television/AppPermissionFragment.java +++ b/PermissionController/src/com/android/permissioncontroller/permission/ui/television/AppPermissionFragment.java @@ -66,13 +66,13 @@ import com.android.permissioncontroller.R; import com.android.permissioncontroller.permission.data.FullStoragePermissionAppsLiveData.FullStoragePackageState; import com.android.permissioncontroller.permission.model.AppPermissionGroup; import com.android.permissioncontroller.permission.model.AppPermissions; -import com.android.permissioncontroller.permission.ui.AdvancedConfirmDialogArgs; import com.android.permissioncontroller.permission.ui.GrantPermissionsViewHandler; import com.android.permissioncontroller.permission.ui.model.AppPermissionViewModel; import com.android.permissioncontroller.permission.ui.model.AppPermissionViewModel.ButtonState; import com.android.permissioncontroller.permission.ui.model.AppPermissionViewModel.ButtonType; import com.android.permissioncontroller.permission.ui.model.AppPermissionViewModel.ChangeRequest; import com.android.permissioncontroller.permission.ui.model.AppPermissionViewModelFactory; +import com.android.permissioncontroller.permission.ui.v33.AdvancedConfirmDialogArgs; import com.android.permissioncontroller.permission.utils.KotlinUtils; import com.android.permissioncontroller.permission.utils.Utils; diff --git a/PermissionController/src/com/android/permissioncontroller/permission/ui/AdvancedConfirmDialogArgs.kt b/PermissionController/src/com/android/permissioncontroller/permission/ui/v33/AdvancedConfirmDialogArgs.kt index 6b26c7e47..b841f3aeb 100644 --- a/PermissionController/src/com/android/permissioncontroller/permission/ui/AdvancedConfirmDialogArgs.kt +++ b/PermissionController/src/com/android/permissioncontroller/permission/ui/v33/AdvancedConfirmDialogArgs.kt @@ -1,28 +1,27 @@ /* * Copyright (C) 2022 The Android Open Source Project * - * Licensed under the Apache License; - * Version 2.0 (the "License"); + * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. * You may obtain a copy of the License at * * http://www.apache.org/licenses/LICENSE-2.0 * - * Unless required by applicable law or agreed to in writing; - * software - * distributed under the License is distributed on an "AS IS" BASIS; - * - * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND; - * either express or implied. + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. * See the License for the specific language governing permissions and * limitations under the License. */ -package com.android.permissioncontroller.permission.ui +package com.android.permissioncontroller.permission.ui.v33 +import android.os.Build import androidx.annotation.DrawableRes +import androidx.annotation.RequiresApi import androidx.annotation.StringRes import com.android.permissioncontroller.permission.ui.model.AppPermissionViewModel +@RequiresApi(Build.VERSION_CODES.TIRAMISU) data class AdvancedConfirmDialogArgs( @DrawableRes val iconId: Int = 0, @StringRes val titleId: Int = 0, diff --git a/PermissionController/src/com/android/permissioncontroller/permission/utils/AdminRestrictedPermissionsUtils.java b/PermissionController/src/com/android/permissioncontroller/permission/utils/v31/AdminRestrictedPermissionsUtils.java index 4944093ac..e9d68c9c6 100644 --- a/PermissionController/src/com/android/permissioncontroller/permission/utils/AdminRestrictedPermissionsUtils.java +++ b/PermissionController/src/com/android/permissioncontroller/permission/utils/v31/AdminRestrictedPermissionsUtils.java @@ -14,7 +14,7 @@ * limitations under the License. */ -package com.android.permissioncontroller.permission.utils; +package com.android.permissioncontroller.permission.utils.v31; import android.Manifest; import android.app.admin.DevicePolicyManager; @@ -24,6 +24,7 @@ import android.os.UserManager; import android.util.ArraySet; import com.android.modules.utils.build.SdkLevel; +import com.android.permissioncontroller.permission.utils.Utils; /** * A class for dealing with permissions that the admin may not grant in certain configurations. diff --git a/PermissionController/src/com/android/permissioncontroller/permission/utils/SubattributionUtils.java b/PermissionController/src/com/android/permissioncontroller/permission/utils/v31/SubattributionUtils.java index 2785eca74..8919953b4 100644 --- a/PermissionController/src/com/android/permissioncontroller/permission/utils/SubattributionUtils.java +++ b/PermissionController/src/com/android/permissioncontroller/permission/utils/v31/SubattributionUtils.java @@ -14,9 +14,8 @@ * limitations under the License. */ -package com.android.permissioncontroller.permission.utils; +package com.android.permissioncontroller.permission.utils.v31; -import android.annotation.SuppressLint; import android.content.Context; import android.content.pm.ApplicationInfo; import android.content.pm.Attribution; @@ -25,6 +24,7 @@ import android.content.pm.PackageManager; import android.content.res.Resources; import android.os.Build; +import androidx.annotation.ChecksSdkIntAtLeast; import androidx.annotation.Nullable; import androidx.annotation.RequiresApi; @@ -42,6 +42,7 @@ public class SubattributionUtils { /** * Returns true if the app supports subattribution. */ + @ChecksSdkIntAtLeast(api = Build.VERSION_CODES.S) public static boolean isSubattributionSupported(Context context, ApplicationInfo appInfo) { if (!SdkLevel.isAtLeastS()) { return false; @@ -50,6 +51,7 @@ public class SubattributionUtils { } /** Returns whether the provided package supports subattribution. */ + @ChecksSdkIntAtLeast(api = Build.VERSION_CODES.S) public static boolean isSubattributionSupported(LightPackageInfo lightPackageInfo) { return SdkLevel.isAtLeastS() && lightPackageInfo.getAreAttributionsUserVisible(); } @@ -59,7 +61,6 @@ public class SubattributionUtils { * {@code null} otherwise. */ @Nullable - @SuppressLint("NewApi") // isSubattributionSupported checks api level public static Map<Integer, String> getAttributionLabels(Context context, PackageInfo pkgInfo) { if (!isSubattributionSupported(context, pkgInfo.applicationInfo)) { return null; @@ -72,7 +73,6 @@ public class SubattributionUtils { * {@code null} otherwise. */ @Nullable - @SuppressLint("NewApi") // isSubattributionSupported checks api level public static Map<Integer, String> getAttributionLabels(Context context, ApplicationInfo appInfo) { if (!isSubattributionSupported(context, appInfo)) { @@ -114,7 +114,6 @@ public class SubattributionUtils { /** Returns the attribution label map for the package if the app supports subattribution. */ @Nullable - @SuppressLint("NewApi") // isSubattributionSupported checks api level public static Map<Integer, String> getAttributionLabels(Context context, LightPackageInfo lightPackageInfo) { if (!isSubattributionSupported(lightPackageInfo)) { |