Make CanvasPixelArray pretent to be Uint8ClampedArray

client/tests/client/dom/CanvasPixelArrayTypeAliasTest.dart

Index: client/tests/client/dom/CanvasPixelArrayTypeAliasTest.dart
diff --git a/client/tests/client/dom/CanvasPixelArrayTypeAliasTest.dart b/client/tests/client/dom/CanvasPixelArrayTypeAliasTest.dart
new file mode 100644
index 0000000000000000000000000000000000000000..f3a44be21afd8eba469779e1060617b56a1645a1
--- /dev/null
+++ b/client/tests/client/dom/CanvasPixelArrayTypeAliasTest.dart
@@ -0,0 +1,62 @@
+#library('CanvasTest');
+#import('../../../../lib/unittest/unittest.dart');
+#import('../../../../lib/unittest/dom_config.dart');
+#import('dart:dom');
+
+// We have aliased the legacy type CanvasPixelArray with the new type
+// Uint8ClampedArray by mapping the CanvasPixelArray type tag to
+// Uint8ClampedArray.  It is not a perfect match since CanvasPixelArray is
+// missing the ArrayBufferView members.  These should appear to be null.
+
+Object confuseType(x) => [1, x, [x], 's'] [1];  // returns 'x'
+
+main() {
+  HTMLCanvasElement canvas;
+  CanvasRenderingContext2D context;
+  int width = 100;
+  int height = 100;
+
+  canvas = document.createElement('canvas');
+  canvas.setAttribute('width', '$width');
+  canvas.setAttribute('height', '$height');
+  document.body.appendChild(canvas);
+
+  context = canvas.getContext('2d');
+
+  useDomConfiguration();
+
+  test('CreateImageData', () {
+    ImageData image = context.createImageData(canvas.width,
+                                              canvas.height);
+    Uint8ClampedArray data = image.data;
+    // It is legal for the dart2js compiler to believe the type of the native
+    // ImageData.data and elides the check, so check the type explicitly:
+    Expect.isTrue(confuseType(data) is Uint8ClampedArray);
+
+    Expect.equals(40000, data.length);
+    checkPixel(data, 0, [0, 0, 0, 0]);
+    checkPixel(data, width * height - 1, [0, 0, 0, 0]);
+
+    data[100] = 200;
+    Expect.equals(200, data[100]);
+  });
+
+  test('PutImageData', () {

[vsm, 2012/04/25 05:38:18]

Isn't this one redundant with the one in CanvasTest?  I don't see any changes
here.

+    ImageData data = context.getImageData(0, 0, width, height);
+    data.data[0] = 25;
+    data.data[3] = 255;
+    context.putImageData(data, 0, 0);
+
+    data = context.getImageData(0, 0, width, height);
+    Expect.equals(25, data.data[0]);
+    Expect.equals(255, data.data[3]);
+  });
+}
+
+void checkPixel(Uint8ClampedArray data, int offset, List<int> rgba)
+{
+  offset *= 4;
+  for (var i = 0; i < 4; ++i) {
+    Expect.equals(rgba[i], data[offset + i]);
+  }
+}