Revert of [telemetry] Implement per-pixel algorithms in Bitmap as a C++ extension.

Revert of https://codereview.chromium.org/108333004/
Reason for revert: Fails on ChromiumOS on main waterfall.

TBR=tonyg@chromium.org,bulach@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=323813

[szym, 2013-12-17 00:35:42]

Created Revert of [telemetry] Implement per-pixel algorithms in Bitmap as a C++
extension.

[commit-bot: I haz the power, 2013-12-17 00:40:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/szym@chromium.org/116903002/1

[commit-bot: I haz the power, 2013-12-17 00:40:34]

Failed to apply patch for tools/telemetry/telemetry/core/bitmap_unittest.py:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file tools/telemetry/telemetry/core/bitmap_unittest.py
  patch unexpectedly ends in middle of line

Patch:       tools/telemetry/telemetry/core/bitmap_unittest.py
Index: tools/telemetry/telemetry/core/bitmap_unittest.py
diff --git a/tools/telemetry/telemetry/core/bitmap_unittest.py
b/tools/telemetry/telemetry/core/bitmap_unittest.py
index
912aa3cdba6a346e919b8fb0d6a5864fdcc5e6c0..618faa018f43c1a17d102fa1c28caa6def7188d4
100644
--- a/tools/telemetry/telemetry/core/bitmap_unittest.py
+++ b/tools/telemetry/telemetry/core/bitmap_unittest.py
@@ -70,8 +70,6 @@
   def testIsEqual(self):
     bmp = bitmap.Bitmap.FromBase64Png(test_png)
     file_bmp = bitmap.Bitmap.FromPngFile(test_png_path)
-    self.assertTrue(bmp.IsEqual(file_bmp, tolerance=1))
-    # Zero tolerance IsEqual has a different implementation.
     self.assertTrue(bmp.IsEqual(file_bmp))
 
   def testDiff(self):
@@ -104,19 +102,6 @@
     diff_bmp.GetPixelColor(2, 1).AssertIsRGB(255, 255, 255)
     diff_bmp.GetPixelColor(2, 2).AssertIsRGB(255, 255, 255)
 
-  def testGetBoundingBox(self):
-    pixels = [0,0,0, 0,0,0, 0,0,0, 0,0,0,
-              0,0,0, 1,0,0, 1,0,0, 0,0,0,
-              0,0,0, 0,0,0, 0,0,0, 0,0,0]
-    bmp = bitmap.Bitmap(3, 4, 3, pixels)
-    box, count = bmp.GetBoundingBox(bitmap.RgbaColor(1, 0, 0))
-    self.assertEquals(box, (1, 1, 2, 1))
-    self.assertEquals(count, 2)
-
-    box, count = bmp.GetBoundingBox(bitmap.RgbaColor(0, 1, 0))
-    self.assertEquals(box, None)
-    self.assertEquals(count, 0)
-
   def testCrop(self):
     pixels = [0,0,0, 0,0,0, 0,0,0, 0,0,0,
               0,0,0, 1,0,0, 1,0,0, 0,0,0,
@@ -129,18 +114,3 @@
     bmp.GetPixelColor(0, 0).AssertIsRGB(1, 0, 0)
     bmp.GetPixelColor(1, 0).AssertIsRGB(1, 0, 0)
     self.assertEquals(bmp.pixels, bytearray([1,0,0, 1,0,0]))
-
-  def testHistogram(self):
-    pixels = [1,2,3, 1,2,3, 1,2,3, 1,2,3,
-              1,2,3, 8,7,6, 5,4,6, 1,2,3,
-              1,2,3, 8,7,6, 5,4,6, 1,2,3]
-    bmp = bitmap.Bitmap(3, 4, 3, pixels)
-    bmp.Crop(1, 1, 2, 2)
-
-    histogram = bmp.ColorHistogram()
-    self.assertEquals(sum(histogram), bmp.width * bmp.height * 3)
-    self.assertEquals(histogram[5], 2)
-    self.assertEquals(histogram[8], 2)
-    self.assertEquals(histogram[4 + 256], 2)
-    self.assertEquals(histogram[7 + 256], 2)
-    self.assertEquals(histogram[6 + 512], 4)

[szym, 2013-12-17 00:55:14]

rmistry: This one has issues both with CQ and with "patchset too large to
download".

[rmistry, 2013-12-17 01:55:40]

On 2013/12/17 00:55:14, szym wrote:
> rmistry: This one has issues both with CQ and with "patchset too large to
> download".

Note to self:
"patch unexpectedly ends in middle of line" is due to-
This error seems to occur when the patch file ends on a line that includes
nothing but whitespace characters. Patch files should always end with a Unix
return character. If you encounter this error, check to see that the last line
of your patch does not contain whitespace characters.

"patchset too large to download":
This needs to be investigated, seems to not cause revert failures and can see
individual patches, but cannot see the complete patch due to do this.