From d0858f3862adb75523234838b450f606483effb7 Mon Sep 17 00:00:00 2001
From: M66B <M66B@users.noreply.github.com>
Date: Wed, 19 Aug 2020 11:17:01 +0200
Subject: [PATCH] Use view handler for boundary callbacks

---
 .../java/eu/faircode/email/BoundaryCallbackMessages.java    | 6 ++----
 app/src/main/java/eu/faircode/email/FragmentMessages.java   | 2 +-
 app/src/main/java/eu/faircode/email/ViewModelMessages.java  | 5 +++--
 3 files changed, 6 insertions(+), 7 deletions(-)

diff --git a/app/src/main/java/eu/faircode/email/BoundaryCallbackMessages.java b/app/src/main/java/eu/faircode/email/BoundaryCallbackMessages.java
index e2597a2c61..b4f18a5bdb 100644
--- a/app/src/main/java/eu/faircode/email/BoundaryCallbackMessages.java
+++ b/app/src/main/java/eu/faircode/email/BoundaryCallbackMessages.java
@@ -85,9 +85,7 @@ public class BoundaryCallbackMessages extends PagedList.BoundaryCallback<TupleMe
     private int pageSize;
 
     private IBoundaryCallbackMessages intf;
-
     private Handler handler;
-
     private State state;
 
     private static final int SEARCH_LIMIT_DEVICE = 1000;
@@ -111,9 +109,9 @@ public class BoundaryCallbackMessages extends PagedList.BoundaryCallback<TupleMe
         this.pageSize = pageSize;
     }
 
-    void setCallback(IBoundaryCallbackMessages intf) {
-        this.handler = new Handler();
+    void setCallback(IBoundaryCallbackMessages intf, Handler handler) {
         this.intf = intf;
+        this.handler = handler;
         this.state = new State();
     }
 
diff --git a/app/src/main/java/eu/faircode/email/FragmentMessages.java b/app/src/main/java/eu/faircode/email/FragmentMessages.java
index 95f705c160..87b4e85f1d 100644
--- a/app/src/main/java/eu/faircode/email/FragmentMessages.java
+++ b/app/src/main/java/eu/faircode/email/FragmentMessages.java
@@ -4331,7 +4331,7 @@ public class FragmentMessages extends FragmentBase implements SharedPreferences.
                 getContext(), getViewLifecycleOwner(),
                 viewType, type, account, folder, thread, id, filter_archive, criteria, server);
 
-        vmodel.setCallback(getViewLifecycleOwner(), callback);
+        vmodel.setCallback(getViewLifecycleOwner(), callback, view.getHandler());
         vmodel.setObserver(getViewLifecycleOwner(), observer);
     }
 
diff --git a/app/src/main/java/eu/faircode/email/ViewModelMessages.java b/app/src/main/java/eu/faircode/email/ViewModelMessages.java
index 87b0da0053..73a53a3539 100644
--- a/app/src/main/java/eu/faircode/email/ViewModelMessages.java
+++ b/app/src/main/java/eu/faircode/email/ViewModelMessages.java
@@ -23,6 +23,7 @@ import android.content.Context;
 import android.content.SharedPreferences;
 import android.os.Build;
 import android.os.Bundle;
+import android.os.Handler;
 import android.text.TextUtils;
 import android.util.Pair;
 
@@ -516,9 +517,9 @@ public class ViewModelMessages extends ViewModel {
             this.boundary = boundary;
         }
 
-        void setCallback(LifecycleOwner owner, BoundaryCallbackMessages.IBoundaryCallbackMessages callback) {
+        void setCallback(LifecycleOwner owner, BoundaryCallbackMessages.IBoundaryCallbackMessages callback, Handler handler) {
             if (boundary != null) {
-                boundary.setCallback(callback);
+                boundary.setCallback(callback, handler);
 
                 owner.getLifecycle().addObserver(new LifecycleObserver() {
                     @OnLifecycleEvent(Lifecycle.Event.ON_DESTROY)