From 2a0f99067ab5583dea05d497b10385c704dc5627 Mon Sep 17 00:00:00 2001
From: emanuele-f <black.silver@hotmail.it>
Date: Sat, 13 Aug 2022 11:56:32 +0200
Subject: [PATCH] Prevent ANR while adding payload chunks

---
 .../remote_capture/adapters/PayloadAdapter.java        | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/app/src/main/java/com/emanuelef/remote_capture/adapters/PayloadAdapter.java b/app/src/main/java/com/emanuelef/remote_capture/adapters/PayloadAdapter.java
index fa8a1469..e5da542c 100644
--- a/app/src/main/java/com/emanuelef/remote_capture/adapters/PayloadAdapter.java
+++ b/app/src/main/java/com/emanuelef/remote_capture/adapters/PayloadAdapter.java
@@ -274,6 +274,7 @@ public class PayloadAdapter extends RecyclerView.Adapter<PayloadAdapter.PayloadV
 
     @Override
     public int getItemCount() {
+        // TODO remove loop, as it can generate ANRs on high number of elements
         int count = 0;
 
         for(AdapterChunk aChunk: mChunks)
@@ -336,6 +337,8 @@ public class PayloadAdapter extends RecyclerView.Adapter<PayloadAdapter.PayloadV
     }
 
     public void handleChunksAdded(int tot_chunks) {
+        int items_count = -1;
+
         for(int i = mHandledChunks; i<tot_chunks; i++) {
             PayloadChunk chunk = mConn.getPayloadChunk(i);
             if(chunk == null)
@@ -352,9 +355,12 @@ public class PayloadAdapter extends RecyclerView.Adapter<PayloadAdapter.PayloadV
                 else
                     mHttpRes.handleChunk(chunk);
             } else {
-                int insert_pos = getItemCount();
+                // TODO remove temporary caching due to slow getItemCount
+                if(items_count == -1)
+                    items_count = getItemCount();
                 mChunks.add(new AdapterChunk(chunk, mChunks.size()));
-                notifyItemInserted(insert_pos);
+                notifyItemInserted(items_count);
+                items_count += 1;
             }
         }