Merge "Don't setOccluded in onLaunchAnimationStart." into tm-qpr-dev
This commit is contained in:
committed by
Android (Google) Code Review
commit
716cce695e
@@ -453,6 +453,8 @@ public class KeyguardService extends Service {
|
|||||||
|
|
||||||
@Override // Binder interface
|
@Override // Binder interface
|
||||||
public void setOccluded(boolean isOccluded, boolean animate) {
|
public void setOccluded(boolean isOccluded, boolean animate) {
|
||||||
|
Log.d(TAG, "setOccluded(" + isOccluded + ")");
|
||||||
|
|
||||||
Trace.beginSection("KeyguardService.mBinder#setOccluded");
|
Trace.beginSection("KeyguardService.mBinder#setOccluded");
|
||||||
checkPermission();
|
checkPermission();
|
||||||
mKeyguardViewMediator.setOccluded(isOccluded, animate);
|
mKeyguardViewMediator.setOccluded(isOccluded, animate);
|
||||||
|
|||||||
@@ -837,9 +837,7 @@ public class KeyguardViewMediator extends CoreStartable implements Dumpable,
|
|||||||
private final ActivityLaunchAnimator.Controller mOccludeAnimationController =
|
private final ActivityLaunchAnimator.Controller mOccludeAnimationController =
|
||||||
new ActivityLaunchAnimator.Controller() {
|
new ActivityLaunchAnimator.Controller() {
|
||||||
@Override
|
@Override
|
||||||
public void onLaunchAnimationStart(boolean isExpandingFullyAbove) {
|
public void onLaunchAnimationStart(boolean isExpandingFullyAbove) {}
|
||||||
setOccluded(true /* occluded */, false /* animate */);
|
|
||||||
}
|
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
public void onLaunchAnimationCancelled() {
|
public void onLaunchAnimationCancelled() {
|
||||||
@@ -925,6 +923,7 @@ public class KeyguardViewMediator extends CoreStartable implements Dumpable,
|
|||||||
RemoteAnimationTarget[] wallpapers,
|
RemoteAnimationTarget[] wallpapers,
|
||||||
RemoteAnimationTarget[] nonApps,
|
RemoteAnimationTarget[] nonApps,
|
||||||
IRemoteAnimationFinishedCallback finishedCallback) throws RemoteException {
|
IRemoteAnimationFinishedCallback finishedCallback) throws RemoteException {
|
||||||
|
Log.d(TAG, "UnoccludeAnimator#onAnimationStart. Set occluded = false.");
|
||||||
setOccluded(false /* isOccluded */, true /* animate */);
|
setOccluded(false /* isOccluded */, true /* animate */);
|
||||||
|
|
||||||
if (apps == null || apps.length == 0 || apps[0] == null) {
|
if (apps == null || apps.length == 0 || apps[0] == null) {
|
||||||
@@ -1676,6 +1675,8 @@ public class KeyguardViewMediator extends CoreStartable implements Dumpable,
|
|||||||
* Notify us when the keyguard is occluded by another window
|
* Notify us when the keyguard is occluded by another window
|
||||||
*/
|
*/
|
||||||
public void setOccluded(boolean isOccluded, boolean animate) {
|
public void setOccluded(boolean isOccluded, boolean animate) {
|
||||||
|
Log.d(TAG, "setOccluded(" + isOccluded + ")");
|
||||||
|
|
||||||
Trace.beginSection("KeyguardViewMediator#setOccluded");
|
Trace.beginSection("KeyguardViewMediator#setOccluded");
|
||||||
if (DEBUG) Log.d(TAG, "setOccluded " + isOccluded);
|
if (DEBUG) Log.d(TAG, "setOccluded " + isOccluded);
|
||||||
mInteractionJankMonitor.cancel(CUJ_LOCKSCREEN_TRANSITION_FROM_AOD);
|
mInteractionJankMonitor.cancel(CUJ_LOCKSCREEN_TRANSITION_FROM_AOD);
|
||||||
@@ -1706,6 +1707,7 @@ public class KeyguardViewMediator extends CoreStartable implements Dumpable,
|
|||||||
*/
|
*/
|
||||||
private void handleSetOccluded(boolean isOccluded, boolean animate) {
|
private void handleSetOccluded(boolean isOccluded, boolean animate) {
|
||||||
Trace.beginSection("KeyguardViewMediator#handleSetOccluded");
|
Trace.beginSection("KeyguardViewMediator#handleSetOccluded");
|
||||||
|
Log.d(TAG, "handleSetOccluded(" + isOccluded + ")");
|
||||||
synchronized (KeyguardViewMediator.this) {
|
synchronized (KeyguardViewMediator.this) {
|
||||||
if (mHiding && isOccluded) {
|
if (mHiding && isOccluded) {
|
||||||
// We're in the process of going away but WindowManager wants to show a
|
// We're in the process of going away but WindowManager wants to show a
|
||||||
@@ -3213,16 +3215,18 @@ public class KeyguardViewMediator extends CoreStartable implements Dumpable,
|
|||||||
// internal state to reflect that immediately, vs. waiting for the launch animator to
|
// internal state to reflect that immediately, vs. waiting for the launch animator to
|
||||||
// begin. Otherwise, calls to setShowingLocked, etc. will not know that we're about to
|
// begin. Otherwise, calls to setShowingLocked, etc. will not know that we're about to
|
||||||
// be occluded and might re-show the keyguard.
|
// be occluded and might re-show the keyguard.
|
||||||
|
Log.d(TAG, "OccludeAnimator#onAnimationStart. Set occluded = true.");
|
||||||
setOccluded(true /* isOccluded */, false /* animate */);
|
setOccluded(true /* isOccluded */, false /* animate */);
|
||||||
}
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
public void onAnimationCancelled(boolean isKeyguardOccluded) throws RemoteException {
|
public void onAnimationCancelled(boolean isKeyguardOccluded) throws RemoteException {
|
||||||
super.onAnimationCancelled(isKeyguardOccluded);
|
super.onAnimationCancelled(isKeyguardOccluded);
|
||||||
setOccluded(isKeyguardOccluded /* occluded */, false /* animate */);
|
|
||||||
|
|
||||||
Log.d(TAG, "Occlude animation cancelled by WM. "
|
Log.d(TAG, "Occlude animation cancelled by WM. "
|
||||||
+ "Setting occluded state to: " + mOccluded);
|
+ "Setting occluded state to: " + isKeyguardOccluded);
|
||||||
|
setOccluded(isKeyguardOccluded /* occluded */, false /* animate */);
|
||||||
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -1761,6 +1761,7 @@ public class CentralSurfacesImpl extends CoreStartable implements
|
|||||||
// activity is exited.
|
// activity is exited.
|
||||||
if (mKeyguardStateController.isShowing()
|
if (mKeyguardStateController.isShowing()
|
||||||
&& !mKeyguardStateController.isKeyguardGoingAway()) {
|
&& !mKeyguardStateController.isKeyguardGoingAway()) {
|
||||||
|
Log.d(TAG, "Setting occluded = true in #startActivity.");
|
||||||
mKeyguardViewMediator.setOccluded(true /* isOccluded */,
|
mKeyguardViewMediator.setOccluded(true /* isOccluded */,
|
||||||
true /* animate */);
|
true /* animate */);
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user