diff options
| author | 2024-02-20 14:46:26 -0800 | |
|---|---|---|
| committer | 2024-02-20 22:54:22 +0000 | |
| commit | 22364c0130c4c979bb207abb7ffe0deaa6b9a5b7 (patch) | |
| tree | f064ca1bc615d731ea8b114c8c2e955170464c5d | |
| parent | cb27fa5a3c50283eaab05e5dbca790278ceb0388 (diff) | |
[ECM] Rename getPkgName to getPackageName
In SignedPackage and SignedPackageParcel, rename the getPkgName() method
and pkgName field to getPackageName() and packageName.
This is an API change. Fortunately, there is only one consumer of
this API: EnhancedConfirmationService. (Both currently and planned
for future.)
Fix: 325048954
Test: atest FrameworksServicesTests:com.android.server.systemconfig.SystemConfigTest
Change-Id: I3961cadf7e42ce1641f3c89ddaf818225f319390
4 files changed, 13 insertions, 13 deletions
diff --git a/core/api/module-lib-current.txt b/core/api/module-lib-current.txt index 7c4df280eaa7..9c1a8e854e92 100644 --- a/core/api/module-lib-current.txt +++ b/core/api/module-lib-current.txt @@ -135,7 +135,7 @@ package android.content.pm { @FlaggedApi("android.permission.flags.enhanced_confirmation_mode_apis_enabled") public class SignedPackage { method @NonNull public byte[] getCertificateDigest(); - method @NonNull public String getPkgName(); + method @NonNull public String getPackageName(); } } diff --git a/core/java/android/content/pm/SignedPackage.java b/core/java/android/content/pm/SignedPackage.java index 4d1b136915f2..7bffa7722142 100644 --- a/core/java/android/content/pm/SignedPackage.java +++ b/core/java/android/content/pm/SignedPackage.java @@ -35,9 +35,9 @@ public class SignedPackage { private final SignedPackageParcel mData; /** @hide */ - public SignedPackage(@NonNull String pkgName, @NonNull byte[] certificateDigest) { + public SignedPackage(@NonNull String packageName, @NonNull byte[] certificateDigest) { SignedPackageParcel data = new SignedPackageParcel(); - data.pkgName = pkgName; + data.packageName = packageName; data.certificateDigest = certificateDigest; mData = data; } @@ -52,8 +52,8 @@ public class SignedPackage { return mData; } - public @NonNull String getPkgName() { - return mData.pkgName; + public @NonNull String getPackageName() { + return mData.packageName; } public @NonNull byte[] getCertificateDigest() { @@ -64,12 +64,12 @@ public class SignedPackage { public boolean equals(Object o) { if (this == o) return true; if (!(o instanceof SignedPackage that)) return false; - return mData.pkgName.equals(that.mData.pkgName) && Arrays.equals(mData.certificateDigest, - that.mData.certificateDigest); + return mData.packageName.equals(that.mData.packageName) && Arrays.equals( + mData.certificateDigest, that.mData.certificateDigest); } @Override public int hashCode() { - return Objects.hash(mData.pkgName, Arrays.hashCode(mData.certificateDigest)); + return Objects.hash(mData.packageName, Arrays.hashCode(mData.certificateDigest)); } } diff --git a/core/java/android/content/pm/SignedPackageParcel.aidl b/core/java/android/content/pm/SignedPackageParcel.aidl index 7957f7f99d4a..bb4dc8654108 100644 --- a/core/java/android/content/pm/SignedPackageParcel.aidl +++ b/core/java/android/content/pm/SignedPackageParcel.aidl @@ -20,6 +20,6 @@ import android.content.ComponentName; /** @hide */ parcelable SignedPackageParcel { - String pkgName; + String packageName; byte[] certificateDigest; } diff --git a/services/tests/servicestests/src/com/android/server/systemconfig/SystemConfigTest.java b/services/tests/servicestests/src/com/android/server/systemconfig/SystemConfigTest.java index eddff9abec21..f2fc6b16fd17 100644 --- a/services/tests/servicestests/src/com/android/server/systemconfig/SystemConfigTest.java +++ b/services/tests/servicestests/src/com/android/server/systemconfig/SystemConfigTest.java @@ -712,7 +712,7 @@ public class SystemConfigTest { android.permission.flags.Flags.FLAG_ENHANCED_CONFIRMATION_MODE_APIS_ENABLED) public void getEnhancedConfirmationTrustedInstallers_returnsTrustedInstallers() throws IOException { - String pkgName = "com.example.app"; + String packageName = "com.example.app"; String certificateDigestStr = "E9:7A:BC:2C:D1:CA:8D:58:6A:57:0B:8C:F8:60:AA:D2:" + "8D:13:30:2A:FB:C9:00:2C:5D:53:B2:6C:09:A4:85:A0"; @@ -720,7 +720,7 @@ public class SystemConfigTest { .toByteArray(); String contents = "<config>" + "<" + "enhanced-confirmation-trusted-installer" + " " - + "package=\"" + pkgName + "\"" + + "package=\"" + packageName + "\"" + " sha256-cert-digest=\"" + certificateDigestStr + "\"" + "/>" + "</config>"; @@ -734,10 +734,10 @@ public class SystemConfigTest { assertThat(actualTrustedInstallers.size()).isEqualTo(1); SignedPackage actual = actualTrustedInstallers.stream().findFirst().orElseThrow(); - SignedPackage expected = new SignedPackage(pkgName, certificateDigest); + SignedPackage expected = new SignedPackage(packageName, certificateDigest); assertThat(actual.getCertificateDigest()).isEqualTo(expected.getCertificateDigest()); - assertThat(actual.getPkgName()).isEqualTo(expected.getPkgName()); + assertThat(actual.getPackageName()).isEqualTo(expected.getPackageName()); assertThat(actual).isEqualTo(expected); } |