Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(255)

Unified Diff: client/tests/client/html/CanvasPixelArrayTypeAliasTest.dart

Issue 10207033: Make CanvasPixelArray pretent to be Uint8ClampedArray (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: client/tests/client/html/CanvasPixelArrayTypeAliasTest.dart
diff --git a/client/tests/client/html/CanvasPixelArrayTypeAliasTest.dart b/client/tests/client/html/CanvasPixelArrayTypeAliasTest.dart
new file mode 100644
index 0000000000000000000000000000000000000000..96353a4e6da2127b4cfdc9654e2daa7a73df23e9
--- /dev/null
+++ b/client/tests/client/html/CanvasPixelArrayTypeAliasTest.dart
@@ -0,0 +1,62 @@
+#library('CanvasTest');
+#import('../../../../lib/unittest/unittest.dart');
+#import('../../../../lib/unittest/html_config.dart');
+#import('dart:html');
+
+// 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];
+
+main() {
+ CanvasElement canvas;
+ CanvasRenderingContext2D context;
+ int width = 100;
+ int height = 100;
+
+ canvas = new Element.tag('canvas');
+ canvas.width = width;
+ canvas.height = height;
+ document.body.nodes.add(canvas);
+
+ context = canvas.getContext('2d');
+
+ useHtmlConfiguration();
+
+ 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', () {
+ 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]);
+ }
+}

Powered by Google App Engine
This is Rietveld 408576698