Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Revert "Check if task was top of stack before it's removed""

Browse Source
This commit is contained in:
Chong Zhang
2016-08-08 18:52:21 +00:00
committed by Android (Google) Code Review
parent 78ab4f10d4 bffd889cb0
commit c8e978208a
1 changed files with 6 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
services/core/java/com/android/server/am/ActivityStack.java
View File

@@ -3885,8 +3885,7 @@ final class ActivityStack {
r.finishLaunchTickingLocked();
}
private void removeActivityFromHistoryLocked(
ActivityRecord r, TaskRecord oldTop, String reason) {
private void removeActivityFromHistoryLocked(ActivityRecord r, String reason) {
mStackSupervisor.removeChildActivityContainers(r);
finishActivityResultsLocked(r, Activity.RESULT_CANCELED, null);
r.makeFinishingLocked();
@@ -3905,11 +3904,10 @@ final class ActivityStack {
validateAppTokensLocked();
}
final TaskRecord task = r.task;
final TaskRecord topTask = oldTop != null ? oldTop : topTask();
if (task != null && task.removeActivity(r)) {
if (DEBUG_STACK) Slog.i(TAG_STACK,
"removeActivityFromHistoryLocked: last activity removed from " + this);
if (mStackSupervisor.isFocusedStack(this) && task == topTask &&
if (mStackSupervisor.isFocusedStack(this) && task == topTask() &&
task.isOverHomeStack()) {
mStackSupervisor.moveHomeStackTaskToTop(task.getTaskToReturnTo(), reason);
}
@@ -4045,12 +4043,6 @@ final class ActivityStack {
boolean removedFromHistory = false;
// If the activity is finishing, it's no longer considered in topRunningActivityLocked,
// and cleanUpActivityLocked() may change focus to another activity (or task).
// Get the current top task now, as removeActivityFromHistoryLocked() below need this
// to decide whether to return to home stack after removal.
final TaskRecord topTask = topTask();
cleanUpActivityLocked(r, false, false);
final boolean hadApp = r.app != null;
@@ -4085,8 +4077,7 @@ final class ActivityStack {
// up.
//Slog.w(TAG, "Exception thrown during finish", e);
if (r.finishing) {
removeActivityFromHistoryLocked(
r, topTask, reason + " exceptionInScheduleDestroy");
removeActivityFromHistoryLocked(r, reason + " exceptionInScheduleDestroy");
removedFromHistory = true;
skipDestroy = true;
}
@@ -4117,7 +4108,7 @@ final class ActivityStack {
} else {
// remove this record from the history.
if (r.finishing) {
removeActivityFromHistoryLocked(r, topTask, reason + " hadNoApp");
removeActivityFromHistoryLocked(r, reason + " hadNoApp");
removedFromHistory = true;
} else {
if (DEBUG_STATES) Slog.v(TAG_STATES, "Moving to DESTROYED: " + r + " (no app)");
@@ -4148,7 +4139,7 @@ final class ActivityStack {
if (isInStackLocked(r) != null) {
if (r.state == ActivityState.DESTROYING) {
cleanUpActivityLocked(r, true, false);
removeActivityFromHistoryLocked(r, null, reason);
removeActivityFromHistoryLocked(r, reason);
}
}
mStackSupervisor.resumeFocusedStackTopActivityLocked();
@@ -4306,7 +4297,7 @@ final class ActivityStack {
}
cleanUpActivityLocked(r, true, true);
if (remove) {
removeActivityFromHistoryLocked(r, null, "appDied");
removeActivityFromHistoryLocked(r, "appDied");
}
}
}