Merge "Do not update RM config with previous sequence" into sc-dev am: 9b1891b428

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

Change-Id: Id2d6ce9139d214fb172b7fe789b11367ef201fb0

core/java/android/app/ActivityThread.java

@@ -1171,7 +1171,7 @@ public final class ActivityThread extends ClientTransactionHandler
         }
 
         public void scheduleApplicationInfoChanged(ApplicationInfo ai) {
-            mResourcesManager.updatePendingAppInfoUpdates(ai);
+            mResourcesManager.appendPendingAppInfoUpdate(new String[]{ai.sourceDir}, ai);
             mH.removeMessages(H.APPLICATION_INFO_CHANGED, ai);
             sendMessage(H.APPLICATION_INFO_CHANGED, ai);
         }
@@ -5975,16 +5975,12 @@ public final class ActivityThread extends ClientTransactionHandler
             resApk = ref != null ? ref.get() : null;
         }
 
-        final String[] oldResDirs = new String[2];
-
         if (apk != null) {
-            oldResDirs[0] = apk.getResDir();
             final ArrayList<String> oldPaths = new ArrayList<>();
             LoadedApk.makePaths(this, apk.getApplicationInfo(), oldPaths);
             apk.updateApplicationInfo(ai, oldPaths);
         }
         if (resApk != null) {
-            oldResDirs[1] = resApk.getResDir();
             final ArrayList<String> oldPaths = new ArrayList<>();
             LoadedApk.makePaths(this, resApk.getApplicationInfo(), oldPaths);
             resApk.updateApplicationInfo(ai, oldPaths);
@@ -5992,7 +5988,7 @@ public final class ActivityThread extends ClientTransactionHandler
 
         synchronized (mResourcesManager) {
             // Update all affected Resources objects to use new ResourcesImpl
-            mResourcesManager.applyNewResourceDirs(ai, oldResDirs);
+            mResourcesManager.applyAllPendingAppInfoUpdates();
         }
     }
 
@@ -6241,7 +6237,9 @@ public final class ActivityThread extends ClientTransactionHandler
 
                                 synchronized (mResourcesManager) {
                                     // Update affected Resources objects to use new ResourcesImpl
-                                    mResourcesManager.applyNewResourceDirs(aInfo, oldResDirs);
+                                    mResourcesManager.appendPendingAppInfoUpdate(oldResDirs,
+                                            aInfo);
+                                    mResourcesManager.applyAllPendingAppInfoUpdates();
                                 }
                             } catch (RemoteException e) {
                             }

core/java/android/app/ResourcesManager.java

@@ -42,6 +42,7 @@ import android.util.ArrayMap;
 import android.util.ArraySet;
 import android.util.DisplayMetrics;
 import android.util.Log;
+import android.util.Pair;
 import android.util.Slog;
 import android.view.Display;
 import android.view.DisplayAdjustments;
