Merge "Remove content preview from FileContentPreviewUI" into udc-dev am: a94ea8494e

Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/modules/IntentResolver/+/22523189

Change-Id: Ieff93a73aceaea80be28df154c9cb0cd93e9952c
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

2 files changed, 14 insertions, 49 deletions

diff --git a/java/src/com/android/intentresolver/contentpreview/ChooserContentPreviewUi.java b/java/src/com/android/intentresolver/contentpreview/ChooserContentPreviewUi.java
index 318aa627..56027a16 100644
--- a/java/src/com/android/intentresolver/contentpreview/ChooserContentPreviewUi.java
+++ b/java/src/com/android/intentresolver/contentpreview/ChooserContentPreviewUi.java
@@ -155,7 +155,6 @@ public final class ChooserContentPreviewUi {
             return new FileContentPreviewUi(
                     files,
                     actionFactory,
-                    imageLoader,
                     headlineGenerator);
         }
         return new UnifiedContentPreviewUi(
diff --git a/java/src/com/android/intentresolver/contentpreview/FileContentPreviewUi.java b/java/src/com/android/intentresolver/contentpreview/FileContentPreviewUi.java
index e814eb12..e9d65eed 100644
--- a/java/src/com/android/intentresolver/contentpreview/FileContentPreviewUi.java
+++ b/java/src/com/android/intentresolver/contentpreview/FileContentPreviewUi.java
@@ -25,7 +25,6 @@ import android.view.ViewGroup;
 import android.widget.ImageView;
 import android.widget.TextView;
 
-import com.android.intentresolver.ImageLoader;
 import com.android.intentresolver.R;
 import com.android.intentresolver.widget.ActionRow;
 
@@ -40,17 +39,14 @@ class FileContentPreviewUi extends ContentPreviewUi {
 
     private final List<FileInfo> mFiles;
     private final ChooserContentPreviewUi.ActionFactory mActionFactory;
-    private final ImageLoader mImageLoader;
     private final HeadlineGenerator mHeadlineGenerator;
 
     FileContentPreviewUi(
             List<FileInfo> files,
             ChooserContentPreviewUi.ActionFactory actionFactory,
-            ImageLoader imageLoader,
             HeadlineGenerator headlineGenerator) {
         mFiles = files;
         mActionFactory = actionFactory;
-        mImageLoader = imageLoader;
         mHeadlineGenerator = headlineGenerator;
     }
 
@@ -82,30 +78,27 @@ class FileContentPreviewUi extends ContentPreviewUi {
             return contentPreviewLayout;
         }
 
+        FileInfo fileInfo = mFiles.get(0);
+        final CharSequence fileName;
+        final int iconId;
         if (uriCount == 1) {
-            loadFileUriIntoView(mFiles.get(0), contentPreviewLayout, mImageLoader);
+            fileName = fileInfo.getName();
+            iconId = R.drawable.chooser_file_generic;
         } else {
-            FileInfo fileInfo = mFiles.get(0);
             int remUriCount = uriCount - 1;
             Map<String, Object> arguments = new HashMap<>();
             arguments.put(PLURALS_COUNT, remUriCount);
             arguments.put(PLURALS_FILE_NAME, fileInfo.getName());
-            String fileName =
-                    PluralsMessageFormatter.format(resources, arguments, R.string.file_count);
-
-            TextView fileNameView = contentPreviewLayout.findViewById(
-                    com.android.internal.R.id.content_preview_filename);
-            fileNameView.setText(fileName);
-
-            View thumbnailView = contentPreviewLayout.findViewById(
-                    com.android.internal.R.id.content_preview_file_thumbnail);
-            thumbnailView.setVisibility(View.GONE);
-
-            ImageView fileIconView = contentPreviewLayout.findViewById(
-                    com.android.internal.R.id.content_preview_file_icon);
-            fileIconView.setVisibility(View.VISIBLE);
-            fileIconView.setImageResource(R.drawable.ic_file_copy);
+            fileName = PluralsMessageFormatter.format(resources, arguments, R.string.file_count);
+            iconId = R.drawable.ic_file_copy;
         }
+        TextView fileNameView = contentPreviewLayout.findViewById(
+                com.android.internal.R.id.content_preview_filename);
+        fileNameView.setText(fileName);
+
+        ImageView fileIconView = contentPreviewLayout.findViewById(
+                com.android.internal.R.id.content_preview_file_icon);
+        fileIconView.setImageResource(iconId);
 
         final ActionRow actionRow =
                 contentPreviewLayout.findViewById(com.android.internal.R.id.chooser_action_row);
@@ -127,31 +120,4 @@ class FileContentPreviewUi extends ContentPreviewUi {
         }
         return actions;
     }
-
-    private static void loadFileUriIntoView(
-            final FileInfo fileInfo,
-            final View parent,
-            final ImageLoader imageLoader) {
-        TextView fileNameView = parent.findViewById(
-                com.android.internal.R.id.content_preview_filename);
-        fileNameView.setText(fileInfo.getName());
-
-        if (fileInfo.getPreviewUri() != null) {
-            imageLoader.loadImage(
-                    fileInfo.getPreviewUri(),
-                    (bitmap) -> updateViewWithImage(
-                            parent.findViewById(
-                                    com.android.internal.R.id.content_preview_file_thumbnail),
-                            bitmap));
-        } else {
-            View thumbnailView = parent.findViewById(
-                    com.android.internal.R.id.content_preview_file_thumbnail);
-            thumbnailView.setVisibility(View.GONE);
-
-            ImageView fileIconView = parent.findViewById(
-                    com.android.internal.R.id.content_preview_file_icon);
-            fileIconView.setVisibility(View.VISIBLE);
-            fileIconView.setImageResource(R.drawable.chooser_file_generic);
-        }
-    }
 }