[Telemetry] Migrate bitmap.py from bitmaptools.cc to numpy

tools/telemetry/telemetry/core/video.py

Index: tools/telemetry/telemetry/core/video.py
diff --git a/tools/telemetry/telemetry/core/video.py b/tools/telemetry/telemetry/core/video.py
index 4eb4cab3bbcf3714d1e69d18644de58d5c24ecd2..d7bf3e9139311ca028505bf62950e4663c20e984 100644
--- a/tools/telemetry/telemetry/core/video.py
+++ b/tools/telemetry/telemetry/core/video.py
@@ -18,8 +18,6 @@ class Video(object):
   """Utilities for storing and interacting with the video capture."""
 
   def __init__(self, video_file_obj):
-    assert video_file_obj.delete
-    assert not video_file_obj.close_called

[tonyg, 2014/10/21 16:12:57]

Were these asserts incorrect?

[mthiesse, 2014/10/22 15:33:10]

Yeah, these are bizarre assertions, because the video_file_obj passed is just a
string. Removing them allows the test to pass. Is this why it was disabled?

At any rate, I'll re-enable the test because it passes now.

     self._video_file_obj = video_file_obj
     self._tab_contents_bounding_box = None
 
@@ -166,4 +164,5 @@ class Video(object):
         raise StopIteration
       assert num_read == len(frame_data), 'Unexpected frame size: %d' % num_read
       yield (GetFrameTimestampMs(proc.stderr),
-             bitmap.Bitmap(3, dimensions[0], dimensions[1], frame_data))
+             bitmap.Bitmap.FromRGBPixels(dimensions[0], dimensions[1],
+                                         frame_data, 3))