Merge "Don't apply transactions of abort out-of-order." into udc-dev am: bb726e9c7f am: a094ab219e
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/23603509 Change-Id: If97a8cb902e275f2142350b5b6525eb107200d6d Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
@@ -813,6 +813,9 @@ public class Transitions implements RemoteCallable<Transitions>,
|
||||
track.mReadyTransitions.remove(0);
|
||||
track.mActiveTransition = ready;
|
||||
if (ready.mAborted) {
|
||||
if (ready.mStartT != null) {
|
||||
ready.mStartT.apply();
|
||||
}
|
||||
// finish now since there's nothing to animate. Calls back into processReadyQueue
|
||||
onFinish(ready, null, null);
|
||||
return;
|
||||
@@ -940,10 +943,6 @@ public class Transitions implements RemoteCallable<Transitions>,
|
||||
/** Aborts a transition. This will still queue it up to maintain order. */
|
||||
private void onAbort(ActiveTransition transition) {
|
||||
final Track track = mTracks.get(transition.getTrack());
|
||||
// apply immediately since they may be "parallel" operations: We currently we use abort for
|
||||
// thing which are independent to other transitions (like starting-window transfer).
|
||||
transition.mStartT.apply();
|
||||
transition.mFinishT.apply();
|
||||
transition.mAborted = true;
|
||||
|
||||
mTracer.logAborted(transition.mInfo.getDebugId());
|
||||
|
||||
Reference in New Issue
Block a user