commit 953fc62b019dadf30dd02d643b37dae4434c732c
author Midas Chien <midaschieh@google.com> Mon Mar 06 10:31:30 2023 +0000
committer Francescodario Cuzzocrea <bosconovic@gmail.com> Sat Apr 27 11:46:18 2024 +0200
tree 1799b602db5cb59f9dc44d690b08ba2ad84b6c04
parent 67a29f1ced5f7cec0997a2062cac4e8a6fdf6027

libhwc2.1: don't check SKIP_VALIDATE in HWC3

SKIP_VALIDATE is deprecated in HWC3, so we don't need to check it when
executing PresentOrValidateDisplay.

Bug: 271531314
Test: scroll
Change-Id: Iebc97333d773943a3c89b1756743d0459c66129d

base/hwc3/ComposerCommandEngine.cpp

diff --git a/base/hwc3/ComposerCommandEngine.cpp b/base/hwc3/ComposerCommandEngine.cpp
index f8c4861..3b06fef 100644
--- a/base/hwc3/ComposerCommandEngine.cpp
+++ b/base/hwc3/ComposerCommandEngine.cpp
@@ -206,16 +206,12 @@
 void ComposerCommandEngine::executePresentOrValidateDisplay(
         int64_t display, const std::optional<ClockMonotonicTimestamp> expectedPresentTime) {
     executeSetExpectedPresentTimeInternal(display, expectedPresentTime);
-
-    int err;
     // First try to Present as is.
-    if (mHal->hasCapability(Capability::SKIP_VALIDATE)) {
-        err = mResources->mustValidateDisplay(display) ? IComposerClient::EX_NOT_VALIDATED
-                                                       : executePresentDisplay(display);
-        if (!err) {
-            mWriter->setPresentOrValidateResult(display, PresentOrValidate::Result::Presented);
-            return;
-        }
+    auto err = mResources->mustValidateDisplay(display) ? IComposerClient::EX_NOT_VALIDATED
+                                                        : executePresentDisplay(display);
+    if (!err) {
+        mWriter->setPresentOrValidateResult(display, PresentOrValidate::Result::Presented);
+        return;
     }
 
     // Fallback to validate