am b921b37b: am a327dbdd: am ecb0e637: Fix issue #2541849: System crash when an app with active notifications...

This commit is contained in:
Dianne Hackborn
2010-04-07 22:44:56 -07:00
committed by Android Git Automerger

View File

@@ -33,6 +33,7 @@ import android.app.admin.IDevicePolicyManager;
import android.app.backup.IBackupManager; import android.app.backup.IBackupManager;
import android.content.Context; import android.content.Context;
import android.content.ComponentName; import android.content.ComponentName;
import android.content.IIntentReceiver;
import android.content.Intent; import android.content.Intent;
import android.content.IntentFilter; import android.content.IntentFilter;
import android.content.IntentSender; import android.content.IntentSender;
@@ -589,11 +590,11 @@ class PackageManagerService extends IPackageManager.Stub {
} }
sendPackageBroadcast(Intent.ACTION_PACKAGE_ADDED, sendPackageBroadcast(Intent.ACTION_PACKAGE_ADDED,
res.pkg.applicationInfo.packageName, res.pkg.applicationInfo.packageName,
extras); extras, null);
if (update) { if (update) {
sendPackageBroadcast(Intent.ACTION_PACKAGE_REPLACED, sendPackageBroadcast(Intent.ACTION_PACKAGE_REPLACED,
res.pkg.applicationInfo.packageName, res.pkg.applicationInfo.packageName,
extras); extras, null);
} }
if (res.removedInfo.args != null) { if (res.removedInfo.args != null) {
// Remove the replaced package's older resources safely now // Remove the replaced package's older resources safely now
@@ -4496,7 +4497,8 @@ class PackageManagerService extends IPackageManager.Stub {
} }
}; };
private static final void sendPackageBroadcast(String action, String pkg, Bundle extras) { private static final void sendPackageBroadcast(String action, String pkg,
Bundle extras, IIntentReceiver finishedReceiver) {
IActivityManager am = ActivityManagerNative.getDefault(); IActivityManager am = ActivityManagerNative.getDefault();
if (am != null) { if (am != null) {
try { try {
@@ -4506,9 +4508,8 @@ class PackageManagerService extends IPackageManager.Stub {
intent.putExtras(extras); intent.putExtras(extras);
} }
intent.addFlags(Intent.FLAG_RECEIVER_REGISTERED_ONLY_BEFORE_BOOT); intent.addFlags(Intent.FLAG_RECEIVER_REGISTERED_ONLY_BEFORE_BOOT);
am.broadcastIntent( am.broadcastIntent(null, intent, null, finishedReceiver,
null, intent, 0, null, null, null, finishedReceiver != null, false);
null, null, 0, null, null, null, false, false);
} catch (RemoteException ex) { } catch (RemoteException ex) {
} }
} }
@@ -4630,12 +4631,14 @@ class PackageManagerService extends IPackageManager.Stub {
Bundle extras = new Bundle(1); Bundle extras = new Bundle(1);
extras.putInt(Intent.EXTRA_UID, removedUid); extras.putInt(Intent.EXTRA_UID, removedUid);
extras.putBoolean(Intent.EXTRA_DATA_REMOVED, false); extras.putBoolean(Intent.EXTRA_DATA_REMOVED, false);
sendPackageBroadcast(Intent.ACTION_PACKAGE_REMOVED, removedPackage, extras); sendPackageBroadcast(Intent.ACTION_PACKAGE_REMOVED, removedPackage,
extras, null);
} }
if (addedPackage != null) { if (addedPackage != null) {
Bundle extras = new Bundle(1); Bundle extras = new Bundle(1);
extras.putInt(Intent.EXTRA_UID, addedUid); extras.putInt(Intent.EXTRA_UID, addedUid);
sendPackageBroadcast(Intent.ACTION_PACKAGE_ADDED, addedPackage, extras); sendPackageBroadcast(Intent.ACTION_PACKAGE_ADDED, addedPackage,
extras, null);
} }
} }
@@ -6077,8 +6080,8 @@ class PackageManagerService extends IPackageManager.Stub {
extras.putInt(Intent.EXTRA_UID, info.removedUid >= 0 ? info.removedUid : info.uid); extras.putInt(Intent.EXTRA_UID, info.removedUid >= 0 ? info.removedUid : info.uid);
extras.putBoolean(Intent.EXTRA_REPLACING, true); extras.putBoolean(Intent.EXTRA_REPLACING, true);
sendPackageBroadcast(Intent.ACTION_PACKAGE_ADDED, packageName, extras); sendPackageBroadcast(Intent.ACTION_PACKAGE_ADDED, packageName, extras, null);
sendPackageBroadcast(Intent.ACTION_PACKAGE_REPLACED, packageName, extras); sendPackageBroadcast(Intent.ACTION_PACKAGE_REPLACED, packageName, extras, null);
} }
} }
// Force a gc here. // Force a gc here.
@@ -6109,10 +6112,10 @@ class PackageManagerService extends IPackageManager.Stub {
extras.putBoolean(Intent.EXTRA_REPLACING, true); extras.putBoolean(Intent.EXTRA_REPLACING, true);
} }
if (removedPackage != null) { if (removedPackage != null) {
sendPackageBroadcast(Intent.ACTION_PACKAGE_REMOVED, removedPackage, extras); sendPackageBroadcast(Intent.ACTION_PACKAGE_REMOVED, removedPackage, extras, null);
} }
if (removedUid >= 0) { if (removedUid >= 0) {
sendPackageBroadcast(Intent.ACTION_UID_REMOVED, null, extras); sendPackageBroadcast(Intent.ACTION_UID_REMOVED, null, extras, null);
} }
} }
} }
@@ -6819,7 +6822,7 @@ class PackageManagerService extends IPackageManager.Stub {
extras.putStringArray(Intent.EXTRA_CHANGED_COMPONENT_NAME_LIST, nameList); extras.putStringArray(Intent.EXTRA_CHANGED_COMPONENT_NAME_LIST, nameList);
extras.putBoolean(Intent.EXTRA_DONT_KILL_APP, killFlag); extras.putBoolean(Intent.EXTRA_DONT_KILL_APP, killFlag);
extras.putInt(Intent.EXTRA_UID, packageUid); extras.putInt(Intent.EXTRA_UID, packageUid);
sendPackageBroadcast(Intent.ACTION_PACKAGE_CHANGED, packageName, extras); sendPackageBroadcast(Intent.ACTION_PACKAGE_CHANGED, packageName, extras, null);
} }
public String getInstallerPackageName(String packageName) { public String getInstallerPackageName(String packageName) {
@@ -9549,7 +9552,7 @@ class PackageManagerService extends IPackageManager.Stub {
} }
private void sendResourcesChangedBroadcast(boolean mediaStatus, private void sendResourcesChangedBroadcast(boolean mediaStatus,
ArrayList<String> pkgList, int uidArr[]) { ArrayList<String> pkgList, int uidArr[], IIntentReceiver finishedReceiver) {
int size = pkgList.size(); int size = pkgList.size();
if (size > 0) { if (size > 0) {
// Send broadcasts here // Send broadcasts here
@@ -9561,7 +9564,7 @@ class PackageManagerService extends IPackageManager.Stub {
} }
String action = mediaStatus ? Intent.ACTION_EXTERNAL_APPLICATIONS_AVAILABLE String action = mediaStatus ? Intent.ACTION_EXTERNAL_APPLICATIONS_AVAILABLE
: Intent.ACTION_EXTERNAL_APPLICATIONS_UNAVAILABLE; : Intent.ACTION_EXTERNAL_APPLICATIONS_UNAVAILABLE;
sendPackageBroadcast(action, null, extras); sendPackageBroadcast(action, null, extras, finishedReceiver);
} }
} }
@@ -9648,7 +9651,7 @@ class PackageManagerService extends IPackageManager.Stub {
} }
// Send a broadcast to let everyone know we are done processing // Send a broadcast to let everyone know we are done processing
if (pkgList.size() > 0) { if (pkgList.size() > 0) {
sendResourcesChangedBroadcast(true, pkgList, uidArr); sendResourcesChangedBroadcast(true, pkgList, uidArr, null);
} }
if (doGc) { if (doGc) {
Runtime.getRuntime().gc(); Runtime.getRuntime().gc();
@@ -9686,10 +9689,15 @@ class PackageManagerService extends IPackageManager.Stub {
} }
// Send broadcasts // Send broadcasts
if (pkgList.size() > 0) { if (pkgList.size() > 0) {
sendResourcesChangedBroadcast(false, pkgList, uidArr); sendResourcesChangedBroadcast(false, pkgList, uidArr, new IIntentReceiver.Stub() {
} public void performReceive(Intent intent, int resultCode, String data, Bundle extras,
// Force gc boolean ordered, boolean sticky) throws RemoteException {
// Force gc now that everyone is done cleaning up, to release
// references on assets.
Runtime.getRuntime().gc(); Runtime.getRuntime().gc();
}
});
}
// Just unmount all valid containers. // Just unmount all valid containers.
for (SdInstallArgs args : keys) { for (SdInstallArgs args : keys) {
synchronized (mInstallLock) { synchronized (mInstallLock) {
@@ -9779,7 +9787,7 @@ class PackageManagerService extends IPackageManager.Stub {
} }
if (returnCode == PackageManager.MOVE_SUCCEEDED) { if (returnCode == PackageManager.MOVE_SUCCEEDED) {
// Send resources unavailable broadcast // Send resources unavailable broadcast
sendResourcesChangedBroadcast(false, pkgList, uidArr); sendResourcesChangedBroadcast(false, pkgList, uidArr, null);
// Update package code and resource paths // Update package code and resource paths
synchronized (mInstallLock) { synchronized (mInstallLock) {
synchronized (mPackages) { synchronized (mPackages) {
@@ -9830,7 +9838,7 @@ class PackageManagerService extends IPackageManager.Stub {
} }
} }
// Send resources available broadcast // Send resources available broadcast
sendResourcesChangedBroadcast(true, pkgList, uidArr); sendResourcesChangedBroadcast(true, pkgList, uidArr, null);
} }
} }
} }