Bug 1324028 - IconGenerator: Do not add alpha channel to colors of generated icons. r?mcomella draft
authorSebastian Kaspari <s.kaspari@gmail.com>
Tue, 08 Aug 2017 17:36:50 +0200
changeset 642609 5b88422d4c3ffe483529af06b56c21b63b0a7103
parent 642605 8339104adceec24f681cd2f84a56a524cb5bd8d2
child 725056 c8b2a92487670bd7cf0f8c0a0014ec357b788395
push id72821
push users.kaspari@gmail.com
push dateTue, 08 Aug 2017 15:37:53 +0000
reviewersmcomella
bugs1324028
milestone57.0a1
Bug 1324028 - IconGenerator: Do not add alpha channel to colors of generated icons. r?mcomella MozReview-Commit-ID: H4qaSSOlMxT
mobile/android/base/java/org/mozilla/gecko/icons/loader/IconGenerator.java
--- a/mobile/android/base/java/org/mozilla/gecko/icons/loader/IconGenerator.java
+++ b/mobile/android/base/java/org/mozilla/gecko/icons/loader/IconGenerator.java
@@ -82,17 +82,17 @@ public class IconGenerator implements Ic
         paint.setTextSize(textSize);
         paint.setAntiAlias(true);
 
         canvas.drawText(character,
                 canvas.getWidth() / 2,
                 (int) ((canvas.getHeight() / 2) - ((paint.descent() + paint.ascent()) / 2)),
                 paint);
 
-        return IconResponse.createGenerated(favicon, color & 0x7FFFFFFF);
+        return IconResponse.createGenerated(favicon, color);
     }
 
     /**
      * Get a representative character for the given URL.
      *
      * For example this method will return "f" for "http://m.facebook.com/foobar".
      */
     @VisibleForTesting static String getRepresentativeCharacter(String url) {