commit 008238ef439d7cdd8b227863261f5ed7086aba52
author Sunny Goyal <sunnygoyal@google.com> Mon Jun 27 11:25:20 2022 -0700
committer Sunny Goyal <sunnygoyal@google.com> Mon Jun 27 11:25:57 2022 -0700
tree fb3ee4428b2497042bee57152a0251b9e3494f8f
parent 4a0129fa5f8f916988f74503801f9a9537545d4e

Fixing widget size cache loop end-boundary

Bug: 236961658
Test: Verified on device
Change-Id: Ied629cd875a01abc45a7d055edfde5e156de53ea

src/com/android/launcher3/widget/picker/WidgetsRecyclerView.java

diff --git a/src/com/android/launcher3/widget/picker/WidgetsRecyclerView.java b/src/com/android/launcher3/widget/picker/WidgetsRecyclerView.java
index 4c0e0d5..35fa7a4 100644
--- a/src/com/android/launcher3/widget/picker/WidgetsRecyclerView.java
+++ b/src/com/android/launcher3/widget/picker/WidgetsRecyclerView.java
@@ -173,11 +173,12 @@
     @Override
     protected int getItemsHeight(int untilIndex) {
         // Initialize cache
-        int childCount = Math.min(getChildCount(), getAdapter().getItemCount());
+        int childCount = getChildCount();
         int startPosition;
         if (childCount > 0
                 && ((startPosition = getChildAdapterPosition(getChildAt(0))) != NO_POSITION)) {
-            for (int i = 0; i < childCount; i++) {
+            int loopCount = Math.min(getChildCount(), getAdapter().getItemCount() - startPosition);
+            for (int i = 0; i < loopCount; i++) {
                 mCachedSizes.put(
                         mAdapter.getItemViewType(startPosition + i),
                         getChildAt(i).getMeasuredHeight());