Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Revert "Always apply compat changes to system apps."" am: 612c2c731d

Browse Source 
am: a105206a92

Change-Id: Ie52fbad2208d57872bfe9aaa92831e2c1375772e
This commit is contained in:
Anna Trostanetski
2019-08-23 03:29:31 -07:00
committed by android-build-merger
parent 4088527e0c a105206a92
commit 9b42b7095d
2 changed files with 0 additions and 34 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
services/core/java/com/android/server/compat/CompatChange.java
View File

@@ -118,12 +118,6 @@ public final class CompatChange {
* @return {@code true} if the change should be enabled for the package. * @return {@code true} if the change should be enabled for the package.
*/ */
boolean isEnabled(ApplicationInfo app) { boolean isEnabled(ApplicationInfo app) {
if (app.isSystemApp()) {
// All changes are enabled for system apps, and we do not support overrides.
// Compatibility issues for system apps should be addressed in the app itself when
// the compatibility change is made.
return true;
}
if (mPackageOverrides != null && mPackageOverrides.containsKey(app.packageName)) { if (mPackageOverrides != null && mPackageOverrides.containsKey(app.packageName)) {
return mPackageOverrides.get(app.packageName); return mPackageOverrides.get(app.packageName);
} }

28
services/tests/servicestests/src/com/android/server/compat/CompatConfigTest.java
View File

@@ -172,34 +172,6 @@ public class CompatConfigTest {
assertThat(pc.lookupChangeId("MY_CHANGE")).isEqualTo(-1L); assertThat(pc.lookupChangeId("MY_CHANGE")).isEqualTo(-1L);
} }
@Test
public void testSystemAppDisabledChangeEnabled() {
CompatConfig pc = new CompatConfig();
pc.addChange(new CompatChange(1234L, "MY_CHANGE", -1, true)); // disabled
ApplicationInfo sysApp = makeAppInfo("system.app", 1);
sysApp.flags |= ApplicationInfo.FLAG_SYSTEM;
assertThat(pc.isChangeEnabled(1234L, sysApp)).isTrue();
}
@Test
public void testSystemAppOverrideIgnored() {
CompatConfig pc = new CompatConfig();
pc.addChange(new CompatChange(1234L, "MY_CHANGE", -1, false));
pc.addOverride(1234L, "system.app", false);
ApplicationInfo sysApp = makeAppInfo("system.app", 1);
sysApp.flags |= ApplicationInfo.FLAG_SYSTEM;
assertThat(pc.isChangeEnabled(1234L, sysApp)).isTrue();
}
@Test
public void testSystemAppTargetSdkIgnored() {
CompatConfig pc = new CompatConfig();
pc.addChange(new CompatChange(1234L, "MY_CHANGE", 2, false));
ApplicationInfo sysApp = makeAppInfo("system.app", 1);
sysApp.flags |= ApplicationInfo.FLAG_SYSTEM;
assertThat(pc.isChangeEnabled(1234L, sysApp)).isTrue();
}
@Test @Test
public void testReadConfig() { public void testReadConfig() {
Change[] changes = {new Change(1234L, "MY_CHANGE1", false, 2), new Change(1235L, Change[] changes = {new Change(1234L, "MY_CHANGE1", false, 2), new Change(1235L,