Snap for 5704971 from 660c234e9ad68cd23feb2e22cb43bd7a645b24e6 to qt-release (e928d0b1) · Commits · e / os / android_frameworks_base

core/java/android/content/pm/RegisteredServicesCache.java

+20 −39

Original line number	Diff line number	Diff line
		@@ -17,7 +17,6 @@
		package android.content.pm;

		import android.Manifest;
		import android.annotation.Nullable;
		import android.annotation.UnsupportedAppUsage;
		import android.content.BroadcastReceiver;
		import android.content.ComponentName;
		@@ -177,8 +176,7 @@ public abstract class RegisteredServicesCache<V> {
		mContext.registerReceiver(mUserRemovedReceiver, userFilter);
		}

		@VisibleForTesting
		protected void handlePackageEvent(Intent intent, int userId) {
		private void handlePackageEvent(Intent intent, int userId) {
		// Don't regenerate the services map when the package is removed or its
		// ASEC container unmounted as a step in replacement. The subsequent
		// _ADDED / _AVAILABLE call will regenerate the map in the final state.
		@@ -240,9 +238,6 @@ public abstract class RegisteredServicesCache<V> {

		public void invalidateCache(int userId) {
		synchronized (mServicesLock) {
		if (DEBUG) {
		Slog.d(TAG, "invalidating cache for " + userId + " " + mInterfaceName);
		}
		final UserServices<V> user = findOrCreateUserLocked(userId);
		user.services = null;
		onServicesChangedLocked(userId);
		@@ -472,37 +467,16 @@ public abstract class RegisteredServicesCache<V> {
		* or null to assume that everything is affected.
		* @param userId the user for whom to update the services map.
		*/
		private void generateServicesMap(@Nullable int[] changedUids, int userId) {
		private void generateServicesMap(int[] changedUids, int userId) {
		if (DEBUG) {
		Slog.d(TAG, "generateServicesMap() for " + userId + ", changed UIDs = "
		+ Arrays.toString(changedUids));
		}

		synchronized (mServicesLock) {
		final UserServices<V> user = findOrCreateUserLocked(userId);
		final boolean cacheInvalid = user.services == null;
		if (cacheInvalid) {
		user.services = Maps.newHashMap();
		}

		final ArrayList<ServiceInfo<V>> serviceInfos = new ArrayList<>();
		final List<ResolveInfo> resolveInfos = queryIntentServices(userId);

		for (ResolveInfo resolveInfo : resolveInfos) {
		try {
		// when changedUids == null, we want to do a rescan of everything, this means
		// it's the initial scan, and containsUid will trivially return true
		// when changedUids != null, we got here because a package changed, but
		// invalidateCache could have been called (thus user.services == null), and we
		// should query from PackageManager again
		if (!cacheInvalid
		&& !containsUid(
		changedUids, resolveInfo.serviceInfo.applicationInfo.uid)) {
		if (DEBUG) {
		Slog.d(TAG, "Skipping parseServiceInfo for " + resolveInfo);
		}
		continue;
		}
		ServiceInfo<V> info = parseServiceInfo(resolveInfo);
		if (info == null) {
		Log.w(TAG, "Unable to load service info " + resolveInfo.toString());
		@@ -514,6 +488,13 @@ public abstract class RegisteredServicesCache<V> {
		}
		}

		synchronized (mServicesLock) {
		final UserServices<V> user = findOrCreateUserLocked(userId);
		final boolean firstScan = user.services == null;
		if (firstScan) {
		user.services = Maps.newHashMap();
		}

		StringBuilder changes = new StringBuilder();
		boolean changed = false;
		for (ServiceInfo<V> info : serviceInfos) {
		@@ -534,7 +515,7 @@ public abstract class RegisteredServicesCache<V> {
		changed = true;
		user.services.put(info.type, info);
		user.persistentServices.put(info.type, info.uid);
		if (!(user.mPersistentServicesFileDidNotExist && cacheInvalid)) {
		if (!(user.mPersistentServicesFileDidNotExist && firstScan)) {
		notifyListener(info.type, userId, false /* removed */);
		}
		} else if (previousUid == info.uid) {

core/java/com/android/internal/app/ChooserActivity.java

+8 −3

Original line number	Diff line number	Diff line
		@@ -1534,7 +1534,11 @@ public class ChooserActivity extends ResolverActivity {
		if (driList.get(i).getResolvedComponentName().equals(
		resultList.get(j).getTargetComponent())) {
		ShortcutManager.ShareShortcutInfo shareShortcutInfo = resultList.get(j);
		ChooserTarget chooserTarget = convertToChooserTarget(shareShortcutInfo);
		// Incoming results are ordered but without a score. Create a score
		// based on the index in order to be sorted appropriately when joined
		// with legacy direct share api results.
		float score = Math.max(1.0f - (0.05f * j), 0.0f);
		ChooserTarget chooserTarget = convertToChooserTarget(shareShortcutInfo, score);
		chooserTargets.add(chooserTarget);
		if (mDirectShareAppTargetCache != null && appTargets != null) {
		mDirectShareAppTargetCache.put(chooserTarget, appTargets.get(j));
		@@ -1580,7 +1584,8 @@ public class ChooserActivity extends ResolverActivity {
		return false;
		}

		private ChooserTarget convertToChooserTarget(ShortcutManager.ShareShortcutInfo shareShortcut) {
		private ChooserTarget convertToChooserTarget(ShortcutManager.ShareShortcutInfo shareShortcut,
		float score) {
		ShortcutInfo shortcutInfo = shareShortcut.getShortcutInfo();
		Bundle extras = new Bundle();
		extras.putString(Intent.EXTRA_SHORTCUT_ID, shortcutInfo.getId());
		@@ -1591,7 +1596,7 @@ public class ChooserActivity extends ResolverActivity {
		null,
		// The ranking score for this target (0.0-1.0); the system will omit items with low
		// scores when there are too many Direct Share items.
		1.0f,
		score,
		// The name of the component to be launched if this target is chosen.
		shareShortcut.getTargetComponent().clone(),
		// The extra values here will be merged into the Intent when this target is chosen.

core/java/com/android/internal/os/ZygoteInit.java

+7 −0

Original line number	Diff line number	Diff line
		@@ -188,6 +188,13 @@ public class ZygoteInit {
		System.loadLibrary("android");
		System.loadLibrary("compiler_rt");
		System.loadLibrary("jnigraphics");

		// tolerate missing sfplugin_ccodec which is only present on Codec 2 devices
		try {
		System.loadLibrary("sfplugin_ccodec");
		} catch (Error \| RuntimeException e) {
		Log.w(TAG, "Problem preloading sfplugin_ccodec: " + e);
		}
		}

		native private static void nativePreloadAppProcessHALs();

core/java/com/android/internal/policy/DecorView.java

+1 −1

Original line number	Diff line number	Diff line
		@@ -1311,7 +1311,7 @@ public class DecorView extends FrameLayout implements RootViewSurfaceTaker, Wind
		return semiTransparentBarColor;
		} else if ((flags & FLAG_DRAWS_SYSTEM_BAR_BACKGROUNDS) == 0) {
		return Color.BLACK;
		} else if (scrimTransparent && barColor == Color.TRANSPARENT) {
		} else if (scrimTransparent && Color.alpha(barColor) == 0) {
		boolean light = (sysuiVis & lightSysuiFlag) != 0;
		return light ? SCRIM_LIGHT : semiTransparentBarColor;
		} else {

core/tests/coretests/src/android/content/pm/RegisteredServicesCacheTest.java

+0 −41

Original line number	Diff line number	Diff line
		@@ -16,7 +16,6 @@

		package android.content.pm;

		import android.content.Intent;
		import android.content.res.Resources;
		import android.os.FileUtils;
		import android.os.Parcel;
		@@ -190,36 +189,6 @@ public class RegisteredServicesCacheTest extends AndroidTestCase {
		assertEquals(0, cache.getPersistentServicesSize(u1));
		}

		/**
		* Check that an optimization to skip a call to PackageManager handles an invalidated cache.
		*
		* We added an optimization in generateServicesMap to only query PackageManager for packages
		* that have been changed, because if a package is unchanged, we have already cached the
		* services info for it, so we can save a query to PackageManager (and save some memory).
		* However, if invalidateCache was called, we cannot optimize, and must do a full query.
		* The initial optimization was buggy because it failed to check for an invalidated cache, and
		* only scanned the changed packages, given in the ACTION_PACKAGE_CHANGED intent (b/122912184).
		*/
		public void testParseServiceInfoOptimizationHandlesInvalidatedCache() {
		TestServicesCache cache = new TestServicesCache();
		cache.addServiceForQuerying(U0, r1, newServiceInfo(t1, UID1));
		cache.addServiceForQuerying(U0, r2, newServiceInfo(t2, UID2));
		assertEquals(2, cache.getAllServicesSize(U0));

		// simulate the client of the cache invalidating it
		cache.invalidateCache(U0);

		// there should be 0 services (userServices.services == null ) at this point, but we don't
		// call getAllServicesSize since that would force a full scan of packages,
		// instead we trigger a package change in a package that is in the list of services
		Intent intent = new Intent(Intent.ACTION_PACKAGE_CHANGED);
		intent.putExtra(Intent.EXTRA_UID, UID1);
		cache.handlePackageEvent(intent, U0);

		// check that the optimization does a full query and caches both services
		assertEquals(2, cache.getAllServicesSize(U0));
		}

		private static RegisteredServicesCache.ServiceInfo<TestServiceType> newServiceInfo(
		TestServiceType type, int uid) {
		final ComponentInfo info = new ComponentInfo();
		@@ -297,11 +266,6 @@ public class RegisteredServicesCacheTest extends AndroidTestCase {
		map = new HashMap<>();
		mServices.put(userId, map);
		}
		// in actual cases, resolveInfo should always have a serviceInfo, since we specifically
		// query for intent services
		resolveInfo.serviceInfo = new android.content.pm.ServiceInfo();
		resolveInfo.serviceInfo.applicationInfo =
		new ApplicationInfo(serviceInfo.componentInfo.applicationInfo);
		map.put(resolveInfo, serviceInfo);
		}

		@@ -340,11 +304,6 @@ public class RegisteredServicesCacheTest extends AndroidTestCase {
		public void onUserRemoved(int userId) {
		super.onUserRemoved(userId);
		}

		@Override
		public void handlePackageEvent(Intent intent, int userId) {
		super.handlePackageEvent(intent, userId);
		}
		}

		static class TestSerializer implements XmlSerializerAndParser<TestServiceType> {