Photo Picker Dialog: Add caching for the decoded images.

chrome/android/java/src/org/chromium/chrome/browser/photo_picker/PickerBitmapViewHolder.java

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.chrome.browser.photo_picker;
 
 import android.graphics.Bitmap;
+import android.os.AsyncTask;
 import android.support.v7.widget.RecyclerView.ViewHolder;
 import android.text.TextUtils;
 
 import java.util.List;
 
 /**
  * Holds on to a {@link PickerBitmapView} that displays information about a picker bitmap.
  */
 public class PickerBitmapViewHolder
         extends ViewHolder implements DecoderServiceHost.ImageDecodedCallback {
+    // The size (in dp) of the low-res thumbnails.
+    private static final int GRAINY_THUMBNAIL_SIZE_DP = 12;

[Theresa, 2017/05/08 19:34:42]

Nit: Put this in dimens.xml and use Resources.getDimensionPixelSize() to
retrieve the value in px.

[Finnur, 2017/05/09 11:53:56]

Done.

+
     // Our parent category.
     private PickerCategoryView mCategoryView;
 
     // The bitmap view we are holding on to.
     private final PickerBitmapView mItemView;
 
+    // The screen density.
+    private final float mDensity;
+
     // The request we are showing the bitmap for.
     private PickerBitmap mBitmapDetails;
 
     /**
      * The PickerBitmapViewHolder.
      * @param itemView The {@link PickerBitmapView} view for showing the image.
      */
     public PickerBitmapViewHolder(PickerBitmapView itemView) {
         super(itemView);
         mItemView = itemView;
+        mDensity = itemView.getContext().getResources().getDisplayMetrics().density;
     }
 
     // DecoderServiceHost.ImageDecodedCallback
 
     @Override
     public void imageDecodedCallback(String filePath, Bitmap bitmap) {
         if (bitmap == null || bitmap.getWidth() == 0 || bitmap.getHeight() == 0) {
             return;
         }
 
+        if (mCategoryView.getHighResBitmaps().get(filePath) == null) {
+            mCategoryView.getHighResBitmaps().put(filePath, bitmap);
+        }
+
+        if (mCategoryView.getLowResBitmaps().get(filePath) == null) {
+            new BitmapScalerTask(mCategoryView.getLowResBitmaps(), filePath,
+                    (int) (GRAINY_THUMBNAIL_SIZE_DP * mDensity))
+                    .executeOnExecutor(AsyncTask.THREAD_POOL_EXECUTOR, bitmap);
+        }
+
         if (!TextUtils.equals(mBitmapDetails.getFilePath(), filePath)) {
             return;
         }
 
         mItemView.setThumbnailBitmap(bitmap);
     }
 
     /**
      * Display a single item from |position| in the PickerCategoryView.
      * @param categoryView The PickerCategoryView to use to fetch the image.
      * @param position The position of the item to fetch.
      */
     public void displayItem(PickerCategoryView categoryView, int position) {
         mCategoryView = categoryView;
 
         List<PickerBitmap> pickerBitmaps = mCategoryView.getPickerBitmaps();
         mBitmapDetails = pickerBitmaps.get(position);
 
         if (mBitmapDetails.type() == PickerBitmap.CAMERA
                 || mBitmapDetails.type() == PickerBitmap.GALLERY) {
             mItemView.initialize(mBitmapDetails, null, false);
             return;
         }
 
-        // TODO(finnur): Use cached image, if available.
-
-        mItemView.initialize(mBitmapDetails, null, true);
+        String filePath = mBitmapDetails.getFilePath();
+        Bitmap original = mCategoryView.getHighResBitmaps().get(filePath);
+        if (original != null) {
+            mItemView.initialize(mBitmapDetails, original, false);
+            return;
+        }
 
         int size = mCategoryView.getImageSize();
-        mCategoryView.getDecoderServiceHost().decodeImage(mBitmapDetails.getFilePath(), size, this);
+        Bitmap placeholder = mCategoryView.getLowResBitmaps().get(filePath);
+        if (placeholder != null) {
+            // Scaling the image up takes between 3-4 ms on average (Nexus 6 phone debug build).

[Theresa, 2017/05/08 19:34:42]

nit: This type of information is likely to get out of date. I think it would be
useful to include a bug # here that has a comment with the stats and relevant
information (e.g. device type, OS version, Chrome version), so that we can look
back a couple of years from now and know what analysis was done.

[Finnur, 2017/05/09 11:53:56]

Done.

+            placeholder = BitmapUtils.scale(placeholder, size, false);
+            mItemView.initialize(mBitmapDetails, placeholder, true);
+        } else {
+            mItemView.initialize(mBitmapDetails, null, true);
+        }
+
+        mCategoryView.getDecoderServiceHost().decodeImage(filePath, size, this);
     }
 
     /**
      * Returns the file path of the current request.
      */
     public String getFilePath() {
         return mBitmapDetails == null ? null : mBitmapDetails.getFilePath();
     }
 }