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

am 04c6d73f: Revert "DO NOT MERGE Fix race condition between binder deaths"

Browse Source 
* commit '04c6d73fd0e4472505275688ef8497bc6697647c':
  Revert "DO NOT MERGE Fix race condition between binder deaths"
This commit is contained in:
Justin Koh
2014-09-26 01:41:55 +00:00
committed by Android Git Automerger
parent 6ae7ea942f 04c6d73fd0
commit 1c487938c4
3 changed files with 2 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
services/core/java/com/android/server/wm/AppWindowToken.java
View File

@@ -245,15 +245,6 @@ class AppWindowToken extends WindowToken {
return false; return false;
} }
void removeAllWindows() {
for (int winNdx = allAppWindows.size() - 1; winNdx >= 0; --winNdx) {
WindowState win = allAppWindows.get(winNdx);
if (WindowManagerService.DEBUG_WINDOW_MOVEMENT) Slog.w(WindowManagerService.TAG,
"removeAllWindows: removing win=" + win);
win.mService.removeWindowLocked(win.mSession, win);
}
}
@Override @Override
void dump(PrintWriter pw, String prefix) { void dump(PrintWriter pw, String prefix) {
super.dump(pw, prefix); super.dump(pw, prefix);

2
services/core/java/com/android/server/wm/TaskStack.java
View File

@@ -155,7 +155,7 @@ public class TaskStack {
final ArrayList<WindowState> windows = activities.get(activityNdx).allAppWindows; final ArrayList<WindowState> windows = activities.get(activityNdx).allAppWindows;
for (int winNdx = windows.size() - 1; winNdx >= 0; --winNdx) { for (int winNdx = windows.size() - 1; winNdx >= 0; --winNdx) {
final WindowStateAnimator winAnimator = windows.get(winNdx).mWinAnimator; final WindowStateAnimator winAnimator = windows.get(winNdx).mWinAnimator;
if (winAnimator.isAnimating() || winAnimator.mWin.mExiting) { if (winAnimator.isAnimating() && !winAnimator.isDummyAnimation()) {
return true; return true;
} }
} }

4
services/core/java/com/android/server/wm/WindowManagerService.java
View File

@@ -3540,7 +3540,7 @@ public class WindowManagerService extends IWindowManager.Stub
return; return;
} }
final Task oldTask = mTaskIdToTask.get(atoken.groupId); final Task oldTask = mTaskIdToTask.get(atoken.groupId);
oldTask.removeAppToken(atoken); removeAppFromTaskLocked(atoken);
atoken.groupId = groupId; atoken.groupId = groupId;
Task newTask = mTaskIdToTask.get(groupId); Task newTask = mTaskIdToTask.get(groupId);
@@ -4563,8 +4563,6 @@ public class WindowManagerService extends IWindowManager.Stub
} }
void removeAppFromTaskLocked(AppWindowToken wtoken) { void removeAppFromTaskLocked(AppWindowToken wtoken) {
wtoken.removeAllWindows();
final Task task = mTaskIdToTask.get(wtoken.groupId); final Task task = mTaskIdToTask.get(wtoken.groupId);
if (task != null) { if (task != null) {
if (!task.removeAppToken(wtoken)) { if (!task.removeAppToken(wtoken)) {