commit 9af30706d1e640a9e0347b7c5ec6d4a3e766ea45
author Melody Hsu <melodymhsu@google.com> Wed Apr 17 18:23:43 2024 +0000
committer Melody Hsu <melodymhsu@google.com> Wed Apr 17 18:23:43 2024 +0000
tree b0e807c0ee635dc6174792bf620e714d08fe90d8
parent 5494c190af80adf979896e59a06cfedbfd94942d

[legacy sf flag] - skipReportingTransformHint

Removing legacy frontend SF flag, skipReportingTransformHint
and its associated methods no longer used.

Bug: b/330785038
Test: presubmit
Change-Id: Ie7acd1d5ec362657e392115558a37d6d953dacf4

services/surfaceflinger/Layer.cpp

diff --git a/services/surfaceflinger/Layer.cpp b/services/surfaceflinger/Layer.cpp
index 073bad3..e127106 100644
--- a/services/surfaceflinger/Layer.cpp
+++ b/services/surfaceflinger/Layer.cpp
@@ -1549,10 +1549,6 @@
     return usage;
 }
 
-void Layer::skipReportingTransformHint() {
-    mSkipReportingTransformHint = true;
-}
-
 void Layer::updateTransformHint(ui::Transform::RotationFlags transformHint) {
     if (mFlinger->mDebugDisableTransformHint || transformHint & ui::Transform::ROT_INVALID) {
         transformHint = ui::Transform::ROT_0;
@@ -2976,13 +2972,7 @@
 
 void Layer::releasePendingBuffer(nsecs_t dequeueReadyTime) {
     for (const auto& handle : mDrawingState.callbackHandles) {
-        if (mFlinger->mLayerLifecycleManagerEnabled) {
-            handle->transformHint = mTransformHint;
-        } else {
-            handle->transformHint = mSkipReportingTransformHint
-                    ? std::nullopt
-                    : std::make_optional<uint32_t>(mTransformHintLegacy);
-        }
+        handle->transformHint = mTransformHint;
         handle->dequeueReadyTime = dequeueReadyTime;
         handle->currentMaxAcquiredBufferCount =
                 mFlinger->getMaxAcquiredBufferCountForCurrentRefreshRate(mOwnerUid);
@@ -4326,7 +4316,6 @@
     if (mTransformHintLegacy == ui::Transform::ROT_INVALID) {
         mTransformHintLegacy = displayTransformHint;
     }
-    mSkipReportingTransformHint = false;
 }
 
 const std::shared_ptr<renderengine::ExternalTexture>& Layer::getExternalTexture() const {