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

Unified Diff: third_party/WebKit/LayoutTests/fast/canvas/resources/canvas-Float32ImageData.js

Issue 2636533002: Revert of Implement color management for ImageData
Patch Set: Created 3 years, 11 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: third_party/WebKit/LayoutTests/fast/canvas/resources/canvas-Float32ImageData.js
diff --git a/third_party/WebKit/LayoutTests/fast/canvas/resources/canvas-Float32ImageData.js b/third_party/WebKit/LayoutTests/fast/canvas/resources/canvas-Float32ImageData.js
deleted file mode 100644
index 838297b06a8f180ee18909bc181961251f063078..0000000000000000000000000000000000000000
--- a/third_party/WebKit/LayoutTests/fast/canvas/resources/canvas-Float32ImageData.js
+++ /dev/null
@@ -1,121 +0,0 @@
-function setF16Color(float32ImageData, i, f16Color) {
- var s = i * 4;
- float32ImageData[s] = f16Color[0];
- float32ImageData[s + 1] = f16Color[1];
- float32ImageData[s + 2] = f16Color[2];
- float32ImageData[s + 3] = f16Color[3];
-}
-
-function getF16Color(float32ImageData, i) {
- var result = [];
- var s = i * 4;
- for (var j = 0; j < 4; j++) {
- result[j] = float32ImageData[s + j];
- }
- return result;
-}
-
-function compareF16Colors(f16Color1, f16Color2) {
- for (var j = 0; j < 4; j++)
- if (f16Color1[j] != f16Color2[j])
- return false;
- return true;
-}
-
-test(function() {
-
- float32ImageData = new Float32ImageData(100, 50);
- assert_equals(float32ImageData.width, 100,
- "The width of the float32ImageData should be the width we passed to the constructor.");
- assert_equals(float32ImageData.height, 50,
- "The height of the float32ImageData should be the height we passed to the constructor.");
-
- var f16ColorData = getF16Color(float32ImageData.data, 4);
- assert_equals(f16ColorData[0], 0,
- "The default ImageData color is transparent black.");
- assert_equals(f16ColorData[1], 0,
- "The default ImageData color is transparent black.");
- assert_equals(f16ColorData[2], 0,
- "The default ImageData color is transparent black.");
- assert_equals(f16ColorData[3], 0,
- "The default ImageData color is transparent black.");
-
- var testColor = new Float32Array([0x00003c00, 0x00003c01, 0x00003c02, 0x00003c03]);
- setF16Color(float32ImageData.data, 4, testColor);
- f16ColorData = getF16Color(float32ImageData.data, 4);
- assert_equals(f16ColorData[0], 0x00003c00,
- "The red component of f16 color is the value we set.");
- assert_equals(f16ColorData[1], 0x00003c01,
- "The green component of f16 color is the value we set.");
- assert_equals(f16ColorData[2], 0x00003c02,
- "The blue component of f16 color is the value we set.");
- assert_equals(f16ColorData[3], 0x00003c03,
- "The alpha component of f16 color is the value we set.");
-
- assert_throws(null, function() {new Float32ImageData(10);},
- "Float32ImageData constructor requires both width and height.");
- assert_throws("IndexSizeError", function() {new Float32ImageData(0,10);},
- "Float32ImageData width must be greater than zero.");
- assert_throws("IndexSizeError", function() {new Float32ImageData(10,0);},
- "Float32ImageData height must be greater than zero.");
- assert_throws("IndexSizeError", function() {new Float32ImageData('width', 'height');},
- "Float32ImageData width and height must be numbers.");
- assert_throws("IndexSizeError", function() {new Float32ImageData(1 << 31, 1 << 31);},
- "Float32ImageData width multiplied by height must be less than 2^30 to avoid buffer size overflow.");
-
- assert_throws(null, function() {new Float32ImageData(new Float32Array(0));},
- "The Float32Array passed to the constructor cannot have a length of zero.");
- assert_throws("IndexSizeError", function() {new Float32ImageData(new Float32Array(27), 2);},
- "The size of Float32Array passed to the constructor must be divisible by 4 * width.");
- assert_throws("IndexSizeError", function() {new Float32ImageData(new Float32Array(28), 7, 0);},
- "The size of Float32Array passed to the constructor must be equal to 4 * width * height.");
- assert_throws(null, function() {new Float32ImageData(self, 4, 4);},
- "The object passed to the constructor as data array should be of type Float32Array.");
- assert_throws(null, function() {new Float32ImageData(null, 4, 4);},
- "The object passed to the constructor as data array should be of type Float32Array.");
- assert_throws("IndexSizeError", function() {new Float32ImageData(float32ImageData.data, 0);},
- "The size of Float32Array passed to the constructor must be divisible by 4 * width.");
- assert_throws("IndexSizeError", function() {new Float32ImageData(float32ImageData.data, 13);},
- "The size of Float32Array passed to the constructor must be divisible by 4 * width.");
- assert_throws("IndexSizeError", function() {new Float32ImageData(float32ImageData.data, 1 << 31);},
- "The size of Float32Array passed to the constructor must be divisible by 4 * width.");
- assert_throws("IndexSizeError", function() {new Float32ImageData(float32ImageData.data, 'biggish');},
- "The width parameter must be a number.");
- assert_throws("IndexSizeError", function() {new Float32ImageData(float32ImageData.data, 1 << 24, 1 << 31);},
- "Float32ImageData width multiplied by height must be less than 2^30 to avoid buffer size overflow.");
- assert_throws("IndexSizeError", function() {new Float32ImageData(float32ImageData.data, 1 << 31, 1 << 24);},
- "Float32ImageData width multiplied by height must be less than 2^30 to avoid buffer size overflow.");
- assert_equals((new Float32ImageData(new Float32Array(28), 7)).height, 1,
- "The height must be equal to size of the data array divided by 4 * width.");
-
- float32ImageDataFromData = new Float32ImageData(float32ImageData.data, 100);
- assert_equals(float32ImageDataFromData.width, 100,
- "The width of the float32ImageDataFromData should be the same as that of float32ImageData.");
- assert_equals(float32ImageDataFromData.height, 50,
- "The height of the float32ImageDataFromData should be the same as that of float32ImageData.");
- assert_true(float32ImageDataFromData.data == float32ImageData.data,
- "The pixel data buffer of float32ImageDataFromData should be the same as that of float32ImageData.");
- assert_true(compareF16Colors(getF16Color(float32ImageDataFromData.data, 10),
- getF16Color(float32ImageData.data, 10)),
- "The color of a pixel from float32ImageDataFromData should be the same as that of float32ImageData.");
- setF16Color(float32ImageData.data, 10, testColor);
- assert_true(compareF16Colors(getF16Color(float32ImageDataFromData.data, 10),
- getF16Color(float32ImageData.data, 10)),
- "Setting the color of a pixel from float32ImageData must cascade to the same pixel of float32ImageDataFromData.");
-
- var data = new Float32Array(400);
- data[22] = 129;
- float32ImageDataFromData = new Float32ImageData(data, 20, 5);
- assert_equals(float32ImageDataFromData.width, 20,
- "The width of the float32ImageData should be the width we passed to the constructor.");
- assert_equals(float32ImageDataFromData.height, 5,
- "The height of the Float32ImageData must be equal to size of the Float32Array divided by 4 * width.");
- assert_true(float32ImageDataFromData.data == data,
- "The pixel data buffer of float32ImageDataFromData should be the same buffer passed to the constructor.");
- assert_true(compareF16Colors(getF16Color(float32ImageDataFromData.data, 2), getF16Color(data, 2)),
- "The pixel data of float32ImageDataFromData should be the same as that of the buffer passed to the constructor.");
- setF16Color(float32ImageDataFromData.data, 2, testColor);
- assert_true(compareF16Colors(getF16Color(float32ImageDataFromData.data, 2), getF16Color(data, 2)),
- "Setting the color of a pixel from float32ImageDataFromData must cascade to the same pixel of the buffer passed to the constructor.");
-
-}, 'This test examines the correct behavior of Float32ImageData API.');

Powered by Google App Engine
This is Rietveld 408576698