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

Merge "Notify IORap without including packages for secondary dex opt." into rvc-dev am: eddf0c94d2

Browse Source 
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/12280079

Change-Id: I516152d64468fd248b2a8d5dc5127567f538b8b3
This commit is contained in:
Calin Juravle
2020-08-06 19:23:07 +00:00
committed by Automerger Merge Worker
parent 4820e9d4e4 eddf0c94d2
commit 724d8f1740
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
services/core/java/com/android/server/pm/BackgroundDexOptService.java
View File

@@ -337,6 +337,7 @@ public class BackgroundDexOptService extends JobService {
private int idleOptimizePackages(PackageManagerService pm, ArraySet<String> pkgs, private int idleOptimizePackages(PackageManagerService pm, ArraySet<String> pkgs,
long lowStorageThreshold) { long lowStorageThreshold) {
ArraySet<String> updatedPackages = new ArraySet<>(); ArraySet<String> updatedPackages = new ArraySet<>();
ArraySet<String> updatedPackagesDueToSecondaryDex = new ArraySet<>();
try { try {
final boolean supportSecondaryDex = supportSecondaryDex(); final boolean supportSecondaryDex = supportSecondaryDex();
@@ -391,11 +392,14 @@ public class BackgroundDexOptService extends JobService {
} }
int secondaryResult = optimizePackages(pm, pkgs, lowStorageThreshold, int secondaryResult = optimizePackages(pm, pkgs, lowStorageThreshold,
/*isForPrimaryDex*/ false, updatedPackages); /*isForPrimaryDex*/ false, updatedPackagesDueToSecondaryDex);
return secondaryResult; return secondaryResult;
} finally { } finally {
// Always let the pinner service know about changes. // Always let the pinner service know about changes.
notifyPinService(updatedPackages); notifyPinService(updatedPackages);
// Only notify IORap the primary dex opt, because we don't want to
// invalidate traces unnecessary due to b/161633001 and that it's
// better to have a trace than no trace at all.
notifyPackagesUpdated(updatedPackages); notifyPackagesUpdated(updatedPackages);
} }
} }