Home shortcut should use touch icons when possible.

chrome/android/java/src/org/chromium/chrome/browser/BookmarkUtils.java

 // Copyright (c) 2012 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;
 
 import android.app.ActivityManager;
 import android.content.Context;
 import android.content.Intent;
 import android.graphics.Bitmap;
 import android.graphics.Canvas;
 import android.graphics.Color;
 import android.graphics.LinearGradient;
 import android.graphics.Paint;
 import android.graphics.Path;
+import android.graphics.PorterDuff;
+import android.graphics.PorterDuffXfermode;
 import android.graphics.Rect;
 import android.graphics.RectF;
 import android.graphics.drawable.BitmapDrawable;
 import android.graphics.drawable.Drawable;
 import android.net.Uri;
 import android.os.Build;
 import android.util.Log;
 
 import org.chromium.chrome.R;
 
 /**
  * Util class for bookmarks.
  */
 public class BookmarkUtils {
 
     // There is no public string defining this intent so if Home changes the value, we
     // have to update this string.
     private static final String INSTALL_SHORTCUT = "com.android.launcher.action.INSTALL_SHORTCUT";
     private static final int DEFAULT_RGB_VALUE = 145;
     private static final String TAG = "BookmarkUtils";
     public static final String REUSE_URL_MATCHING_TAB_ELSE_NEW_TAB =
             "REUSE_URL_MATCHING_TAB_ELSE_NEW_TAB";
+    private static final int INSET_DIMENSION_FOR_TOUCHICON = 1;
     private static final int SDK_VERSION_FOR_ACCESS_TO_METHODS = 15;
 
     /**
      * Creates an intent that will add a shortcut to the home screen.
      * @param context Context used to create the intent.
      * @param url Url of the bookmark.
      * @param title Title of the bookmark.
      * @param favicon Bookmark favicon.
      * @param rValue Red component of the dominant favicon color.
      * @param gValue Green component of the dominant favicon color.
@@ skipping 30 matching lines @@
      * @param favicon Bookmark favicon bitmap.
      * @param rValue Red component of the dominant favicon color.
      * @param gValue Green component of the dominant favicon color.
      * @param bValue Blue component of the dominant favicon color.
      * @return Bitmap Either the touch-icon or the newly created favicon.
      */
     private static Bitmap createIcon(Context context, Bitmap favicon, int rValue,
             int gValue, int bValue) {
         Bitmap bitmap = null;
         ActivityManager am = (ActivityManager) context.getSystemService(Context.ACTIVITY_SERVICE);
-        int iconDimension = am.getLauncherLargeIconSize();
-        int iconDensity = am.getLauncherLargeIconDensity();
+        final int iconDimension = am.getLauncherLargeIconSize();
+        final int iconDensity = am.getLauncherLargeIconDensity();
         try {
             bitmap = Bitmap.createBitmap(iconDimension, iconDimension, Bitmap.Config.ARGB_8888);
             Canvas canvas = new Canvas(bitmap);
             Rect iconBounds = new Rect(0, 0, bitmap.getWidth(), bitmap.getHeight());
             if (favicon == null) {
                 favicon = getBitmapFromResourceId(context, R.drawable.globe_favicon, iconDensity);
                 rValue = gValue = bValue = DEFAULT_RGB_VALUE;
             }
-            Bitmap icon = getIconBackground(context, iconDensity);
-            if (icon != null) {
-                canvas.drawBitmap(icon, null, iconBounds, new Paint(Paint.ANTI_ALIAS_FLAG));
+            final int smallestSide = iconDimension;
+            if (favicon.getWidth() >= smallestSide / 2 && favicon.getHeight() >= smallestSide / 2) {
+                drawTouchIconToCanvas(context, favicon, canvas, iconBounds);
+            } else {
+                Bitmap icon = getIconBackground(context, iconDensity);
+                if (icon != null) {
+                    canvas.drawBitmap(icon, null, iconBounds, new Paint(Paint.ANTI_ALIAS_FLAG));
+                }
+                drawFaviconToCanvas(context, favicon, canvas, iconBounds, rValue, gValue, bValue);
             }
-            drawFaviconToCanvas(context, favicon, canvas, iconBounds, rValue, gValue, bValue);
             canvas.setBitmap(null);
         } catch (OutOfMemoryError e) {
             Log.w(TAG, "OutOfMemoryError while trying to draw bitmap on canvas.");
         }
         return bitmap;
     }
 
     /**
      * Get the icon background asset.
      * @param context Context used to create the intent.
@@ skipping 12 matching lines @@
         }
         if (drawable instanceof BitmapDrawable) {
             BitmapDrawable bd = (BitmapDrawable) drawable;
             return bd.getBitmap();
         }
         assert false : "The drawable was not a bitmap drawable as expected";
         return null;
     }
 
     /**
+     * Use touch-icon or higher-resolution favicon and round the corners.
+     * @param context    Context used to get resources.
+     * @param touchIcon  Touch icon bitmap.
+     * @param canvas     Canvas that holds the touch icon.
+     * @param iconBounds Rectangle bounds needed for adding rounded corners of the touch icon.
+     */
+    private static void drawTouchIconToCanvas(
+            Context context, Bitmap touchIcon, Canvas canvas, Rect iconBounds) {
+        Rect src = new Rect(0, 0, touchIcon.getWidth(), touchIcon.getHeight());
+        Paint paint = new Paint(Paint.ANTI_ALIAS_FLAG);
+        paint.setFilterBitmap(true);
+        canvas.drawBitmap(touchIcon, src, iconBounds, paint);

[Ted C, 2013/02/26 22:44:42]

purely for my understanding, what is the benefit of the second draw instead of
using a clipping path?

[aruslan, 2013/02/27 00:18:09]

It doesn't look like there is a way to anti-alias clip paths/clip regions
(#include <disclaimer>).
The Romain Guy's solution (as brilliant as it is) seems to be a bit overkill in
this particular context.

Hence, the setXfermode.

+        int borderRadii =
+                context.getResources().getDimensionPixelSize(R.dimen.touchicon_border_radii);
+        Path path = new Path();
+        path.setFillType(Path.FillType.INVERSE_WINDING);
+        RectF rect = new RectF(iconBounds);
+        rect.inset(INSET_DIMENSION_FOR_TOUCHICON, INSET_DIMENSION_FOR_TOUCHICON);
+        path.addRoundRect(rect, borderRadii, borderRadii, Path.Direction.CW);
+        paint.setXfermode(new PorterDuffXfermode(PorterDuff.Mode.CLEAR));
+        canvas.drawPath(path, paint);
+    }
+
+    /**
      * Draw the favicon with dominant color.
      * @param context Context used to create the intent.
      * @param favicon favicon bitmap.
      * @param canvas Canvas that holds the favicon.
      * @param iconBounds Rectangle bounds needed to create the homescreen favicon.
      * @param rValue Red component of the dominant favicon color.
      * @param gValue Green component of the dominant favicon color.
      * @param bValue Blue component of the dominant favicon color.
      */
     private static void drawFaviconToCanvas(Context context, Bitmap favicon,
@@ skipping 67 matching lines @@
             Bitmap scaledFavicon = Bitmap.createScaledBitmap(favicon, faviconSize, faviconSize,
                     true);
             canvas.drawBitmap(scaledFavicon,
                     iconBounds.exactCenterX() - scaledFavicon.getWidth() / 2.0f,
                     iconBounds.exactCenterY() - scaledFavicon.getHeight() / 2.0f, null);
         } catch (OutOfMemoryError e) {
             Log.w(TAG, "OutOfMemoryError while trying to draw bitmap on canvas.");
         }
     }
 }