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

Commit 7b02804d authored by Jon Miranda's avatar Jon Miranda Committed by Jonathan Miranda
Browse files

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

Bug: 153821199
Change-Id: I0caa546254bb51ca8b88cdb16d79c054560bf2be
(cherry picked from commit e620872b)
parent fe68c9ee
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);
            }
        }
    }