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

Commit 00ee8263 authored by Robin Lee's avatar Robin Lee
Browse files

Clean up flag enforce_shell_thread_model 

Bug: 351189446
Change-Id: Ib2932a33151a1e6be09ad279f0aa56e8928c1540
Test: N/A
Flag: com.android.window.flags.enforce_shell_thread_model
parent 71724c7f

core/java/android/window/flags/windowing_frontend.aconfig

+0 −10
Original line number Diff line number Diff line
@@ -208,16 +208,6 @@ flag { 
  }

}



flag {

  name: "enforce_shell_thread_model"

  namespace: "windowing_frontend"

  description: "Crash the shell process if someone calls in from the wrong thread"

  bug: "351189446"

  is_fixed_read_only: true

  metadata {

    purpose: PURPOSE_BUGFIX

  }

}

flag {

  name: "custom_animations_behind_translucent"

  namespace: "windowing_frontend"

libs/WindowManager/Shell/src/com/android/wm/shell/transition/Transitions.java

+2 −7
Original line number Diff line number Diff line
@@ -38,7 +38,6 @@ import static android.window.TransitionInfo.FLAG_IS_WALLPAPER; 
import static android.window.TransitionInfo.FLAG_NO_ANIMATION;

import static android.window.TransitionInfo.FLAG_STARTING_WINDOW_TRANSFER_RECIPIENT;



import static com.android.window.flags.Flags.enforceShellThreadModel;

import static com.android.window.flags.Flags.ensureWallpaperInTransitions;

import static com.android.systemui.shared.Flags.returnAnimationFrameworkLibrary;

import static com.android.wm.shell.shared.TransitionUtil.isClosingType;
@@ -936,9 +935,7 @@ public class Transitions implements RemoteCallable<Transitions>, 
    }



    private void onMerged(@NonNull IBinder playingToken, @NonNull IBinder mergedToken) {

        if (enforceShellThreadModel()) {

        mMainExecutor.assertCurrentThread();

        }



        ActiveTransition playing = mKnownTransitions.get(playingToken);

        if (playing == null) {
@@ -1087,9 +1084,7 @@ public class Transitions implements RemoteCallable<Transitions>, 
    }



    private void onFinish(IBinder token, @Nullable WindowContainerTransaction wct) {

        if (enforceShellThreadModel()) {

        mMainExecutor.assertCurrentThread();

        }



        final ActiveTransition active = mKnownTransitions.get(token);

        if (active == null) {