Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit e620872b authored by Jon Miranda's avatar Jon Miranda
Browse files

Add logs to debug playtime crash and band aid fix for release builds.

Bug: 153821199
Change-Id: I0caa546254bb51ca8b88cdb16d79c054560bf2be
parent 51e144ff
Loading
Loading
Loading
Loading
+13 −1
Original line number Diff line number Diff line
@@ -26,6 +26,7 @@ import android.annotation.TargetApi;
import android.content.Context;
import android.os.Build;
import android.os.Handler;
import android.util.Log;

import androidx.annotation.BinderThread;
import androidx.annotation.UiThread;
@@ -37,6 +38,8 @@ import com.android.systemui.shared.system.RemoteAnimationTargetCompat;
public abstract class LauncherAnimationRunner implements RemoteAnimationRunnerCompat,
        WrappedAnimationRunnerImpl {

    private static final String TAG = "LauncherAnimationRunner";

    private final Handler mHandler;
    private final boolean mStartAtFrontOfQueue;
    private AnimationResult mAnimationResult;
@@ -151,7 +154,16 @@ public abstract class LauncherAnimationRunner implements RemoteAnimationRunnerCo

                // Because t=0 has the app icon in its original spot, we can skip the
                // first frame and have the same movement one frame earlier.
                mAnimator.setCurrentPlayTime(getSingleFrameMs(context));
                int singleFrameMs = getSingleFrameMs(context);
                long playTime = singleFrameMs;
                // b/153821199 Add logs to debug crash but ensure release builds do not crash.
                if (Utilities.IS_DEBUG_DEVICE) {
                    Log.e(TAG, "Total duration=[" + mAnimator.getTotalDuration()
                            + "], singleFrameMs=[" + singleFrameMs + "], mAnimator=" + mAnimator);
                } else {
                    playTime = Math.min(singleFrameMs, mAnimator.getTotalDuration());
                }
                mAnimator.setCurrentPlayTime(playTime);
            }
        }
    }