Cleanup

app/src/main/java/eu/faircode/email/ImageHelper.java

@@ -39,7 +39,6 @@ import android.graphics.drawable.Drawable;
 import android.graphics.drawable.LevelListDrawable;
 import android.os.Build;
 import android.os.Handler;
-import android.os.Looper;
 import android.text.TextUtils;
 import android.util.Base64;
 import android.util.DisplayMetrics;
@@ -392,51 +391,43 @@ class ImageHelper {
     }
 
     private static void fitDrawable(final Drawable d, final AnnotatedSource a, final View view) {
-        if (Looper.getMainLooper().getThread() == Thread.currentThread()) {
-            Log.w("fitDrawable UI thread");
-            _fitDrawble(d, a, view);
-        } else {
-            Semaphore semaphore = new Semaphore(0);
+        Semaphore semaphore = new Semaphore(0);
+
+        view.post(new Runnable() {
+            @Override
+            public void run() {
+                Rect bounds = d.getBounds();
+                int w = bounds.width();
+                int h = bounds.height();
+
+                if (a.width == 0 && a.height != 0)
+                    a.width = Math.round(a.height * w / (float) h);
+                if (a.height == 0 && a.width != 0)
+                    a.height = Math.round(a.width * h / (float) w);
+
+                if (a.width != 0 && a.height != 0) {
+                    w = Helper.dp2pixels(view.getContext(), a.width);
+                    h = Helper.dp2pixels(view.getContext(), a.height);
+                    d.setBounds(0, 0, w, h);
+                }
 
-            view.post(new Runnable() {
-                @Override
-                public void run() {
-                    _fitDrawble(d, a, view);
-                    semaphore.release();
+                float width = view.getWidth();
+                if (w > width) {
+                    float scale = width / w;
+                    w = Math.round(w * scale);
+                    h = Math.round(h * scale);
+                    d.setBounds(0, 0, w, h);
                 }
-            });
 
-            try {
-                if (!semaphore.tryAcquire(FIT_DRAWABLE_TIMEOUT, TimeUnit.MILLISECONDS))
-                    Log.e("fitDrawable failed timeout=" + FIT_DRAWABLE_TIMEOUT);
-            } catch (InterruptedException ex) {
-                Log.w(ex);
+                semaphore.release();
             }
-        }
-    }
+        });
 
-    private static void _fitDrawble(Drawable d, AnnotatedSource a, View view) {
-        Rect bounds = d.getBounds();
-        int w = bounds.width();
-        int h = bounds.height();
-
-        if (a.width == 0 && a.height != 0)
-            a.width = Math.round(a.height * w / (float) h);
-        if (a.height == 0 && a.width != 0)
-            a.height = Math.round(a.width * h / (float) w);
-
-        if (a.width != 0 && a.height != 0) {
-            w = Helper.dp2pixels(view.getContext(), a.width);
-            h = Helper.dp2pixels(view.getContext(), a.height);
-            d.setBounds(0, 0, w, h);
-        }
-
-        float width = view.getWidth();
-        if (w > width) {
-            float scale = width / w;
-            w = Math.round(w * scale);
-            h = Math.round(h * scale);
-            d.setBounds(0, 0, w, h);
+        try {
+            if (!semaphore.tryAcquire(FIT_DRAWABLE_TIMEOUT, TimeUnit.MILLISECONDS))
+                Log.e("fitDrawable failed timeout=" + FIT_DRAWABLE_TIMEOUT);
+        } catch (InterruptedException ex) {
+            Log.w(ex);
         }
     }