commit 8792c111275d317b8d17124b7167dc713eef106e
author Dominik Laskowski <domlaskowski@google.com> Tue Jul 12 09:03:39 2022 -0700
committer Dominik Laskowski <domlaskowski@google.com> Fri Aug 05 12:11:59 2022 -0700
tree e35372aa69e6bfe5431e671f9514845a398d04c3
parent 063f82e8f95ad6f1f4e0bbc07ce3fd22c9e1eb15

SF: Clean up postComposition

Avoid repetition and sp<> copies for present fences. Query the present
time once. Remove DisplayDevice::getPageFlipCount.

Bug: 241285191
Test: Build
Change-Id: I6371e25a7422183953c0617dc1e17280f36018e4

services/surfaceflinger/TransactionCallbackInvoker.cpp

diff --git a/services/surfaceflinger/TransactionCallbackInvoker.cpp b/services/surfaceflinger/TransactionCallbackInvoker.cpp
index d2c2e29..e5de759 100644
--- a/services/surfaceflinger/TransactionCallbackInvoker.cpp
+++ b/services/surfaceflinger/TransactionCallbackInvoker.cpp
@@ -178,8 +178,8 @@
     return NO_ERROR;
 }
 
-void TransactionCallbackInvoker::addPresentFence(const sp<Fence>& presentFence) {
-    mPresentFence = presentFence;
+void TransactionCallbackInvoker::addPresentFence(sp<Fence> presentFence) {
+    mPresentFence = std::move(presentFence);
 }
 
 void TransactionCallbackInvoker::sendCallbacks(bool onCommitOnly) {