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

Commit 2606ec0e authored by Melody Hsu's avatar Melody Hsu
Browse files

Place recordLayerHistoryAnimationTx behind flag 

Current block of code uses the legacy layer, which will eventually
be removed. It is already called elsewhere and is doubly called without
the LayerLifecycleManager flag.

Bug: b/330785038
Test: presubmit
Change-Id: Iea6cb58c3885967ccbf0245549f060380bc1d745
parent ae9b557b

services/surfaceflinger/SurfaceFlinger.cpp

+13 −11
Original line number Diff line number Diff line
@@ -5205,6 +5205,7 @@ bool SurfaceFlinger::applyTransactionState(const FrameTimelineInfo& frameTimelin 
                                                             desiredPresentTime, isAutoTimestamp,

                                                             postTime, transactionId);

        }

        if (!mLayerLifecycleManagerEnabled) {

            if ((flags & eAnimation) && resolvedState.state.surface) {

                if (const auto layer = LayerHandle::getLayer(resolvedState.state.surface)) {

                    const auto layerProps = scheduler::LayerProps{
@@ -5219,6 +5220,7 @@ bool SurfaceFlinger::applyTransactionState(const FrameTimelineInfo& frameTimelin 
                }

            }

        }

    }



    transactionFlags |= clientStateFlags;

    transactionFlags |= addInputWindowCommands(inputWindowCommands);