commit 4e4a3ecc47ef0fc36889c65918e43eb156ba7e92
author Bryce Lee <brycelee@google.com> Wed Sep 27 08:25:03 2017 -0700
committer Bryce Lee <brycelee@google.com> Wed Oct 04 12:15:06 2017 -0700
tree 4bf83332aca782b8e29b73c083d5cf43f9042a4e
parent 0debe0b7f59696d034471ec29e0eae244ea50a96

Consolidate updating bounds in ActivityStarter.

ActivityStarter updates the bounds of the host stack or task in
multiple places. This changelist consolidates this duplicated logic
into a single place and adds tests to ensure the expected behavior.

This change also makes ActivityStack create its controller using the
same WindowManagerService instance as its ActivityStackSupervisor.

Fixes: 67091889
Bug: 64144308
Test: bit FrameworksServicesTests:com.android.server.am.ActivityStarterTests
Change-Id: Ibaa8f7966bbeb190723caab0d2788c79cf848f04

services/core/java/com/android/server/am/PinnedActivityStack.java

diff --git a/services/core/java/com/android/server/am/PinnedActivityStack.java b/services/core/java/com/android/server/am/PinnedActivityStack.java
index 468b867..2726979 100644
--- a/services/core/java/com/android/server/am/PinnedActivityStack.java
+++ b/services/core/java/com/android/server/am/PinnedActivityStack.java
@@ -43,7 +43,8 @@
     @Override
     PinnedStackWindowController createStackWindowController(int displayId, boolean onTop,
             Rect outBounds) {
-        return new PinnedStackWindowController(mStackId, this, displayId, onTop, outBounds);
+        return new PinnedStackWindowController(mStackId, this, displayId, onTop, outBounds,
+                mStackSupervisor.mWindowManager);
     }
 
     Rect getDefaultPictureInPictureBounds(float aspectRatio) {