@@ -101,7 +102,7 @@ public class ResourcesManager {
      * ApplicationInfo changes that need to be applied to Resources when the next configuration
      * change occurs.
      */
-    private ArrayList<ApplicationInfo> mPendingAppInfoUpdates;
+    private ArrayList<Pair<String[], ApplicationInfo>> mPendingAppInfoUpdates;
 
     /**
      * A mapping of ResourceImpls and their configurations. These are heavy weight objects
@@ -1273,19 +1274,33 @@ public class ResourcesManager {
         return newKey;
     }
 
-    public void updatePendingAppInfoUpdates(@NonNull ApplicationInfo appInfo) {
+    public void appendPendingAppInfoUpdate(@NonNull String[] oldSourceDirs,
+            @NonNull ApplicationInfo appInfo) {
         synchronized (mLock) {
             if (mPendingAppInfoUpdates == null) {
                 mPendingAppInfoUpdates = new ArrayList<>();
             }
-            // Clear previous app info changes for the package to prevent multiple ResourcesImpl
+            // Clear previous app info changes for a package to prevent multiple ResourcesImpl
-            // recreations when only the last recreation will be used.
+            // recreations when the recreation caused by this update completely overrides the
+            // previous pending changes.
             for (int i = mPendingAppInfoUpdates.size() - 1; i >= 0; i--) {
-                if (appInfo.sourceDir.equals(mPendingAppInfoUpdates.get(i).sourceDir)) {
+                if (ArrayUtils.containsAll(oldSourceDirs, mPendingAppInfoUpdates.get(i).first)) {
                     mPendingAppInfoUpdates.remove(i);
                 }
             }
-            mPendingAppInfoUpdates.add(appInfo);
+            mPendingAppInfoUpdates.add(new Pair<>(oldSourceDirs, appInfo));
+        }
+    }
+
+    public final void applyAllPendingAppInfoUpdates() {
+        synchronized (mLock) {
+            if (mPendingAppInfoUpdates != null) {
+                for (int i = 0, n = mPendingAppInfoUpdates.size(); i < n; i++) {
+                    final Pair<String[], ApplicationInfo> appInfo = mPendingAppInfoUpdates.get(i);
+                    applyNewResourceDirsLocked(appInfo.first, appInfo.second);
+                }
+                mPendingAppInfoUpdates = null;
+            }
         }
     }
 
@@ -1302,18 +1317,7 @@ public class ResourcesManager {
                 Trace.traceBegin(Trace.TRACE_TAG_RESOURCES,
                         "ResourcesManager#applyConfigurationToResources");
 
-                final boolean assetsUpdated = mPendingAppInfoUpdates != null
+                if (!mResConfiguration.isOtherSeqNewer(config) && compat == null) {
-                        && config.assetsSeq > mResConfiguration.assetsSeq;
-                if (assetsUpdated) {
-                    for (int i = 0, n = mPendingAppInfoUpdates.size(); i < n; i++) {
-                        final ApplicationInfo appInfo = mPendingAppInfoUpdates.get(i);
-                        applyNewResourceDirs(appInfo, new String[]{appInfo.sourceDir});
-                    }
-                    mPendingAppInfoUpdates = null;
-                }
-
-                if (!assetsUpdated && !mResConfiguration.isOtherSeqNewer(config)
-                        && compat == null) {
                     if (DEBUG || DEBUG_CONFIGURATION) {
                         Slog.v(TAG, "Skipping new config: curSeq="
                                 + mResConfiguration.seq + ", newSeq=" + config.seq);
@@ -1330,6 +1334,13 @@ public class ResourcesManager {
                             | ActivityInfo.CONFIG_SMALLEST_SCREEN_SIZE;
                 }
 
+                // If a application info update was scheduled to occur in this process but has not
+                // occurred yet, apply it now so the resources objects will have updated paths when
+                // the assets sequence changes.
+                if ((changes & ActivityInfo.CONFIG_ASSETS_PATHS) != 0) {
+                    applyAllPendingAppInfoUpdates();
+                }
+
                 DisplayMetrics displayMetrics = getDisplayMetrics();
                 if (adjustments != null) {
                     // Currently the only case where the adjustment takes effect is to simulate
@@ -1353,7 +1364,7 @@ public class ResourcesManager {
                     }
                 }
 
-                return assetsUpdated || changes != 0;
+                return changes != 0;
             } finally {
                 Trace.traceEnd(Trace.TRACE_TAG_RESOURCES);
             }
@@ -1440,10 +1451,8 @@ public class ResourcesManager {
         }
     }
 
-    // TODO(adamlesinski): Make this accept more than just overlay directories.
+    private void applyNewResourceDirsLocked(@Nullable final String[] oldSourceDirs,
-    void applyNewResourceDirs(@NonNull final ApplicationInfo appInfo,
+            @NonNull final ApplicationInfo appInfo) {
-            @Nullable final String[] oldPaths) {
-        synchronized (mLock) {
         try {
             Trace.traceBegin(Trace.TRACE_TAG_RESOURCES,
                     "ResourcesManager#applyNewResourceDirsLocked");
@@ -1477,7 +1486,7 @@ public class ResourcesManager {
 
                 if (key.mResDir == null
                         || key.mResDir.equals(baseCodePath)
-                            || ArrayUtils.contains(oldPaths, key.mResDir)) {
+                        || ArrayUtils.contains(oldSourceDirs, key.mResDir)) {
                     updatedResourceKeys.put(impl, new ResourcesKey(
                             baseCodePath,
                             copiedSplitDirs,
@@ -1496,7 +1505,6 @@ public class ResourcesManager {
             Trace.traceEnd(Trace.TRACE_TAG_RESOURCES);
         }
     }
-    }
 
     /**
      * Creates an array with the contents of {@param overlayPaths} and the unique elements of