diff options
| author | Android (Google) Code Review <android-gerrit@google.com> | 2009-06-16 13:14:09 -0700 |
|---|---|---|
| committer | Android (Google) Code Review <android-gerrit@google.com> | 2009-06-16 13:14:09 -0700 |
| commit | 5633034f79bfa67c4219f16fd6d23e962e800be2 (patch) | |
| tree | 8566caee5e5acf3a71124fff06596b3b85456fbd /core/java | |
| parent | a6061e043cb6056c303206c2c9870ae3758861f1 (diff) | |
| parent | 12172baa6b72a8996b3f043f18067c9b43ed3bc8 (diff) | |
| download | frameworks_base-5633034f79bfa67c4219f16fd6d23e962e800be2.zip frameworks_base-5633034f79bfa67c4219f16fd6d23e962e800be2.tar.gz frameworks_base-5633034f79bfa67c4219f16fd6d23e962e800be2.tar.bz2 | |
Merge change 4115 into donut
* changes:
* component can be null after destory? This was causing NPE in catch block, which hides the actual exception.
Diffstat (limited to 'core/java')
| -rw-r--r-- | core/java/android/app/ActivityThread.java | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/core/java/android/app/ActivityThread.java b/core/java/android/app/ActivityThread.java index d08fc11..2fe6471 100644 --- a/core/java/android/app/ActivityThread.java +++ b/core/java/android/app/ActivityThread.java @@ -31,6 +31,7 @@ import android.content.pm.InstrumentationInfo; import android.content.pm.PackageManager; import android.content.pm.ProviderInfo; import android.content.pm.ServiceInfo; +import android.content.pm.PackageParser.Component; import android.content.res.AssetManager; import android.content.res.Configuration; import android.content.res.Resources; @@ -3286,10 +3287,10 @@ public final class ActivityThread { throw e; } catch (Exception e) { if (!mInstrumentation.onException(r.activity, e)) { + ComponentName component = r.intent.getComponent(); + String name = component == null ? "[Unknown]" : component.toShortString(); throw new RuntimeException( - "Unable to destroy activity " - + r.intent.getComponent().toShortString() - + ": " + e.toString(), e); + "Unable to destroy activity " + name + ": " + e.toString(), e); } } } |
