Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 2380d02e authored by Ruben Brunk's avatar Ruben Brunk
Browse files

Added locking to ImageLoader.

Bug: 7471850
Change-Id: Ifbaf6e3547fad546a6faa94694bc7d3c7bb4711c
parent 6a94446d
Loading
Loading
Loading
Loading
+13 −0
Original line number Diff line number Diff line
@@ -50,6 +50,7 @@ import java.io.FileNotFoundException;
import java.io.IOException;
import java.io.InputStream;
import java.util.Vector;
import java.util.concurrent.locks.ReentrantLock;

public class ImageLoader {

@@ -83,6 +84,8 @@ public class ImageLoader {
    private Rect mOriginalBounds = null;
    private static int mZoomOrientation = ORI_NORMAL;

    private ReentrantLock mLoadingLock = new ReentrantLock();

    public ImageLoader(FilterShowActivity activity, Context context) {
        mActivity = activity;
        mContext = context;
@@ -99,6 +102,7 @@ public class ImageLoader {
    }

    public void loadBitmap(Uri uri,int size) {
        mLoadingLock.lock();
        mUri = uri;
        mOrientation = getOrientation(mContext, uri);
        mOriginalBitmapSmall = loadScaledBitmap(uri, 160);
@@ -108,6 +112,7 @@ public class ImageLoader {
        }
        mOriginalBitmapLarge = loadScaledBitmap(uri, size);
        updateBitmaps();
        mLoadingLock.unlock();
    }

    public Uri getUri() {
@@ -308,10 +313,12 @@ public class ImageLoader {
    }

    public void addListener(ImageShow imageShow) {
        mLoadingLock.lock();
        if (!mListeners.contains(imageShow)) {
            mListeners.add(imageShow);
        }
        mHiresCache.addObserver(imageShow);
        mLoadingLock.unlock();
    }

    private void warnListeners() {
@@ -333,6 +340,7 @@ public class ImageLoader {
    // move this to a background thread.
    public Bitmap getScaleOneImageForPreset(ImageShow caller, ImagePreset imagePreset, Rect bounds,
            boolean force) {
        mLoadingLock.lock();
        Bitmap bmp = mZoomCache.getImage(imagePreset, bounds);
        if (force || bmp == null) {
            bmp = loadRegionBitmap(mUri, bounds);
@@ -347,12 +355,14 @@ public class ImageLoader {
                return bmp2;
            }
        }
        mLoadingLock.unlock();
        return bmp;
    }

    // Caching method
    public Bitmap getImageForPreset(ImageShow caller, ImagePreset imagePreset,
            boolean hiRes) {
        mLoadingLock.lock();
        if (mOriginalBitmapSmall == null) {
            return null;
        }
@@ -377,13 +387,16 @@ public class ImageLoader {
                mCache.addObserver(caller);
            }
        }
        mLoadingLock.unlock();
        return filteredImage;
    }

    public void resetImageForPreset(ImagePreset imagePreset, ImageShow caller) {
        mLoadingLock.lock();
        mHiresCache.reset(imagePreset);
        mCache.reset(imagePreset);
        mZoomCache.reset(imagePreset);
        mLoadingLock.unlock();
    }

    public void saveImage(ImagePreset preset, final FilterShowActivity filterShowActivity,