Show the image header for the Context Menu

chrome/android/java/src/org/chromium/chrome/browser/contextmenu/TabularContextMenuUi.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.contextmenu;
 
 import android.app.Activity;
 import android.app.Dialog;
 import android.content.DialogInterface;
+import android.content.res.Resources;
+import android.graphics.Bitmap;
+import android.graphics.Canvas;
+import android.graphics.Shader;
+import android.graphics.drawable.BitmapDrawable;
+import android.graphics.drawable.Drawable;
 import android.support.design.widget.TabLayout;
 import android.support.v4.view.ViewPager;
 import android.support.v7.app.AlertDialog;
 import android.text.TextUtils;
 import android.util.Pair;
 import android.view.LayoutInflater;
 import android.view.View;
 import android.view.ViewGroup;
 import android.widget.AdapterView;
 import android.widget.BaseAdapter;
+import android.widget.ImageView;
 import android.widget.ListView;
 import android.widget.TextView;
 
 import org.chromium.base.ApiCompatibilityUtils;
 import org.chromium.base.Callback;
 import org.chromium.base.VisibleForTesting;
 import org.chromium.chrome.R;
 
 import java.util.ArrayList;
 import java.util.List;
 
 /**
  * A custom dialog that separates each group into separate tabs. It uses a dialog instead.
  */
 public class TabularContextMenuUi implements ContextMenuUi, AdapterView.OnItemClickListener {
     private Dialog mDialog;
     private Callback<Integer> mCallback;
     private int mMenuItemHeight;
+    private ContextMenuHelper mContextMenuHelper;
+
+    /**
+     * A context menu that sperates types by tabs.
+     * @param contextMenuHelper This is used to retrieve the thumbnail natively from the Context

[Theresa, 2017/03/27 18:28:26]

nit: The {@link ContextMenuHelper} used to retrieve the thumbnail from native.

Or something like that.

[JJ, 2017/03/27 20:47:12]

Done.

+     *                          Menu Helper.
+     */
+    TabularContextMenuUi(ContextMenuHelper contextMenuHelper) {
+        mContextMenuHelper = contextMenuHelper;
+    }
 
     @Override
     public void displayMenu(Activity activity, ContextMenuParams params,
             List<Pair<Integer, List<ContextMenuItem>>> items, Callback<Integer> onItemClicked,
             final Runnable onMenuShown, final Runnable onMenuClosed) {
         mCallback = onItemClicked;
         mDialog = createDialog(activity, params, items);
         mDialog.getWindow().setBackgroundDrawable(ApiCompatibilityUtils.getDrawable(
                 activity.getResources(), R.drawable.context_menu_bg));
 
@@ skipping 27 matching lines @@
         Dialog dialog = new Dialog(activity);
         dialog.setContentView(createPagerView(activity, params, itemGroups));
         return dialog;
     }
 
     /**
      * Creates the view of a context menu. Based off the Context Type, it'll adjust the list of
      * items and display only the ones that'll be on that specific group.
      * @param activity Used to get the resources of an item.
      * @param params used to create the header text.
-     * @param items A set of Items to display in a context menu. Filtered based off the type.
+     * @param itemGroup A set of Items to display in a context menu. Filtered based off the type.
      * @return Returns a filled LinearLayout with all the context menu items
      */
     @VisibleForTesting
     ViewGroup createSingleContextView(Activity activity, ContextMenuParams params,
-            List<ContextMenuItem> items, int maxCount) {
+            Pair<Integer, List<ContextMenuItem>> itemGroup, int maxCount) {

[Theresa, 2017/03/27 18:28:26]

Can the title string id and list of items be separate parameters? Also, can the
new param be a boolean used to control whether the context menu is for an image
so that we can check a boolean below rather than checking if the title ==
R.string.contextmenu_image_title?

[JJ, 2017/03/27 20:47:12]

 Yeah we can do that.

+        List<ContextMenuItem> items = itemGroup.second;
+        int title = itemGroup.first;
         ViewGroup baseLayout = (ViewGroup) LayoutInflater.from(activity).inflate(
                 R.layout.tabular_context_menu_page, null);
         ListView listView = (ListView) baseLayout.findViewById(R.id.selectable_items);
-        displayHeaderIfVisibleItems(params, baseLayout);
+
+        if (title == R.string.contextmenu_image_title) {
+            displayImageHeader(baseLayout, params, activity.getResources());
+        } else {
+            displayHeaderIfVisibleItems(params, baseLayout);
+        }
 
         // Set the list adapter and get the height to display it appropriately in a dialog.
         TabularContextMenuListAdapter listAdapter =
                 new TabularContextMenuListAdapter(items, activity);
         ViewGroup.LayoutParams layoutParams = listView.getLayoutParams();
         layoutParams.height = measureApproximateListViewHeight(listView, listAdapter, maxCount);
         listView.setLayoutParams(layoutParams);
         listView.setAdapter(listAdapter);
         listView.setOnItemClickListener(this);
 
         return baseLayout;
     }
 
     private void displayHeaderIfVisibleItems(ContextMenuParams params, ViewGroup baseLayout) {
         String headerText = ChromeContextMenuPopulator.createHeaderText(params);
         TextView headerTextView = (TextView) baseLayout.findViewById(R.id.context_header_text);
         if (TextUtils.isEmpty(headerText)) {
+            baseLayout.findViewById(R.id.context_header_layout).setVisibility(View.GONE);
             headerTextView.setVisibility(View.GONE);
             baseLayout.findViewById(R.id.context_divider).setVisibility(View.GONE);
             return;
         }
         headerTextView.setVisibility(View.VISIBLE);
         headerTextView.setText(headerText);
     }
 
+    private void displayImageHeader(
+            ViewGroup baseLayout, ContextMenuParams params, Resources resources) {
+        final ImageView imageView = (ImageView) baseLayout.findViewById(R.id.context_header_image);
+        TextView headerTextView = (TextView) baseLayout.findViewById(R.id.context_header_text);
+        String headerText = params.getTitleText();
+        if (!TextUtils.isEmpty(headerText)) {
+            headerTextView.setText(headerText);
+            headerTextView.setVisibility(View.VISIBLE);
+        } else {
+            displayHeaderIfVisibleItems(params, baseLayout);
+            // The image is still around so disabling the header is uneccessary
+            baseLayout.findViewById(R.id.context_header_layout).setVisibility(View.VISIBLE);

[Theresa, 2017/03/27 18:28:26]

I think this could be simplified:

displayHeaderIfVisibleItems(...)

// #displayHeaderIfVisibleItems() sets these two views to GONE if the header
// text is empty, but they should still be visible because we have an image
// to display.
baseLayout.findViewById(R.id.context_header_layout).setVisibility(View.VISIBLE);
baseLayout.findViewById(R.id.context_divider).setVisibility(View.VISIBLE);

[JJ, 2017/03/27 20:47:12]

Done.

[Theresa, 2017/03/28 17:23:08]

I meant the whole top of the method

private displayImageHeader(...) {
    displayHeaderIfVisibleItems(params, baseLayout);

    // #displayHeaderIfVisibleItems() sets these two views to GONE if the header
    // text is empty, but they should still be visible because we have an image
    // to display.
   
baseLayout.findViewById(R.id.context_header_layout).setVisibility(View.VISIBLE);
    baseLayout.findViewById(R.id.context_divider).setVisibility(View.VISIBLE);

    final ImageView imageView = (ImageView)
baseLayout.findViewById(R.id.context_header_image);
    setBackgroundForImageView(imageView, resources);
    ....
}

[JJ, 2017/03/28 23:19:20]

Oh whoops, apologies.

+            baseLayout.findViewById(R.id.context_divider).setVisibility(View.VISIBLE);
+        }
+        setBackgroundForImageView(imageView, resources);
+
+        mContextMenuHelper.setOnThumbnailReceivedListener(
+                new ContextMenuHelper.OnThumbnailReceivedListener() {
+                    @Override
+                    public void onThumbnailReceived(Bitmap bitmap) {
+                        imageView.setImageBitmap(bitmap);
+                    }
+                });
+        mContextMenuHelper.getThumbnail();
+    }
+
+    private void setBackgroundForImageView(ImageView imageView, Resources resources) {
+        Bitmap bitmap = null;
+        Drawable drawable =
+                ApiCompatibilityUtils.getDrawable(resources, R.drawable.checkerboard_background);
+        bitmap = Bitmap.createBitmap(drawable.getIntrinsicWidth(), drawable.getIntrinsicHeight(),
+                Bitmap.Config.ARGB_8888);
+        Canvas canvas = new Canvas(bitmap);
+        drawable.setBounds(0, 0, canvas.getWidth(), canvas.getHeight());
+        drawable.draw(canvas);
+        BitmapDrawable bm = new BitmapDrawable(resources, bitmap);
+        bm.setTileModeXY(Shader.TileMode.REPEAT, Shader.TileMode.REPEAT);
+        imageView.setVisibility(View.VISIBLE);
+        imageView.setBackground(bm);

[Theresa, 2017/03/27 18:28:26]

This method seems more complex than needed. I think you can create a background
XML with android:tileMode=repeat and set that as imageView background rather
than programatically constructing the background.

http://stackoverflow.com/questions/9279012/android-image-background-repeat

[JJ, 2017/03/27 20:47:12]

haha we tried. However it was deemed impossible if the drawable is an xml so we
have to do it pragmatically. 

+    }
+
     /**
      * To save time measuring the height, this method gets an item if the height has not been
      * previous measured and multiplies it by count of the total amount of items. It is fine if the
      * height too small as the ListView will scroll through the other values.
      * @param listView The ListView to measure the surrounding padding.
      * @param listAdapter The adapter which contains the items within the list.
      * @return Returns the combined height of the padding of the ListView and the approximate height
      *         of the ListView based off the an item.
      */
     private int measureApproximateListViewHeight(
@@ skipping 19 matching lines @@
     @VisibleForTesting
     View createPagerView(Activity activity, ContextMenuParams params,
             List<Pair<Integer, List<ContextMenuItem>>> itemGroups) {
         View view = LayoutInflater.from(activity).inflate(R.layout.tabular_context_menu, null);
 
         List<Pair<String, ViewGroup>> viewGroups = new ArrayList<>();
         int maxCount = 0;
         for (Pair<Integer, List<ContextMenuItem>> itemGroup : itemGroups) {
             maxCount = Math.max(maxCount, itemGroup.second.size());
             viewGroups.add(new Pair<>(activity.getString(itemGroup.first),
-                    createSingleContextView(activity, params, itemGroup.second, maxCount)));
+                    createSingleContextView(activity, params, itemGroup, maxCount)));
         }
         TabularContextMenuViewPager pager =
                 (TabularContextMenuViewPager) view.findViewById(R.id.custom_pager);
         pager.setAdapter(new TabularContextMenuPagerAdapter(viewGroups));
 
         TabLayout tabLayout = (TabLayout) view.findViewById(R.id.tab_layout);
         if (itemGroups.size() <= 1) {
             tabLayout.setVisibility(View.GONE);
         }
         tabLayout.setupWithViewPager((ViewPager) view.findViewById(R.id.custom_pager));
 
         return view;
     }
 
     @Override
     public void onItemClick(AdapterView<?> adapterView, View view, int position, long id) {
         mDialog.dismiss();
         mCallback.onResult((int) id);
     }
 }