Bug 1389343 - Make default background grey so it can occupied the entire favicon. draft
authorNevin Chen <cnevinchen@gmail.com>
Mon, 04 Sep 2017 13:36:47 +0800
changeset 667610 ade1c0857eaabe5377b71f4392d5db0a61b22321
parent 658358 8e05298328da75f3056a9f1f9609938870d756a0
child 732441 42fd719c2193767afea84a8810e24f4799ccc34f
push id80773
push userbmo:cnevinchen@gmail.com
push dateWed, 20 Sep 2017 11:23:34 +0000
bugs1389343
milestone57.0a1
Bug 1389343 - Make default background grey so it can occupied the entire favicon. MozReview-Commit-ID: 93ZWdpnlIuX
mobile/android/base/java/org/mozilla/gecko/icons/processing/ColorProcessor.java
--- a/mobile/android/base/java/org/mozilla/gecko/icons/processing/ColorProcessor.java
+++ b/mobile/android/base/java/org/mozilla/gecko/icons/processing/ColorProcessor.java
@@ -16,17 +16,17 @@ import org.mozilla.gecko.icons.IconRespo
 import org.mozilla.gecko.util.HardwareUtils;
 
 /**
  * Processor implementation to extract the dominant color from the icon and attach it to the icon
  * response object.
  */
 public class ColorProcessor implements Processor {
     private static final String LOGTAG = "GeckoColorProcessor";
-    private static final int DEFAULT_COLOR = 0; // 0 == No color
+    private static final int DEFAULT_COLOR = 0xFFB1B1B3; // 0 == No color, here we use photon color
 
     @Override
     public void process(IconRequest request, IconResponse response) {
         if (response.hasColor()) {
             return;
         }
 
         final Bitmap bitmap = response.getBitmap();