commit 8629ed8c3ff6265fe414955e906f074be8eb5dae
author mashixing <mashixing@xiaomi.com> Tue Nov 28 16:55:22 2023 +0800
committer ShiXing Ma <mashixing@xiaomi.corp-partner.google.com> Thu Nov 30 07:55:37 2023 +0000
tree f8537b4be30199feb9f8e419b98592cb3a91f404
parent 55c8a818ecd3987bd1ccef1a8bc6066ae58a7a13

Fix assertion checks in libs/hwui/Mesh.cpp

The original code contained assertions that checked `fICount` and `fIOffset`, which seem to be mistakenly copied from the Skia codebase. This commit corrects the assertions to properly reflect the variables used in the AOSP codebase, updating them to `mIndexCount` and `mIndexOffset` as intended. The correction ensures accurate runtime checks for the expected conditions and prevents compilation error due to undeclared identifier.

Bug: 313566801

Change-Id: I1432598be14a6960d0dbe373e1ac997f5f5f8ee6
Signed-off-by: Shixing Ma <mashixing@xiaomi.corp-partner.google.com>

libs/hwui/Mesh.cpp

diff --git a/libs/hwui/Mesh.cpp b/libs/hwui/Mesh.cpp
index e59bc95..37a7d74 100644
--- a/libs/hwui/Mesh.cpp
+++ b/libs/hwui/Mesh.cpp
@@ -90,8 +90,8 @@
             FAIL_MESH_VALIDATE("%s mode requires at least %zu vertices but vertex count is %zu.",
                                modeToStr(meshMode), min_vcount_for_mode(meshMode), mVertexCount);
         }
-        SkASSERT(!fICount);
-        SkASSERT(!fIOffset);
+        LOG_ALWAYS_FATAL_IF(mIndexCount != 0);
+        LOG_ALWAYS_FATAL_IF(mIndexOffset != 0);
     }
 
     if (!sm.ok()) {