summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Louis Chang <louischang@google.com> 2018-10-09 01:19:28 +0000
committer Android (Google) Code Review <android-gerrit@google.com> 2018-10-09 01:19:28 +0000
commite8fddd7291812e75380b07f057e7e235746b7737 (patch)
tree9f624abb7ab79a92ad5a14a226e6c2e360be7b56
parentd1bbfbe90f3ec08e7da3dae46ef7e9e0f190dfe5 (diff)
parent1bb884a858df14f5943dc590cf6c11d4dc13fcd4 (diff)
Merge "Fix NullPointerException in ActivityStack"
-rw-r--r--services/core/java/com/android/server/am/ActivityStack.java18
1 files changed, 10 insertions, 8 deletions
diff --git a/services/core/java/com/android/server/am/ActivityStack.java b/services/core/java/com/android/server/am/ActivityStack.java
index ebfaf0fe9a7e..6388423d04b4 100644
--- a/services/core/java/com/android/server/am/ActivityStack.java
+++ b/services/core/java/com/android/server/am/ActivityStack.java
@@ -163,7 +163,6 @@ import java.io.FileDescriptor;
import java.io.PrintWriter;
import java.lang.ref.WeakReference;
import java.util.ArrayList;
-import java.util.Iterator;
import java.util.List;
import java.util.Objects;
import java.util.Set;
@@ -1681,13 +1680,16 @@ class ActivityStack<T extends StackWindowController> extends ConfigurationContai
if (prev != null) {
prev.resumeKeyDispatchingLocked();
- final long diff = prev.app.getCpuTime() - prev.cpuTimeAtResume;
- if (prev.hasProcess() && prev.cpuTimeAtResume > 0 && diff > 0) {
- final Runnable r = PooledLambda.obtainRunnable(
- ActivityManagerInternal::updateForegroundTimeIfOnBattery,
- mService.mAmInternal, prev.info.packageName, prev.info.applicationInfo.uid,
- diff);
- mService.mH.post(r);
+ if (prev.hasProcess() && prev.cpuTimeAtResume > 0) {
+ final long diff = prev.app.getCpuTime() - prev.cpuTimeAtResume;
+ if (diff > 0) {
+ final Runnable r = PooledLambda.obtainRunnable(
+ ActivityManagerInternal::updateForegroundTimeIfOnBattery,
+ mService.mAmInternal, prev.info.packageName,
+ prev.info.applicationInfo.uid,
+ diff);
+ mService.mH.post(r);
+ }
}
prev.cpuTimeAtResume = 0; // reset it
}