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

Commit df7c7255 authored by Robert Carr's avatar Robert Carr Committed by Rob Carr
Browse files

DO NOT MERGE: Track SurfaceComposerClient changes 

surfaceControlStats now includes the acquire count.

Bug: 197269223
Bug: 212846697
Test: Existing tests pass
Change-Id: I29f2e9ba1c100a831be5eddce2c2a442d136d98e
parent 86643bc7

native/android/surface_control.cpp

+3 −3
Original line number Diff line number Diff line
@@ -298,7 +298,7 @@ void ASurfaceTransaction_setOnComplete(ASurfaceTransaction* aSurfaceTransaction, 


        auto& aSurfaceControlStats = aSurfaceTransactionStats.aSurfaceControlStats;



        for (const auto& [surfaceControl, latchTime, acquireTime, presentFence, previousReleaseFence, transformHint, frameEvents] : surfaceControlStats) {

        for (const auto& [surfaceControl, latchTime, acquireTime, presentFence, previousReleaseFence, transformHint, frameEvents, ignore] : surfaceControlStats) {

            ASurfaceControl* aSurfaceControl = reinterpret_cast<ASurfaceControl*>(surfaceControl.get());

            aSurfaceControlStats[aSurfaceControl].acquireTime = acquireTime;

            aSurfaceControlStats[aSurfaceControl].previousReleaseFence = previousReleaseFence;
@@ -650,7 +650,7 @@ void ASurfaceTransaction_setOnCommit(ASurfaceTransaction* aSurfaceTransaction, v 
                for (const auto&

                             [surfaceControl, latchTime, acquireTime, presentFence,

                              previousReleaseFence, transformHint,

                              frameEvents] : surfaceControlStats) {

                              frameEvents, ignore] : surfaceControlStats) {

                    ASurfaceControl* aSurfaceControl =

                            reinterpret_cast<ASurfaceControl*>(surfaceControl.get());

                    aSurfaceControlStats[aSurfaceControl].acquireTime = acquireTime;