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

Commit 91cb84cb authored by Cosmin Băieș's avatar Cosmin Băieș
Browse files

Start tracking IME user animation requests 

This starts tracking IME user animation requests again. These were
temporarily stopped as they required adding the onStart call on the UI
thread call path, which triggered a jank regression.

Now that ImeTracker onStart is also asynchronous, we can start tracking
these types of requests again.

This is essentially a revert of [1].

  [1]: I251edecfd4d26d3bed90448a6062ccd029bbd0ed

Flag: EXEMPT bugfix
Test: enable Swipe up to start search, trigger user animation in all
  apps, observe logcat
Bug: 342111149
Change-Id: Ic35bf71c084fadd291cb6964156f2d6c7ceed63a
parent 2cc68117

core/java/android/view/InsetsController.java

+7 −2
Original line number Diff line number Diff line
@@ -1351,10 +1351,15 @@ public class InsetsController implements WindowInsetsController, InsetsAnimation 
                return interpolator;

            }

        };

        // TODO(b/342111149): Create statsToken here once ImeTracker#onStart becomes async.

        final var statsToken = (types & ime()) == 0

                ? null

                : ImeTracker.forLogging().onStart(ImeTracker.TYPE_USER,

                        ImeTracker.ORIGIN_CLIENT,

                        SoftInputShowHideReason.CONTROL_WINDOW_INSETS_ANIMATION,

                        mHost.isHandlingPointerEvent() /* fromUser */);

        controlAnimationUnchecked(types, cancellationSignal, listener, mFrame, mBounds, spec,

                animationType, getLayoutInsetsDuringAnimationMode(types, fromPredictiveBack),

                false /* useInsetsAnimationThread */, null, fromPredictiveBack);

                false /* useInsetsAnimationThread */, statsToken, fromPredictiveBack);

    }



    private void controlAnimationUnchecked(@InsetsType int types,