diff options
author | Sascha Haeberling <haeberling@google.com> | 2013-08-22 16:07:31 -0700 |
---|---|---|
committer | Android Git Automerger <android-git-automerger@android.com> | 2013-08-22 16:07:31 -0700 |
commit | 779160b3411588dfd018974f83414996dfb97665 (patch) | |
tree | b48c8252a41cdb02141e0148196b022280f9a0fb | |
parent | d567d7d51fc84b9abb9bb5b3c6bf46855234a480 (diff) | |
parent | 00373f8a411042964f019a6355864a4c25fc2238 (diff) | |
download | android_packages_apps_Snap-779160b3411588dfd018974f83414996dfb97665.tar.gz android_packages_apps_Snap-779160b3411588dfd018974f83414996dfb97665.tar.bz2 android_packages_apps_Snap-779160b3411588dfd018974f83414996dfb97665.zip |
am 00373f8a: Merge "Speed up filmstrip by not decoding with default 3000px." into gb-ub-photos-carlsbad
* commit '00373f8a411042964f019a6355864a4c25fc2238':
Speed up filmstrip by not decoding with default 3000px.
-rw-r--r-- | src/com/android/camera/data/AbstractLocalDataAdapterWrapper.java | 1 | ||||
-rw-r--r-- | src/com/android/camera/data/LocalMediaData.java | 18 |
2 files changed, 10 insertions, 9 deletions
diff --git a/src/com/android/camera/data/AbstractLocalDataAdapterWrapper.java b/src/com/android/camera/data/AbstractLocalDataAdapterWrapper.java index 5df87f5a8..ef1e4258c 100644 --- a/src/com/android/camera/data/AbstractLocalDataAdapterWrapper.java +++ b/src/com/android/camera/data/AbstractLocalDataAdapterWrapper.java @@ -51,6 +51,7 @@ public abstract class AbstractLocalDataAdapterWrapper implements LocalDataAdapte public void suggestViewSizeBound(int w, int h) { mSuggestedWidth = w; mSuggestedHeight = h; + mAdapter.suggestViewSizeBound(w, h); } @Override diff --git a/src/com/android/camera/data/LocalMediaData.java b/src/com/android/camera/data/LocalMediaData.java index f80b4a02f..48db0df4c 100644 --- a/src/com/android/camera/data/LocalMediaData.java +++ b/src/com/android/camera/data/LocalMediaData.java @@ -426,20 +426,20 @@ public abstract class LocalMediaData implements LocalData { @Override protected Bitmap doInBackground(Void... v) { - BitmapFactory.Options opts = null; - Bitmap b; - int sample = 1; - while (mDecodeWidth * sample < width - || mDecodeHeight * sample < height) { - sample *= 2; + int sampleSize = 1; + if (width > mDecodeWidth || height > mDecodeHeight) { + int heightRatio = Math.round((float) height / (float) mDecodeHeight); + int widthRatio = Math.round((float) width / (float) mDecodeWidth); + sampleSize = heightRatio < widthRatio ? heightRatio : widthRatio; } - opts = new BitmapFactory.Options(); - opts.inSampleSize = sample; + + BitmapFactory.Options opts = new BitmapFactory.Options(); + opts.inSampleSize = sampleSize; opts.inTempStorage = DECODE_TEMP_STORAGE; if (isCancelled() || !isUsing()) { return null; } - b = BitmapFactory.decodeFile(path, opts); + Bitmap b = BitmapFactory.decodeFile(path, opts); if (orientation != 0) { if (isCancelled() || !isUsing()) { return null; |