From 49e418c64adaead125df239a2c348b800c2af935 Mon Sep 17 00:00:00 2001
From: Sebastian Hengst <archaeopteryx@coole-files.de>
Date: Mon, 28 Aug 2017 16:48:57 +0200
Subject: [PATCH] Backed out changeset 99e579176288 (bug 1324028) for failing
 'test' suite and on request from sebastian. r=backout

---
 .../icons/processing/ColorProcessor.java      | 72 -------------------
 1 file changed, 72 deletions(-)

diff --git a/mobile/android/base/java/org/mozilla/gecko/icons/processing/ColorProcessor.java b/mobile/android/base/java/org/mozilla/gecko/icons/processing/ColorProcessor.java
index 96c93a327813..b3cda1f8aaf3 100644
--- a/mobile/android/base/java/org/mozilla/gecko/icons/processing/ColorProcessor.java
+++ b/mobile/android/base/java/org/mozilla/gecko/icons/processing/ColorProcessor.java
@@ -5,8 +5,6 @@
 
 package org.mozilla.gecko.icons.processing;
 
-import android.graphics.Bitmap;
-import android.support.annotation.ColorInt;
 import android.support.v7.graphics.Palette;
 import android.util.Log;
 
@@ -29,14 +27,6 @@ public class ColorProcessor implements Processor {
             return;
         }
 
-        final Bitmap bitmap = response.getBitmap();
-
-        final @ColorInt Integer edgeColor = getEdgeColor(bitmap);
-        if (edgeColor != null) {
-            response.updateColor(edgeColor);
-            return;
-        }
-
         if (HardwareUtils.isX86System()) {
             // (Bug 1318667) We are running into crashes when using the palette library with
             // specific icons on x86 devices. They take down the whole VM and are not recoverable.
@@ -68,66 +58,4 @@ public class ColorProcessor implements Processor {
 
         response.updateColor(dominantColor);
     }
-
-    /**
-     * If a bitmap has a consistent edge colour (i.e. if all the border pixels have the same colour),
-     * return that colour.
-     * @param bitmap
-     * @return The edge colour. null if there is no consistent edge color.
-     */
-    private @ColorInt Integer getEdgeColor(final Bitmap bitmap) {
-        final int width = bitmap.getWidth();
-        final int height = bitmap.getHeight();
-
-        // Only allocate an array once, with the max width we need once, to minimise the number
-        // of allocations.
-        @ColorInt int[] edge = new int[Math.max(width, height)];
-
-        // Top:
-        bitmap.getPixels(edge, 0, width, 0, 0, width, 1);
-        final @ColorInt Integer edgecolor = getEdgeColorFromSingleDimension(edge, width);
-        if (edgecolor == null) {
-            return null;
-        }
-
-        // Bottom:
-        bitmap.getPixels(edge, 0, width, 0, height - 1, width, 1);
-        if (!edgecolor.equals(getEdgeColorFromSingleDimension(edge, width))) {
-            return null;
-        }
-
-        // Left:
-        bitmap.getPixels(edge, 0, 1, 0, 0, 1, height);
-        if (!edgecolor.equals(getEdgeColorFromSingleDimension(edge, height))) {
-            return null;
-        }
-
-        // Right:
-        bitmap.getPixels(edge, 0, 1, width - 1, 0, 1, height);
-        if (!edgecolor.equals(getEdgeColorFromSingleDimension(edge, height))) {
-            return null;
-        }
-
-        return edgecolor;
-    }
-
-    /**
-     * Obtain the colour for a given edge if all colors are the same.
-     *
-     * @param edge An array containing the color values of the pixels constituting the edge of a bitmap.
-     * @param length The length of the array to be traversed. Must be smaller than, or equal to
-     * the total length of the array.
-     * @return The colour contained within the array, or null if colours vary.
-     */
-    private @ColorInt Integer getEdgeColorFromSingleDimension(@ColorInt int[] edge, int length) {
-        @ColorInt int color = edge[0];
-
-        for (int i = 1; i < length; ++i) {
-            if (edge[i] != color) {
-                return null;
-            }
-        }
-
-        return color;
-    }
 }