Merge "Don\'t allow pinned activities to request visible behind" into nyc-dev am: 02ebaa0

am: 188b013

* commit '188b0139aab72212c190c000e84a033d49d94a2a':
  Don't allow pinned activities to request visible behind

Change-Id: I91a58b64ec8a90ea36c3928d9e3d2ab61c76b6a9
This commit is contained in:
TreeHugger Robot
2016-04-18 04:34:06 +00:00
committed by android-build-merger
2 changed files with 22 additions and 0 deletions

View File

@@ -675,6 +675,14 @@ public class ActivityManager {
return isStaticStack(stackId) || stackId == PINNED_STACK_ID
|| stackId == FREEFORM_WORKSPACE_STACK_ID || stackId == DOCKED_STACK_ID;
}
/**
* Returns true if activities contained in this stack can request visible behind by
* calling {@link Activity#requestVisibleBehind}.
*/
public static boolean activitiesCanRequestVisibleBehind(int stackId) {
return stackId == FULLSCREEN_WORKSPACE_STACK_ID;
}
}
/**

View File

@@ -2430,6 +2430,12 @@ public final class ActivityStackSupervisor implements DisplayListener {
try {
final TaskRecord task = r.task;
if (r == task.stack.getVisibleBehindActivity()) {
// An activity can't be pinned and visible behind at the same time. Go ahead and
// release it from been visible behind before pinning.
requestVisibleBehindLocked(r, false);
}
// Need to make sure the pinned stack exist so we can resize it below...
final ActivityStack stack = getStack(PINNED_STACK_ID, CREATE_IF_NEEDED, ON_TOP);
@@ -2689,6 +2695,14 @@ public final class ActivityStackSupervisor implements DisplayListener {
"requestVisibleBehind: r=" + r + " visible=" + visible + " stack is null");
return false;
}
if (visible && !StackId.activitiesCanRequestVisibleBehind(stack.mStackId)) {
if (DEBUG_VISIBLE_BEHIND) Slog.d(TAG_VISIBLE_BEHIND, "requestVisibleBehind: r=" + r
+ " visible=" + visible + " stackId=" + stack.mStackId
+ " can't contain visible behind activities");
return false;
}
final boolean isVisible = stack.hasVisibleBehindActivity();
if (DEBUG_VISIBLE_BEHIND) Slog.d(TAG_VISIBLE_BEHIND,
"requestVisibleBehind r=" + r + " visible=" + visible + " isVisible=" + isVisible);